Changeset 100803 in vbox for trunk/src/VBox/VMM/VMMAll

Timestamp:

Aug 4, 2023 10:04:07 PM (18 months ago)

Author:

vboxsync

Message:

VMM/IEM: More stats. bugref:10369

Location:

trunk/src/VBox/VMM/VMMAll

Files:

• IEMAllThrdFuncsBltIn.cpp (modified) (7 diffs)
• IEMAllThrdRecompiler.cpp (modified) (5 diffs)
• IEMAllThrdTables.h (modified) (5 diffs)

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

@@ -91 +91 @@
 
 /**
- * Built-in function that checks for pending interrupts that can be delivered.
+ * Built-in function that checks for pending interrupts that can be delivered or
+ * forced action flags.
  *
  * This triggers after the completion of an instruction, so EIP is already at
@@ -120 +121 @@
          pVCpu->cpum.GstCtx.cs.Sel, pVCpu->cpum.GstCtx.eip, fCpu,
          pVCpu->CTX_SUFF(pVM)->fGlobalForcedActions & VM_FF_ALL_MASK, pVCpu->cpum.GstCtx.rflags.Bits.u1IF));
+    STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatCheckIrqBreaks);
     return VINF_IEM_REEXEC_BREAK;
 }
@@ -139 +141 @@
              fExpectedExec, pVCpu->iem.s.fExec, fExpectedExec ^ pVCpu->iem.s.fExec));
     RT_NOREF(uParam1, uParam2);
+    STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatCheckModeBreaks);
     return VINF_IEM_REEXEC_BREAK;
 }
@@ -245 +248 @@
                       pVCpu->iem.s.GCPhysInstrBuf + off, GCPhysRangePageWithOffset, pVCpu->iem.s.pbInstrBuf, __LINE__)); \
                 RT_NOREF(a_cbInstr); \
+                STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatCheckBranchMisses); \
                 return VINF_IEM_REEXEC_BREAK; \
             } \
@@ -278 +282 @@
                       pVCpu->iem.s.GCPhysInstrBuf + offNew, GCPhysRangePageWithOffset, pVCpu->iem.s.pbInstrBuf, __LINE__)); \
                 RT_NOREF(a_cbInstr); \
+                STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatCheckBranchMisses); \
                 return VINF_IEM_REEXEC_BREAK; \
             } \
@@ -311 +316 @@
                   pVCpu->iem.s.GCPhysInstrBuf + off, GCPhysRangePageWithOffset, pVCpu->iem.s.pbInstrBuf, __LINE__)); \
             RT_NOREF(a_cbInstr); \
+            STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatCheckBranchMisses); \
             return VINF_IEM_REEXEC_BREAK; \
         } \
@@ -331 +337 @@
                   pVCpu->cpum.GstCtx.cs.u32Limit, pVCpu->cpum.GstCtx.cs.u64Base, __LINE__)); \
             RT_NOREF(a_pTb, a_cbInstr); \
+            STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatCheckNeedCsLimChecking); \
             return VINF_IEM_REEXEC_BREAK; \
         } \

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

@@ -910 +910 @@
 
     PIEMTHRDEDCALLENTRY const pCall     = &pTb->Thrd.paCalls[pTb->Thrd.cCalls];
+    pCall->idxInstr    = pTb->cInstructions;
     pCall->offOpcode   = offOpcode;
     pCall->idxRange    = idxRange;
@@ -1272 +1273 @@
         pTb->Thrd.cCalls = (uint16_t)(idxCall + 1);
         pCall->enmFunction = kIemThreadedFunc_BltIn_CheckIrq;
+        pCall->idxInstr    = pTb->cInstructions;
         pCall->uUnused0    = 0;
         pCall->offOpcode   = 0;
@@ -1325 +1327 @@
     PIEMTHRDEDCALLENTRY pCall = &pTb->Thrd.paCalls[pTb->Thrd.cCalls++];
     pCall->enmFunction = kIemThreadedFunc_BltIn_CheckIrq;
+    pCall->idxInstr    = pTb->cInstructions;
     pCall->uUnused0    = 0;
     pCall->offOpcode   = 0;
@@ -1497 +1500 @@
             iemThreadedLogCurInstr(pVCpu, "EX");
         }
-        Log9(("%04x:%08RX64: #%d - %d %s\n", pVCpu->cpum.GstCtx.cs.Sel, pVCpu->cpum.GstCtx.rip,
-              pTb->Thrd.cCalls - cCallsLeft - 1, pCallEntry->enmFunction, g_apszIemThreadedFunctions[pCallEntry->enmFunction]));
+        Log9(("%04x:%08RX64: #%d/%d - %d %s\n", pVCpu->cpum.GstCtx.cs.Sel, pVCpu->cpum.GstCtx.rip,
+              pTb->Thrd.cCalls - cCallsLeft - 1, pCallEntry->idxInstr, pCallEntry->enmFunction,
+              g_apszIemThreadedFunctions[pCallEntry->enmFunction]));
 #endif
         VBOXSTRICTRC const rcStrict = g_apfnIemThreadedFunctions[pCallEntry->enmFunction](pVCpu,
@@ -1509 +1513 @@
         else
         {
-            pVCpu->iem.s.pCurTbR3 = NULL;
+            pVCpu->iem.s.cInstructions += pCallEntry->idxInstr; /* This may be one short, but better than zero. */
+            pVCpu->iem.s.pCurTbR3       = NULL;
+            STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatTbExecBreaks);
 
             /* Some status codes are just to get us out of this loop and

trunk/src/VBox/VMM/VMMAll/IEMAllThrdTables.h

@@ -207 +207 @@
             return VINF_IEM_RECOMPILE_END_TB; \
         \
+        uint8_t const idxInstrMc2 = pTb->cInstructions; \
         do { } while (0)
 #define IEM_MC2_EMIT_CALL_0(a_enmFunction) do { \
@@ -213 +214 @@
         PIEMTHRDEDCALLENTRY const pCall = &pTb->Thrd.paCalls[pTb->Thrd.cCalls++]; \
         pCall->enmFunction = a_enmFunction; \
+        pCall->idxInstr    = idxInstrMc2; \
+        pCall->uUnused0    = 0; \
         pCall->offOpcode   = offOpcodeMc2; \
         pCall->cbOpcode    = cbInstrMc2; \
@@ -226 +229 @@
         PIEMTHRDEDCALLENTRY const pCall = &pTb->Thrd.paCalls[pTb->Thrd.cCalls++]; \
         pCall->enmFunction = a_enmFunction; \
+        pCall->idxInstr    = idxInstrMc2; \
+        pCall->uUnused0    = 0; \
         pCall->offOpcode   = offOpcodeMc2; \
         pCall->cbOpcode    = cbInstrMc2; \
@@ -240 +245 @@
         PIEMTHRDEDCALLENTRY const pCall = &pTb->Thrd.paCalls[pTb->Thrd.cCalls++]; \
         pCall->enmFunction = a_enmFunction; \
+        pCall->idxInstr    = idxInstrMc2; \
+        pCall->uUnused0    = 0; \
         pCall->offOpcode   = offOpcodeMc2; \
         pCall->cbOpcode    = cbInstrMc2; \
@@ -255 +262 @@
         PIEMTHRDEDCALLENTRY const pCall = &pTb->Thrd.paCalls[pTb->Thrd.cCalls++]; \
         pCall->enmFunction = a_enmFunction; \
+        pCall->idxInstr    = idxInstrMc2; \
+        pCall->uUnused0    = 0; \
         pCall->offOpcode   = offOpcodeMc2; \
         pCall->cbOpcode    = cbInstrMc2; \