File 0002-bridge.c-prevent-controller-connects-while... of Package openvswitch-dpdk.10750

Overview Repositories Revisions Requests Users Attributes Meta

File 0002-bridge.c-prevent-controller-connects-while-flow-rest.patch of Package openvswitch-dpdk.10750

From 5760d3e36b7032415963f0aae4737fb55748907b Mon Sep 17 00:00:00 2001
From: Zak Whittington <zwhitt.vmware@gmail.com>
Date: Thu, 25 Oct 2018 11:09:09 -0700
Subject: [PATCH 02/14] bridge.c: prevent controller connects while
 flow-restore-wait

When force-reload-kmod is used, it shows an error when reinstalling
tlvs during "Restoring saved flows" step:
OFPT_ERROR (xid=0x4): NXTTMFC_ALREADY_MAPPED

This is caused by a race condition between the restore script,
which calls ofctl, and the connected controllers both adding back
the same TLVs.

The restore script already sets flow-restore-wait to true while
doing flow restoration, and sets it back to false after it is
done, and this patch utilizes that fact to prevent the TLV race.
It does this by preventing vswitchd from connecting to
controllers in the controller table while it is in a
flow-restore-wait state.

With this patch, when bridge_configure_remotes() calls
bridge_get_controllers(), it first checks if flow-restore-wait
has been set, and if so, it ignores any controllers in the
controller database and sets n_controllers to 0.

This solution does preserve the management service controller
which is added via bridge_ofproto_controller_for_mgmt() after
checking whether we should call bridge_get_controllers()
(and thus n_controllers is properly set to 1, etc)

VMware-BZ: 2195377
Signed-off-by: Zak Whittington <zwhitt.vmware@gmail.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
---
 vswitchd/bridge.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index 18c16fb83..d9103c7fb 100644
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -3561,7 +3561,8 @@ bridge_configure_remotes(struct bridge *br,
         ofproto_set_extra_in_band_remotes(br->ofproto, managers, n_managers);
     }
 
-    n_controllers = bridge_get_controllers(br, &controllers);
+    n_controllers = (ofproto_get_flow_restore_wait() ? 0
+                     : bridge_get_controllers(br, &controllers));
 
     ocs = xmalloc((n_controllers + 1) * sizeof *ocs);
     n_ocs = 0;
-- 
2.21.0

Places

File 0002-bridge.c-prevent-controller-connects-while-flow-rest.patch of Package openvswitch-dpdk.10750

Places