Re: [ovs-dev] [PATCH] ofproto/bond: Add knob "all_slaves_active"

2022-03-24 Thread Christophe Fontaine
On Wed, Mar 23, 2022 at 10:12 PM Mike Pattrick wrote: > > On Thu, Mar 17, 2022 at 11:54 AM Christophe Fontaine > wrote: > > > > This config param allows the delivery of broadcast and multicast packets > > to the secondary interface of non-lacp bonds, identical to the same option > > for kernel

Re: [ovs-dev] [PATCH] ofproto/bond: Add knob "all_slaves_active"

2022-03-23 Thread Mike Pattrick
On Thu, Mar 17, 2022 at 11:54 AM Christophe Fontaine wrote: > > This config param allows the delivery of broadcast and multicast packets > to the secondary interface of non-lacp bonds, identical to the same option > for kernel bonds. > > Tested with an ovs-dpdk balance-slb bond with 2 virtual

[ovs-dev] [PATCH] ofproto/bond: Add knob "all_slaves_active"

2022-03-22 Thread Giacon Jr, Guilherme via dev
I would like to request the implementation of this patch, with this patch I will be able to implement my use case on OpenStack. Thanks Gui From: Christophe Fontaine Date: Thu, Mar 17, 2022 at 4:54 PM Subject: [PATCH] ofproto/bond: Add knob "all_slaves_active" To: Cc: Christophe Fontaine

Re: [ovs-dev] [PATCH] ofproto/bond: Add knob "all_slaves_active"

2022-03-17 Thread 0-day Robot
Bleep bloop. Greetings Christophe Fontaine, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 81 characters long (recommended limit is 79) #115 FILE:

[ovs-dev] [PATCH] ofproto/bond: Add knob "all_slaves_active"

2022-03-17 Thread Christophe Fontaine
This config param allows the delivery of broadcast and multicast packets to the secondary interface of non-lacp bonds, identical to the same option for kernel bonds. Tested with an ovs-dpdk balance-slb bond with 2 virtual functions, and a kernel bond with LACP on 2 other virtual functions. Scapy