vswitchd: Update lacp before bonding.

Bonding requires lacp attached status to make decisions about which
links are enabled.  However, LACP does not require any bonding
related information (other than which links are aggregated) to
perform.  Having them reversed causes some subtle bugs in the
bonding code's responsiveness to LACP changes.

diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index 47d4bdd..c624433 100644 (file)
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -923,8 +923,8 @@ bridge_reconfigure(const struct ovsrec_open_vswitch *ovs_cfg)
                 port->miimon_next_update = 0;
             }
 
-            port_update_bonding(port);
             port_update_lacp(port);
+            port_update_bonding(port);
 
             for (j = 0; j < port->n_ifaces; j++) {
                 iface_update_qos(port->ifaces[j], port->cfg->qos);