fix set globalbase pointer on switch_worker_base()

yanet-platform · Jan 31, 2024 · da7e398 · da7e398
1 parent c58f828
commit da7e398
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/dataplane/controlplane.cpp b/dataplane/controlplane.cpp
@@ -1358,16 +1358,16 @@ void cControlPlane::switchGlobalBase()
 {
 	YADECAP_MEMORY_BARRIER_COMPILE;
 
-	dataPlane->switch_worker_base();
-
-	YADECAP_MEMORY_BARRIER_COMPILE;
-
 	{
 		std::lock_guard<std::mutex> guard(dataPlane->currentGlobalBaseId_mutex);
 		dataPlane->currentGlobalBaseId ^= 1;
 	}
 
 	YADECAP_MEMORY_BARRIER_COMPILE;
+
+	dataPlane->switch_worker_base();
+
+	YADECAP_MEMORY_BARRIER_COMPILE;
 }
 
 void cControlPlane::waitAllWorkers()

diff --git a/dataplane/dataplane.cpp b/dataplane/dataplane.cpp
@@ -1591,7 +1591,7 @@ void cDataPlane::switch_worker_base()
 		std::lock_guard<std::mutex> guard(currentGlobalBaseId_mutex);
 		for (const auto& [socket_id, base_next] : base_nexts)
 		{
-			base_next->globalBase = globalBases[socket_id][currentGlobalBaseId ^ 1];
+			base_next->globalBase = globalBases[socket_id][currentGlobalBaseId];
 		}
 	}
 	neighbor.update_worker_base(base_nexts);