bond: Create new 'stable_id' parameter.
authorEthan Jackson <ethan@nicira.com>
Fri, 15 Apr 2011 00:37:29 +0000 (17:37 -0700)
committerEthan Jackson <ethan@nicira.com>
Fri, 15 Apr 2011 18:15:57 +0000 (11:15 -0700)
For BM_STABLE bonds, instead of choosing the sort key in the
qsort() comparator, this patch makes it a configuration setting of
each slave.  This will help wrest LACP out of the bonding code
further in future patches.

lib/bond.c
lib/bond.h
vswitchd/bridge.c

index 3ec1ba2075ed793225cd6b7f1045397d70b0d4fe..da828557262cea95deab9100423ecd18a3a1dad8 100644 (file)
@@ -74,6 +74,7 @@ struct bond_slave {
     uint64_t tx_bytes;          /* Sum across 'tx_bytes' of entries. */
 
     /* BM_STABLE specific bonding info. */
+    uint16_t stb_id;            /* ID used for 'stb_slaves' ordering. */
     size_t stb_idx;             /* Index in 'bond''s 'stb_slaves' array.
                                    Undefined value if participating in a
                                    BTM_STABLE bond or not enabled. */
@@ -357,6 +358,10 @@ bond_reconfigure(struct bond *bond, const struct bond_settings *s)
  * bond.  If 'slave_' already exists within 'bond' then this function
  * reconfigures the existing slave.
  *
+ * 'stb_id' is used in BM_STABLE bonds to guarantee consistent slave choices
+ * across restarts and distributed vswitch instances.  It should be unique per
+ * slave, and preferably consistent across restarts and reconfigurations.
+ *
  * 'netdev' must be the network device that 'slave_' represents.  It is owned
  * by the client, so the client must not close it before either unregistering
  * 'slave_' or destroying 'bond'.
@@ -364,7 +369,8 @@ bond_reconfigure(struct bond *bond, const struct bond_settings *s)
  * If 'bond' has a LACP configuration then 'slave_' should be the same handle
  * used in the LACP module. */
 void
-bond_slave_register(struct bond *bond, void *slave_, struct netdev *netdev)
+bond_slave_register(struct bond *bond, void *slave_, uint16_t stb_id,
+                    struct netdev *netdev)
 {
     struct bond_slave *slave = bond_slave_lookup(bond, slave_);
 
@@ -380,6 +386,11 @@ bond_slave_register(struct bond *bond, void *slave_, struct netdev *netdev)
         bond_enable_slave(slave, slave->up, NULL);
     }
 
+    if (slave->stb_id != stb_id) {
+        bond->stb_need_sort = true;
+        slave->stb_id = stb_id;
+    }
+
     slave->netdev = netdev;
     free(slave->name);
     slave->name = xstrdup(netdev_get_name(netdev));
@@ -1273,18 +1284,10 @@ bond_stb_sort_cmp__(const void *a_, const void *b_)
     const struct bond_slave *const *bp = b_;
     const struct bond_slave *a = *ap;
     const struct bond_slave *b = *bp;
-    const struct lacp *lacp = a->bond->lacp;
-    int a_id, b_id;
-
-    if (lacp) {
-        a_id = lacp_slave_get_port_id(lacp, a->aux);
-        b_id = lacp_slave_get_port_id(lacp, b->aux);
-    } else {
-        a_id = netdev_get_ifindex(a->netdev);
-        b_id = netdev_get_ifindex(b->netdev);
-    }
+    uint16_t aid = a->stb_id;
+    uint16_t bid = b->stb_id;
 
-    return (a_id == b_id ? 0 : (a_id < b_id ? -1 : 1));
+    return aid < bid ? -1 : aid > bid;
 }
 
 static bool
index cb6563af15ce3cb4ee399067475d3e0a53972d6e..e3690ceed214cb5662af9ba6a92b2825c7666743 100644 (file)
@@ -75,7 +75,8 @@ struct bond *bond_create(const struct bond_settings *);
 void bond_destroy(struct bond *);
 
 bool bond_reconfigure(struct bond *, const struct bond_settings *);
-void bond_slave_register(struct bond *, void *slave_, struct netdev *);
+void bond_slave_register(struct bond *, void *slave_,
+                         uint16_t stable_id, struct netdev *);
 void bond_slave_unregister(struct bond *, const void *slave);
 
 void bond_run(struct bond *, struct tag_set *);
index 9b918af5e371c89c9c55a4e5c422884d66367a0a..d0c43bcd44ce3b838c1b6457666040436673bfcd 100644 (file)
@@ -3174,7 +3174,11 @@ port_reconfigure_bond(struct port *port)
     }
 
     LIST_FOR_EACH (iface, port_elem, &port->ifaces) {
-        bond_slave_register(iface->port->bond, iface, iface->netdev);
+        uint16_t stable_id = (port->lacp
+                              ? lacp_slave_get_port_id(port->lacp, iface)
+                              : iface->dp_ifidx);
+        bond_slave_register(iface->port->bond, iface, stable_id,
+                            iface->netdev);
     }
 }