From: Ethan Jackson <ethan@nicira.com>
Date: Wed, 9 Mar 2011 01:20:01 +0000 (-0800)
Subject: vswitchd: Update lacp before bonding.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=afabe15a7aea7a588ea1711ce00bb524fe1733c2;p=openvswitch

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 47d4bdd7..c6244331 100644
--- 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);