0025396: Crash occurs when using TBB allocator on an Intel architecture not supportin...

During initialization of memory manager, check if SSE2 instructions are supported, when MMGT_OPT=2 is in effect. If not then use MMgrRaw instead of MMgrTBBalloc. It is to avoid runtime crash when running on a CPU that supports SSE but does not support SSE2 (some modifications of AMD Sempron).

Fix broken compilation on MSVC for x64 platform

Correct the last fix.

diff --git a/src/Standard/Standard.cxx b/src/Standard/Standard.cxx
index 4980f551711095b0b0d01b29a57cd9ec28c693ab..4f79abb650924e6a29fc53058dafb5467c671e64 100644 (file)
--- a/src/Standard/Standard.cxx
+++ b/src/Standard/Standard.cxx
@@ -78,6 +78,38 @@ Standard_MMgrFactory::Standard_MMgrFactory()
   char* aVar;
   aVar = getenv ("MMGT_OPT");
   Standard_Integer anAllocId   = (aVar ?  atoi (aVar): OCCT_MMGT_OPT_DEFAULT);
+
+#if defined(_WIN32) && !defined(_WIN64)
+  static const DWORD _SSE2_FEATURE_BIT(0x04000000);
+  if ( anAllocId == 2 )
+  {
+    // CR25396: Check if SSE2 instructions are supported, if not then use MMgrRaw
+    // instead of MMgrTBBalloc. It is to avoid runtime crash when running on a 
+    // CPU that supports SSE but does not support SSE2 (some modifications of
+    // AMD Sempron).
+    DWORD volatile dwFeature;
+    _asm
+    {
+      push eax
+      push ebx
+      push ecx
+      push edx
+
+      // get the CPU feature bits
+      mov eax, 1
+      cpuid
+      mov dwFeature, edx
+
+      pop edx
+      pop ecx
+      pop ebx
+      pop eax
+    }
+    if ((dwFeature & _SSE2_FEATURE_BIT) == 0)
+      anAllocId = 0;
+  }
+#endif
+
   aVar = getenv ("MMGT_CLEAR");
   Standard_Boolean toClear     = (aVar ? (atoi (aVar) != 0) : Standard_True);