On Fri, Mar 02, 2012 at 02:23:56PM -0800, Ethan Jackson wrote:
> This patch shouldn't actually affect the behavior of LACP bonds
> because the bond module won't choose to send traffic out a slave
> whose carrier is down.
> 
> Signed-off-by: Ethan Jackson <et...@nicira.com>

I see where port->carrier_seq gets initialized but not where it gets
updated.

I'd prefer that the commit message explain the user-observable change
that this commit causes.  When we talked about it in person, I think
you said that it just makes "ovs-appctl lacp/show" output make a bit
more sense.

Thanks,

Ben.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to