[ovs-dev] [mii 2/2] bond: Remove useless slave->up flag.

[Ethan Jackson]

Theres no longer any reason not to simply call netdev_get_carrier().
---
 lib/bond.c |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/lib/bond.c b/lib/bond.c
index f5d0110..6a8d846 100644
--- a/lib/bond.c
+++ b/lib/bond.c
@@ -323,12 +323,11 @@ bond_slave_register(struct bond *bond, void *slave_, uint32_t stb_id,
         slave->bond = bond;
         slave->aux = slave_;
         slave->delay_expires = LLONG_MAX;
-        slave->up = netdev_get_carrier(netdev);
         slave->name = xstrdup(netdev_get_name(netdev));
         bond->bond_revalidate = true;
 
         slave->enabled = false;
-        bond_enable_slave(slave, slave->up, NULL);
+        bond_enable_slave(slave, netdev_get_carrier(netdev), NULL);
     }
 
     if (slave->stb_id != stb_id) {
@@ -419,11 +418,6 @@ bond_run(struct bond *bond, struct tag_set *tags, bool lacp_negotiated)
 
     bond->lacp_negotiated = lacp_negotiated;
 
-    /* Update link status. */
-    HMAP_FOR_EACH (slave, hmap_node, &bond->slaves) {
-        slave->up = netdev_get_carrier(slave->netdev);
-    }
-
     if (bond->monitor) {
         netdev_monitor_flush(bond->monitor);
     }
@@ -1278,7 +1272,7 @@ bond_link_status_update(struct bond_slave *slave, struct tag_set *tags)
     struct bond *bond = slave->bond;
     bool up;
 
-    up = slave->up && slave->lacp_may_enable;
+    up = netdev_get_carrier(slave->netdev) && slave->lacp_may_enable;
     if ((up == slave->enabled) != (slave->delay_expires == LLONG_MAX)) {
         static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 20);
         VLOG_INFO_RL(&rl, "interface %s: link state %s",
-- 
1.7.4.4