Changeset 15789 in vbox

Timestamp:

Jan 5, 2009 10:00:04 AM (16 years ago)

Author:

vboxsync

Message:

Protect code that relies on not being rescheduled to another cpu. (harmless, but triggers bug checks in Linux)

File:

• trunk/src/VBox/VMM/VMMR0/HWACCMR0.cpp (modified) (6 diffs)

trunk/src/VBox/VMM/VMMR0/HWACCMR0.cpp

@@ -798 +798 @@
 {
     int             rc;
-    RTCPUID         idCpu = RTMpCpuId();
-    PHWACCM_CPUINFO pCpu = &HWACCMR0Globals.aCpuInfo[idCpu];
 
     AssertReturn(pVM, VERR_INVALID_PARAMETER);
@@ -850 +848 @@
     }
 
+    RTCCUINTREG     fFlags = ASMIntDisableFlags();
+    RTCPUID         idCpu = RTMpCpuId();
+    PHWACCM_CPUINFO pCpu = &HWACCMR0Globals.aCpuInfo[idCpu];
+
     ASMAtomicWriteBool(&pCpu->fInUse, true);
 
@@ -856 +858 @@
 
     ASMAtomicWriteBool(&pCpu->fInUse, false);
+    ASMSetFlags(fFlags);
 
     return rc;
@@ -870 +873 @@
 {
     int             rc;
-    RTCPUID         idCpu = RTMpCpuId();
-    PHWACCM_CPUINFO pCpu = &HWACCMR0Globals.aCpuInfo[idCpu];
 
     AssertReturn(pVM, VERR_INVALID_PARAMETER);
@@ -882 +883 @@
     AssertReturn(!ASMAtomicReadBool(&HWACCMR0Globals.fSuspended), VERR_HWACCM_SUSPEND_PENDING);
 
+    RTCCUINTREG     fFlags = ASMIntDisableFlags();
+    RTCPUID         idCpu = RTMpCpuId();
+    PHWACCM_CPUINFO pCpu = &HWACCMR0Globals.aCpuInfo[idCpu];
+
     ASMAtomicWriteBool(&pCpu->fInUse, true);
 
@@ -888 +893 @@
 
     ASMAtomicWriteBool(&pCpu->fInUse, false);
+    ASMSetFlags(fFlags);
     return rc;
 }