From: Ethan Jackson <ethan@nicira.com>
Date: Mon, 13 Jun 2011 20:45:05 +0000 (-0700)
Subject: lacp: Loosen lacp_slave_is_current().
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b6466afa85a9960f9424089f41c1a6e14009c994;p=openvswitch

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.
---

diff --git a/lib/lacp.c b/lib/lacp.c
index 3fe5eff3..eaf01c3c 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'. */