[ovs-dev] [cfm 7/7] bond: Use CFM for slave status.

Ben Pfaff blp at nicira.com
Fri May 20 18:21:43 UTC 2011


On Thu, May 19, 2011 at 06:53:26PM -0700, Ethan Jackson wrote:
> With this patch, if CFM is configured on a slave, and in a faulted
> state, it will be disabled by the bond module.

I think that the comment on bond_slave_set_may_enable() should be
updated to mention reasons why a slave would be enabled or disabled.

The comment on the 'may_enable' member is now a bit stale, although I
don't know how to make it accurate and still fit on one line.  Maybe
it should stay as-is.

Should we update vswitch.xml?

Otherwise looks good.



More information about the dev mailing list