Changeset 90598 in vbox

Timestamp:

Aug 10, 2021 1:12:43 PM (4 years ago)

Author:

vboxsync

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

146209

Message:

VMM: Speed up VMMGetCpuId in ring-0 by using hash table (via new GVMMR0GetGVCpuByGVMandEMT call). bugref:6695

File:

• trunk/src/VBox/VMM/VMMAll/VMMAll.cpp (modified) (1 diff)

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

@@ -187 +187 @@
 
 #elif defined(IN_RING0)
-    if (pVM->cCpus == 1)
-        return 0;
-    VMCPUID const cCpus = pVM->cCpus;
-
-    /* Search first by host cpu id (most common case)
-     * and then by native thread id (page fusion case).
-     */
-    if (!RTThreadPreemptIsEnabled(NIL_RTTHREAD))
-    {
-        /** @todo r=ramshankar: This doesn't buy us anything in terms of performance
-         *        leaving it here for hysterical raisins and as a reference if we
-         *        implemented a hashing approach in the future. */
-        RTCPUID idHostCpu = RTMpCpuId();
-
-        /** @todo optimize for large number of VCPUs when that becomes more common. */
-        for (VMCPUID idCpu = 0; idCpu < cCpus; idCpu++)
-        {
-            PVMCPUCC pVCpu = VMCC_GET_CPU(pVM, idCpu);
-            if (pVCpu->idHostCpu == idHostCpu)
-                return pVCpu->idCpu;
-        }
-    }
-
-    /* RTThreadGetNativeSelf had better be cheap. */
-    RTNATIVETHREAD hThread = RTThreadNativeSelf();
-
-    /** @todo optimize for large number of VCPUs when that becomes more common. */
-    for (VMCPUID idCpu = 0; idCpu < cCpus; idCpu++)
-    {
-        PVMCPUCC pVCpu = VMCC_GET_CPU(pVM, idCpu);
-        if (pVCpu->hNativeThreadR0 == hThread)
-            return pVCpu->idCpu;
-    }
-    return NIL_VMCPUID;
+    PVMCPUCC pVCpu = GVMMR0GetGVCpuByGVMandEMT(pVM, NIL_RTNATIVETHREAD);
+    return pVCpu ? pVCpu->idCpu : NIL_VMCPUID;
 
 #else /* RC: Always EMT(0) */