Changeset 101096 in vbox for trunk/src/VBox/VMM/VMMR3

Timestamp:

Sep 12, 2023 10:59:30 PM (17 months ago)

Author:

vboxsync

Message:

VMM/IEM: Fixed a bug in the TB allocator pruning code and another one wrt initial chunk allocations. bugref:10369

File:

• trunk/src/VBox/VMM/VMMR3/IEMR3.cpp (modified) (3 diffs)

trunk/src/VBox/VMM/VMMR3/IEMR3.cpp

@@ -116 +116 @@
 
 #ifdef VBOX_WITH_IEM_RECOMPILER
-    /** @cfgm{/IEM/InitialTbCount, uint32_t, 32768}
-     * Initial (minimum) number of TBs per EMT in ring-3. */
-    uint32_t cInitialTbs = 0;
-    rc = CFGMR3QueryU32Def(pIem, "InitialTbCount", &cInitialTbs, _32K);
-    AssertLogRelRCReturn(rc, rc);
-    if (cInitialTbs < _16K || cInitialTbs > _8M)
-        return VMSetError(pVM, VERR_OUT_OF_RANGE, RT_SRC_POS,
-                          "InitialTbCount value %u (%#x) is out of range (min %u, max %u)", cInitialTbs, cInitialTbs, _16K, _8M);
-
     /** @cfgm{/IEM/MaxTbCount, uint32_t, 524288}
      * Max number of TBs per EMT. */
@@ -130 +121 @@
     rc = CFGMR3QueryU32Def(pIem, "MaxTbCount", &cMaxTbs, _512K);
     AssertLogRelRCReturn(rc, rc);
-    if (cMaxTbs < cInitialTbs || cMaxTbs > _8M)
+    if (cMaxTbs < _16K || cMaxTbs > _8M)
         return VMSetError(pVM, VERR_OUT_OF_RANGE, RT_SRC_POS,
-                          "MaxTbCount value %u (%#x) is out of range (min %u, max %u)", cMaxTbs, cMaxTbs, cInitialTbs, _8M);
+                          "MaxTbCount value %u (%#x) is out of range (min %u, max %u)", cMaxTbs, cMaxTbs, _16K, _8M);
+
+    /** @cfgm{/IEM/InitialTbCount, uint32_t, 32678}
+     * Initial (minimum) number of TBs per EMT in ring-3. */
+    uint32_t cInitialTbs = 0;
+    rc = CFGMR3QueryU32Def(pIem, "InitialTbCount", &cInitialTbs, RT_MIN(cMaxTbs, _32K));
+    AssertLogRelRCReturn(rc, rc);
+    if (cInitialTbs < _16K || cInitialTbs > _8M)
+        return VMSetError(pVM, VERR_OUT_OF_RANGE, RT_SRC_POS,
+                          "InitialTbCount value %u (%#x) is out of range (min %u, max %u)", cInitialTbs, cInitialTbs, _16K, _8M);
+
+    /* Check that the two values makes sense together. Expect user/api to do
+       the right thing or get lost. */
+    if (cInitialTbs > cMaxTbs)
+        return VMSetError(pVM, VERR_OUT_OF_RANGE, RT_SRC_POS,
+                          "InitialTbCount value %u (%#x) is higher than the MaxTbCount value %u (%#x)",
+                          cInitialTbs, cInitialTbs, cMaxTbs, cMaxTbs);
 #endif
 
@@ -285 +292 @@
 
         PIEMTBALLOCATOR const pTbAllocator = pVCpu->iem.s.pTbAllocatorR3;
-        STAMR3RegisterF(pVM, (void *)&pTbAllocator->StatAllocs,         STAMTYPE_COUNTER,   STAMVISIBILITY_ALWAYS, STAMUNIT_OCCURENCES,
-                        "Translation block allocations",                "/IEM/CPU%u/re/cTbAllocs", idCpu);
-        STAMR3RegisterF(pVM, (void *)&pTbAllocator->StatFrees,          STAMTYPE_COUNTER,   STAMVISIBILITY_ALWAYS, STAMUNIT_OCCURENCES,
-                        "Translation block frees",                      "/IEM/CPU%u/re/cTbFrees", idCpu);
+        STAMR3RegisterF(pVM, (void *)&pTbAllocator->StatAllocs,         STAMTYPE_COUNTER,   STAMVISIBILITY_ALWAYS, STAMUNIT_CALLS,
+                        "Translation block allocations",                "/IEM/CPU%u/re/cTbAllocCalls", idCpu);
+        STAMR3RegisterF(pVM, (void *)&pTbAllocator->StatFrees,          STAMTYPE_COUNTER,   STAMVISIBILITY_ALWAYS, STAMUNIT_CALLS,
+                        "Translation block frees",                      "/IEM/CPU%u/re/cTbFreeCalls", idCpu);
 # ifdef VBOX_WITH_STATISTICS
         STAMR3RegisterF(pVM, (void *)&pTbAllocator->StatPrune,          STAMTYPE_PROFILE,   STAMVISIBILITY_ALWAYS, STAMUNIT_TICKS_PER_CALL,