[release/9.0-rc2] dont try to capture threadId for NativeAOT (#108091)

* dont try to capture threadId for NativeAOT * add config to capture bgc threadid --------- Co-authored-by: Manish Godse <61718172+mangod9@users.noreply.github.com>
dotnet · Sep 23, 2024 · 200d05e · 200d05e
1 parent 55c018f
commit 200d05e
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 10 deletions.
diff --git a/src/coreclr/gc/gc.cpp b/src/coreclr/gc/gc.cpp
@@ -26593,16 +26593,19 @@ void gc_heap::add_to_hc_history_worker (hc_history* hist, int* current_index, hc
     current_hist->concurrent_p = (bool)settings.concurrent;
     current_hist->bgc_thread_running = (bool)bgc_thread_running;
 
-#if defined(TARGET_AMD64) && defined(TARGET_WINDOWS) && !defined(_DEBUG)
-    int bgc_thread_os_id = 0;
-
-    if (bgc_thread)
+#if defined(TARGET_AMD64) && defined(TARGET_WINDOWS) && !defined(_DEBUG) && !defined(FEATURE_NATIVEAOT)
+    if (GCConfig::GetGCLogBGCThreadId())
     {
-        bgc_thread_os_id = (int)(*(size_t*)((uint8_t*)bgc_thread + 0x130));
-    }
+        int bgc_thread_os_id = 0;
+
+        if (bgc_thread)
+        {
+            bgc_thread_os_id = (int)(*(size_t*)((uint8_t*)bgc_thread + 0x130));
+        }
 
-    current_hist->bgc_thread_os_id = bgc_thread_os_id;
-#endif //TARGET_AMD64 && TARGET_WINDOWS && !_DEBUG
+        current_hist->bgc_thread_os_id = bgc_thread_os_id;
+    }
+#endif //TARGET_AMD64 && TARGET_WINDOWS && !_DEBUG && !FEATURE_NATIVEAOT
 #endif //BACKGROUND_GC
 
     *current_index  = (*current_index + 1) % max_hc_history_count;

diff --git a/src/coreclr/gc/gcconfig.h b/src/coreclr/gc/gcconfig.h
@@ -141,7 +141,10 @@ class GCConfigStringHolder
     STRING_CONFIG(GCPath,                    "GCPath",                    "System.GC.Path",                                        "Specifies the path of the standalone GC implementation.")                                \
     INT_CONFIG   (GCSpinCountUnit,           "GCSpinCountUnit",           NULL,                                0,                  "Specifies the spin count unit used by the GC.")                                          \
     INT_CONFIG   (GCDynamicAdaptationMode,   "GCDynamicAdaptationMode",   "System.GC.DynamicAdaptationMode",   1,                  "Enable the GC to dynamically adapt to application sizes.")                               \
-    INT_CONFIG   (GCDTargetTCP,              "GCDTargetTCP",              "System.GC.DTargetTCP",              0,                  "Specifies the target tcp for DATAS")
+    INT_CONFIG   (GCDTargetTCP,              "GCDTargetTCP",              "System.GC.DTargetTCP",              0,                  "Specifies the target tcp for DATAS")                                                     \
+    BOOL_CONFIG  (GCLogBGCThreadId,          "GCLogBGCThreadId",          NULL,                                false,              "Specifies if BGC ThreadId should be logged")                                         
+
+
 // This class is responsible for retreiving configuration information
 // for how the GC should operate.
 class GCConfig

diff --git a/src/coreclr/gc/gcpriv.h b/src/coreclr/gc/gcpriv.h
@@ -618,7 +618,7 @@ struct hc_history
     // invalid fields on the Thread object such as m_OSThreadId. This is to help with debugging that problem so I
     // only enable it for retail builds on Windows. We can extend this with a GCToEEInterface interface method to get the offset
     // of that particular field on the Thread object.
-#if defined(TARGET_AMD64) && defined(TARGET_WINDOWS) && !defined(_DEBUG)
+#if defined(TARGET_AMD64) && defined(TARGET_WINDOWS) && !defined(_DEBUG) && !defined(FEATURE_NATIVEAOT)
     int bgc_thread_os_id;
 #endif
     short bgc_t_join_join_lock;