Changeset 31413 in vbox for trunk/include

Timestamp:

Aug 5, 2010 3:24:55 PM (15 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

64478

Message:

iprt: ASMCpuId() and ASMGetApicId() return different data on different CPU cores so use volatile in the asm statement

File:

• trunk/include/iprt/asm-amd64-x86.h (modified) (3 diffs)

trunk/include/iprt/asm-amd64-x86.h

@@ -470 +470 @@
 #  ifdef RT_ARCH_AMD64
     RTCCUINTREG uRAX, uRBX, uRCX, uRDX;
-    __asm__ ("cpuid\n\t"
-             : "=a" (uRAX),
-               "=b" (uRBX),
-               "=c" (uRCX),
-               "=d" (uRDX)
+    __asm__ __volatile__ ("cpuid\n\t"
+                          : "=a" (uRAX),
+                            "=b" (uRBX),
+                            "=c" (uRCX),
+                            "=d" (uRDX)
              : "0" (uOperator));
     *(uint32_t *)pvEAX = (uint32_t)uRAX;
@@ -481 +481 @@
     *(uint32_t *)pvEDX = (uint32_t)uRDX;
 #  else
-    __asm__ ("xchgl %%ebx, %1\n\t"
-             "cpuid\n\t"
-             "xchgl %%ebx, %1\n\t"
-             : "=a" (*(uint32_t *)pvEAX),
-               "=r" (*(uint32_t *)pvEBX),
-               "=c" (*(uint32_t *)pvECX),
-               "=d" (*(uint32_t *)pvEDX)
-             : "0" (uOperator));
+    __asm__ __volatile__ ("xchgl %%ebx, %1\n\t"
+                          "cpuid\n\t"
+                          "xchgl %%ebx, %1\n\t"
+                         : "=a" (*(uint32_t *)pvEAX),
+                           "=r" (*(uint32_t *)pvEBX),
+                           "=c" (*(uint32_t *)pvECX),
+                           "=d" (*(uint32_t *)pvEDX)
+                         : "0" (uOperator));
 #  endif
 
@@ -906 +906 @@
 #  ifdef RT_ARCH_AMD64
     RTCCUINTREG uSpill;
-    __asm__ ("cpuid"
-             : "=a" (uSpill),
-               "=b" (xBX)
-             : "0" (1)
-             : "rcx", "rdx");
+    __asm__ __volatile__ ("cpuid"
+                          : "=a" (uSpill),
+                            "=b" (xBX)
+                          : "0" (1)
+                          : "rcx", "rdx");
 #  elif (defined(PIC) || defined(__PIC__)) && defined(__i386__)
     RTCCUINTREG uSpill;
-    __asm__ ("mov   %%ebx,%1\n\t"
-             "cpuid\n\t"
-             "xchgl %%ebx,%1\n\t"
-             : "=a" (uSpill),
-               "=r" (xBX)
-             : "0" (1)
-             : "ecx", "edx");
+    __asm__ __volatile__ ("mov   %%ebx,%1\n\t"
+                          "cpuid\n\t"
+                          "xchgl %%ebx,%1\n\t"
+                          : "=a" (uSpill),
+                            "=r" (xBX)
+                          : "0" (1)
+                          : "ecx", "edx");
 #  else
     RTCCUINTREG uSpill;
-    __asm__ ("cpuid"
-             : "=a" (uSpill),
-               "=b" (xBX)
-             : "0" (1)
-             : "ecx", "edx");
+    __asm__ __volatile__ ("cpuid"
+                          : "=a" (uSpill),
+                            "=b" (xBX)
+                          : "0" (1)
+                          : "ecx", "edx");
 #  endif