Changeset 100695 in vbox for trunk/src/VBox/VMM

Timestamp:

Jul 25, 2023 11:03:20 AM (17 months ago)

Author:

vboxsync

Message:

VMM/IEM,STAM: Some TB stats. bugref:10369

Location:

trunk/src/VBox/VMM

Files:

• VMMAll/IEMAllThreadedRecompiler.cpp (modified) (1 diff)
• VMMR3/IEMR3.cpp (modified) (1 diff)
• VMMR3/STAM.cpp (modified) (5 diffs)
• include/IEMInternal.h (modified) (1 diff)

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

@@ -741 +741 @@
     pTb->pNext = g_TbCache.apHash[idxHash];
     g_TbCache.apHash[idxHash] = pTb;
+    STAM_REL_PROFILE_ADD_PERIOD(&pVCpu->iem.s.StatTbThreadedInstr, pTb->cInstructions);
+    STAM_REL_PROFILE_ADD_PERIOD(&pVCpu->iem.s.StatTbThreadedCalls, pTb->Thrd.cCalls);
     Log12(("TB added: %p %RGp LB %#x fl=%#x idxHash=%#x\n", pTb, pTb->GCPhysPc, pTb->cbOpcodes, pTb->fFlags, idxHash));
-    RT_NOREF(pVM, pVCpu);
+    RT_NOREF(pVM);
 }
 

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

@@ -170 +170 @@
 #ifdef VBOX_WITH_IEM_RECOMPILER
         STAMR3RegisterF(pVM, (void *)&pVCpu->iem.s.cTbExec,             STAMTYPE_U64_RESET, STAMVISIBILITY_ALWAYS, STAMUNIT_NONE,
-                        "Executed translation block",               "/IEM/CPU%u/cTbExec", idCpu);
+                        "Executed translation block",                   "/IEM/CPU%u/cTbExec", idCpu);
         STAMR3RegisterF(pVM, (void *)&pVCpu->iem.s.cTbAllocs,           STAMTYPE_U64,       STAMVISIBILITY_ALWAYS, STAMUNIT_NONE,
-                        "Translation block allocations",            "/IEM/CPU%u/cTbAllocs", idCpu);
+                        "Translation block allocations",                "/IEM/CPU%u/cTbAllocs", idCpu);
         STAMR3RegisterF(pVM, (void *)&pVCpu->iem.s.cTbFrees,            STAMTYPE_U64,       STAMVISIBILITY_ALWAYS, STAMUNIT_NONE,
-                        "Translation block frees",                  "/IEM/CPU%u/cTbFrees", idCpu);
+                        "Translation block frees",                      "/IEM/CPU%u/cTbFrees", idCpu);
         STAMR3RegisterF(pVM, (void *)&pVCpu->iem.s.cTbLookupHits,       STAMTYPE_U64_RESET, STAMVISIBILITY_ALWAYS, STAMUNIT_NONE,
-                        "Translation block lookup hits",            "/IEM/CPU%u/cTbLookupHits", idCpu);
+                        "Translation block lookup hits",                "/IEM/CPU%u/cTbLookupHits", idCpu);
         STAMR3RegisterF(pVM, (void *)&pVCpu->iem.s.cTbLookupMisses,     STAMTYPE_U64_RESET, STAMVISIBILITY_ALWAYS, STAMUNIT_NONE,
-                        "Translation block lookup misses",          "/IEM/CPU%u/cTbLookupMisses", idCpu);
+                        "Translation block lookup misses",              "/IEM/CPU%u/cTbLookupMisses", idCpu);
+
+        STAMR3RegisterF(pVM, (void *)&pVCpu->iem.s.StatTbThreadedCalls, STAMTYPE_PROFILE, STAMVISIBILITY_ALWAYS, STAMUNIT_CALLS_PER_TB,
+                        "Calls per threaded translation block",         "/IEM/CPU%u/ThrdCallsPerTb", idCpu);
+        STAMR3RegisterF(pVM, (void *)&pVCpu->iem.s.StatTbThreadedInstr, STAMTYPE_PROFILE, STAMVISIBILITY_ALWAYS, STAMUNIT_INSTR_PER_TB,
+                        "Instruction per threaded translation block",   "/IEM/CPU%u/ThrdInstrPerTb", idCpu);
 #endif
 

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

@@ -3042 +3042 @@
         case STAMUNIT_NONE:                 return "";
         case STAMUNIT_CALLS:                return "calls";
+        case STAMUNIT_CALLS_PER_TB:         return "calls/tb";
         case STAMUNIT_COUNT:                return "count";
         case STAMUNIT_BYTES:                return "bytes";
@@ -3059 +3060 @@
         case STAMUNIT_PCT:                  return "%";
         case STAMUNIT_HZ:                   return "Hz";
+        case STAMUNIT_INSTR:                return "instr";
+        case STAMUNIT_INSTR_PER_TB:         return "instr/tb";
 
         default:
@@ -3079 +3082 @@
         case STAMUNIT_NONE:                 return "";
         case STAMUNIT_CALLS:                return "calls";
+        case STAMUNIT_CALLS_PER_TB:         return "calls";
         case STAMUNIT_COUNT:                return "count";
         case STAMUNIT_BYTES:                return "bytes";
@@ -3096 +3100 @@
         case STAMUNIT_PCT:                  return "%";
         case STAMUNIT_HZ:                   return "Hz";
+        case STAMUNIT_INSTR:                return "instr";
+        case STAMUNIT_INSTR_PER_TB:         return "instr";
 
         default:
@@ -3121 +3127 @@
         case STAMUNIT_NONE:                 return "times";
         case STAMUNIT_GOOD_BAD:             return "bad";
+        case STAMUNIT_CALLS_PER_TB:         return "tbs";
+        case STAMUNIT_INSTR_PER_TB:         return "tbs";
         default:
             AssertMsgFailed(("Wrong unit %d\n", enmUnit));

trunk/src/VBox/VMM/include/IEMInternal.h

@@ -1148 +1148 @@
     /** Spaced reserved for recompiler data / alignment. */
     bool                    afRecompilerStuff1[4];
+    /** Threaded TB statistics: Number of instructions per TB. */
+    STAMPROFILE             StatTbThreadedInstr;
+    /** Threaded TB statistics: Number of calls per TB. */
+    STAMPROFILE             StatTbThreadedCalls;
     /** @} */