Moved a static field initialization from Thread to ProcessorIdCache (#114227)

Presence of `.cctor` in `Thread` can cause circular dependency if Lock needs to block while Thread .cctor has not run yet.

1. Lock needs to wait on a WaitHandle
2. WaitHandle needs Thread.CurrentThread
3. if Thread's .cctor has not run yet, it needs to run.     
(it is unusual for this to be the first use of Thread, but the activation pattern in #113949 made it possible)
4. .cctor needs to take a Lock, so we go to `#1`

Fixes: #113949

Author: VSadov

src/libraries/System.Private.CoreLib/src/System/Threading/ProcessorIdCache.cs

@@ -22,6 +22,10 @@ internal static class ProcessorIdCache
         // We will not adjust higher than this though.
         private const int MaxIdRefreshRate = 5000;
 
+        // a speed check will determine refresh rate of the cache and will report if caching is not advisable.
+        // we will record that in a readonly static so that it could become a JIT constant and bypass caching entirely.
+        private static readonly bool s_isProcessorNumberReallyFast = ProcessorIdCache.ProcessorNumberSpeedCheck();
+
         private static int RefreshCurrentProcessorId()
         {
             int currentProcessorId = Thread.GetCurrentProcessorNumber();
@@ -44,6 +48,9 @@ private static int RefreshCurrentProcessorId()
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         internal static int GetCurrentProcessorId()
         {
+            if (s_isProcessorNumberReallyFast)
+                return Thread.GetCurrentProcessorNumber();
+
             int currentProcessorIdCache = t_currentProcessorIdCache--;
             if ((currentProcessorIdCache & ProcessorIdCacheCountDownMask) == 0)
             {

src/libraries/System.Private.CoreLib/src/System/Threading/Thread.cs

@@ -719,16 +719,9 @@ public static void SetData(LocalDataStoreSlot slot, object? value)
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         public static int GetCurrentProcessorId()
         {
-            if (s_isProcessorNumberReallyFast)
-                return GetCurrentProcessorNumber();
-
             return ProcessorIdCache.GetCurrentProcessorId();
         }
 
-        // a speed check will determine refresh rate of the cache and will report if caching is not advisable.
-        // we will record that in a readonly static so that it could become a JIT constant and bypass caching entirely.
-        private static readonly bool s_isProcessorNumberReallyFast = ProcessorIdCache.ProcessorNumberSpeedCheck();
-
 #if FEATURE_WASM_MANAGED_THREADS
         [ThreadStatic]
         public static bool ThrowOnBlockingWaitOnJSInteropThread;