On Fri, Apr 15, 2011 at 01:49:19PM -0700, Ethan Jackson wrote:
> Whether or not a given slave is current with its LACP protocol
> messages can be very interesting to a controller.  If an interface
> is not current, it usually indicates a connectivity problem or
> misconfiguration of some sort.

There is a missing space before the '=' in the initializer for
db_current in iface_refresh_lacp_stats().

In vswitch.xml, I would add a sentence something like, "This column
will be empty if LACP is not enabled."

Otherwise this looks good.  Thank you!
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to