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. --- lib/lacp.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/lacp.c b/lib/lacp.c index 3fe5eff..eaf01c3 100644 --- a/lib/lacp.c +++ b/lib/lacp.c @@ -347,7 +347,7 @@ lacp_slave_get_port_id(const struct lacp *lacp, const void *slave_) bool lacp_slave_is_current(const struct lacp *lacp, const void *slave_) { - return slave_lookup(lacp, slave_)->status == LACP_CURRENT; + return slave_lookup(lacp, slave_)->status != LACP_DEFAULTED; } /* This function should be called periodically to update 'lacp'. */ -- 1.7.5.2 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev