Changeset 31359 in vbox

Timestamp:

Aug 4, 2010 3:31:04 PM (14 years ago)

Author:

vboxsync

Message:

Keep track of the native R0 thread handle for each EMT too. Use that to find the right VCPU

Location:

trunk

Files:

• include/VBox/vm.h (modified) (2 diffs)
• include/VBox/vm.mac (modified) (1 diff)
• src/VBox/VMM/VMMAll/VMMAll.cpp (modified) (4 diffs)
• src/VBox/VMM/VMMR0/VMMR0.cpp (modified) (1 diff)

trunk/include/VBox/vm.h

@@ -102 +102 @@
     /** The native thread handle. */
     RTNATIVETHREAD          hNativeThread;
+    /** The native R0 thread handle. (different from the R3 handle!) */
+    RTNATIVETHREAD          hNativeThreadR0;
     /** Which host CPU ID is this EMT running on.
      * Only valid when in RC or HWACCMR0 with scheduling disabled. */
@@ -114 +116 @@
      *          following it (to grow into and align the struct size).
      *   */
-    uint8_t                 abAlignment1[HC_ARCH_BITS == 32 ? 28 : 12];
+    uint8_t                 abAlignment1[HC_ARCH_BITS == 32 ? 24 : 4];
 
     /** CPUM part. */

trunk/include/VBox/vm.mac

@@ -120 +120 @@
 
     .hNativeThread          RTR0PTR_RES 1
+    .hNativeThreadR0        RTR0PTR_RES 1
     .idHostCpu              resd 1
 

trunk/src/VBox/VMM/VMMAll/VMMAll.cpp

@@ -61 +61 @@
         return 0;
 
-    /* Search first by host cpu id (most common case)
-     * and then by native thread id (page fusion case).
-     */
-    /* RTMpCpuId had better be cheap. */ 
-    RTCPUID idHostCpu = RTMpCpuId(); 
-
-    /** @todo optimize for large number of VCPUs when that becomes more common. */
-    for (VMCPUID idCpu = 0; idCpu < pVM->cCpus; idCpu++)
-    {
-        PVMCPU pVCpu = &pVM->aCpus[idCpu];
-
-        if (pVCpu->idHostCpu == idHostCpu)
-            return pVCpu->idCpu;
-    }
-
     /* RTThreadGetNativeSelf had better be cheap. */
     RTNATIVETHREAD hThread = RTThreadNativeSelf();
@@ -84 +69 @@
         PVMCPU pVCpu = &pVM->aCpus[idCpu];
 
-        if (pVCpu->hNativeThread == hThread)
+        if (pVCpu->hNativeThreadR0 == hThread)
             return pVCpu->idCpu;
     }
@@ -115 +100 @@
         return &pVM->aCpus[0];
 
-    /* Search first by host cpu id (most common case)
-     * and then by native thread id (page fusion case).
-     */
-
-    /* RTMpCpuId had better be cheap. */ 
-    RTCPUID idHostCpu = RTMpCpuId(); 
-
-    /** @todo optimize for large number of VCPUs when that becomes more common. */
-    for (VMCPUID idCpu = 0; idCpu < pVM->cCpus; idCpu++)
-    {
-        PVMCPU pVCpu = &pVM->aCpus[idCpu];
-
-        if (pVCpu->idHostCpu == idHostCpu)
-            return pVCpu;
-    }
-
     /* RTThreadGetNativeSelf had better be cheap. */
     RTNATIVETHREAD hThread = RTThreadNativeSelf();
@@ -139 +108 @@
         PVMCPU pVCpu = &pVM->aCpus[idCpu];
 
-        if (pVCpu->hNativeThread == hThread)
+        if (pVCpu->hNativeThreadR0 == hThread)
             return pVCpu;
     }

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

@@ -965 +965 @@
             PVMCPU pVCpu = &pVM->aCpus[idCpu];
 
+            if (pVCpu->hNativeThreadR0 == NIL_RTNATIVETHREAD)
+                pVCpu->hNativeThreadR0 = RTThreadNativeSelf();
+
             /* Make sure that log flushes can jump back to ring-3; annoying to get an incomplete log (this is risky though as the code doesn't take this into account). */
             int rc = GMMR0CheckSharedModulesStart(pVM);