use std::atomic to protect concurrent access to the call_id_ static d…

…ata member Protect concurrent access to the Blif::call_id_ using an atomic data type: it is cheaper than a lock and for a machine word it is sufficent. Resolves rmp in #5981. Signed-off-by: Calin Cascaval <[email protected]>
The-OpenROAD-Project · Oct 21, 2024 · 100e382 · 100e382
1 parent c12d979
commit 100e382
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/rmp/include/rmp/blif.h b/src/rmp/include/rmp/blif.h
@@ -38,6 +38,7 @@
 #include <stdio.h>
 #include <unistd.h>
 
+#include <atomic>
 #include <map>
 #include <set>
 #include <string>
@@ -90,7 +91,7 @@ class Blif
   std::string const1_cell_port_;
   std::map<std::string, std::pair<float, float>> requireds_;
   std::map<std::string, std::pair<float, float>> arrivals_;
-  static int call_id_;
+  static std::atomic<int> call_id_;
 };
 
 }  // namespace rmp
diff --git a/src/rmp/src/blif.cpp b/src/rmp/src/blif.cpp
@@ -62,7 +62,7 @@ using utl::RMP;
 
 namespace rmp {
 
-int Blif::call_id_ = 0;
+std::atomic<int> Blif::call_id_ = 0;
 
 Blif::Blif(Logger* logger,
            sta::dbSta* sta,