From b6466afa85a9960f9424089f41c1a6e14009c994 Mon Sep 17 00:00:00 2001 From: Ethan Jackson Date: Mon, 13 Jun 2011 13:45:05 -0700 Subject: [PATCH] lacp: Loosen lacp_slave_is_current(). 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 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/lacp.c b/lib/lacp.c index 3fe5eff32..eaf01c3c9 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'. */ -- 2.20.1