[ovs-dev] [PATCH 2/2] lacp: New "strict" lacp mode.

Ethan Jackson ethan at nicira.com
Tue Apr 19 17:34:28 UTC 2011


Thanks for the reviews, I merged this series.

On Tue, Apr 19, 2011 at 9:56 AM, Ben Pfaff <blp at nicira.com> wrote:
> On Mon, Apr 18, 2011 at 03:39:15PM -0700, Ethan Jackson wrote:
>> When LACP negotiations are unsuccessful, OVS falls back to standard
>> balance-slb bonding.  In some cases, users may want to require
>> successful LACP negotiations for any slaves to be enabled at all.
>> This patch implements a new "strict" mode which disables all slaves
>> when LACP negotiations are unsuccessful.
>
> There's a typo in the vswitch.xml update: switch(e)s.
>
> There's a pre-existing typo in the lacp_update_attached()
> function-level comment: ...controlled *by* 'lacp'...
>
> Otherwise looks good.  Thank you!
>



More information about the dev mailing list