Thanks for the review.

On Tue, Jun 14, 2011 at 09:54, Ben Pfaff <b...@nicira.com> wrote:
> On Mon, Jun 13, 2011 at 01:49:59PM -0700, Ethan Jackson wrote:
>> The lacp_slave_is_current() function is used to indicate to a
>> controller failover status of the LACP module.  However, the result
>> of this function is more strict than the failover logic.  Thus, the
>> function will generally return false quite a bit before a failover
>> actually happens.  This patch loosens lacp_slave_is_current() so
>> that it changes in-line with the failover logic.
>
> Looks good to me.  Thank you!
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to