Re: [ovs-dev] [bond 6/7] lacp: Require successful LACP negotiations when configured.

2012-01-16 Thread Ben Pfaff
I agree that this is much easier to understand. It looks good to me. Thank you. On Fri, Jan 13, 2012 at 11:56:49AM -0800, Ethan Jackson wrote: > Here's the original version of the patch that I'd written. I had remvoed the > LACP_CONFIGURED enum because I don't think it's strictly necessary. The

[ovs-dev] [bond 6/7] lacp: Require successful LACP negotiations when configured.

2012-01-13 Thread Ethan Jackson
Here's the original version of the patch that I'd written. I had remvoed the LACP_CONFIGURED enum because I don't think it's strictly necessary. The same behavior can be achieved by dropping all traffic on non-enabled slaves in the admissibility function. However, I think this patch is *much* ea