From: Jan Kotas <jkotas@microsoft.com>
Date: Tue, 8 Oct 2019 02:04:18 +0000 (-0700)
Subject: Enable optimized single-proc allocation helpers for single-proc x86/x64 systems only... 
X-Git-Tag: submit/tizen/20210909.063632~11030^2~435
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=595a75721e718f69de309e1ebfed63d5fc427903;p=platform%2Fupstream%2Fdotnet%2Fruntime.git

Enable optimized single-proc allocation helpers for single-proc x86/x64 systems only (dotnet/coreclr#27014)

Use maximum number of processors the process may run on to determine whether it is ok to use
single-proc allocation helpers. It is not sufficient to depend on current process affinity since
that can change during the process lifetime.

Also, the single-proc allocation helpers work well on x86/x64 systems only because of they depend
on atomic non-interlocked increment instruction for good performance. Such instruction is available
on x86/x64 only. Disable them everywhere else.

Fixes dotnet/coreclr#26990

Commit migrated from https://github.com/dotnet/coreclr/commit/4e702da2c06ded8239d5eb2f47619604efbf3fde
---

diff --git a/src/coreclr/src/vm/gcheaputilities.h b/src/coreclr/src/vm/gcheaputilities.h
index 40889e9..11f066c 100644
--- a/src/coreclr/src/vm/gcheaputilities.h
+++ b/src/coreclr/src/vm/gcheaputilities.h
@@ -122,12 +122,12 @@ public:
 
     static bool UseThreadAllocationContexts()
     {
-        // When running on a single-proc system, it's more efficient to use a single global
+        // When running on a single-proc Intel system, it's more efficient to use a single global
         // allocation context for SOH allocations than to use one for every thread.
-#if defined(_TARGET_ARM_) || defined(FEATURE_PAL) || defined(FEATURE_REDHAWK)
-        return true;
+#if (defined(_TARGET_X86_) || defined(_TARGET_AMD64_)) && !defined(FEATURE_PAL)
+        return IsServerHeap() || ::g_SystemInfo.dwNumberOfProcessors != 1 || CPUGroupInfo::CanEnableGCCPUGroups();
 #else
-        return IsServerHeap() || ::GetCurrentProcessCpuCount() != 1;
+        return true;
 #endif
 
     }