ThreadBack: fix race found by tsan

nokia · Jan 31, 2024 · 8883732 · 8883732
1 parent fcca6d6
commit 8883732
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/src/log/UniCoutLog.hpp b/src/log/UniCoutLog.hpp
@@ -13,6 +13,8 @@
 //
 // - CORE:
 //   . cout
+//
+// - NOT MT safe: since (static) member; so shall ONLY use in main thread
 // ***********************************************************************************************
 #ifndef UNI_COUT_LOG_HPP_
 #define UNI_COUT_LOG_HPP_

diff --git a/src/log/UniSmartLog.hpp b/src/log/UniSmartLog.hpp
@@ -10,6 +10,8 @@
 //
 // - CORE:
 //   . smartLog_
+//
+// - NOT MT safe: since (static) member; so shall ONLY use in main thread
 // ***********************************************************************************************
 #ifndef UNI_SMART_LOG_HPP_
 #define UNI_SMART_LOG_HPP_

diff --git a/ut/thread/ThreadBackTest.cpp b/ut/thread/ThreadBackTest.cpp
@@ -60,7 +60,7 @@ TEST_F(ThreadBackTest, GOLD_entryFn_inNewThread_thenBackFn_inMainThread_withSema
         {
             EXPECT_NE(this_thread::get_id(), mt_threadID) << "REQ: in new thread";
             mt_threadID = this_thread::get_id();
-            DBG("MT_ThreadEntryFN(): thread id=" << mt_threadID);
+            HID("MT_ThreadEntryFN(): thread id=" << mt_threadID);  // not MT safe but debug only
             return true;
         },
         // ThreadBackFN