Changeset 8860

Timestamp:

05/15/08 16:43:16 (3 months ago)

Author:

vboxsync

Message:

Some flushing statistics

Files:

• trunk/src/VBox/VMM/HWACCM.cpp (modified) (1 diff)
• trunk/src/VBox/VMM/HWACCMInternal.h (modified) (1 diff)
• trunk/src/VBox/VMM/VMMR0/HWSVMR0.cpp (modified) (5 diffs)

trunk/src/VBox/VMM/HWACCM.cpp

@@ -149 +149 @@
-    STAM_REG(pVM, &pVM->hwaccm.s.StatIntReinject,   STAMTYPE_COUNTER, "/HWACCM/Irq/Reinject",           STAMUNIT_OCCURENCES,    "Nr of occurances");
-    STAM_REG(pVM, &pVM->hwaccm.s.StatPendingHostIrq,STAMTYPE_COUNTER, "/HWACCM/Irq/PendingOnHost",      STAMUNIT_OCCURENCES,    "Nr of occurances");
+
+    STAM_REG(pVM, &pVM->hwaccm.s.StatFlushPageManual,       STAMTYPE_COUNTER, "/HWACCM/Flush/Page/Manual", STAMUNIT_OCCURENCES,    "Nr of occurances");
+    STAM_REG(pVM, &pVM->hwaccm.s.StatFlushTLBManual,        STAMTYPE_COUNTER, "/HWACCM/Flush/TLB/Manual",  STAMUNIT_OCCURENCES,    "Nr of occurances");
+    STAM_REG(pVM, &pVM->hwaccm.s.StatFlushTLBCRxChange,     STAMTYPE_COUNTER, "/HWACCM/Flush/TLB/CRx",     STAMUNIT_OCCURENCES,    "Nr of occurances");
+    STAM_REG(pVM, &pVM->hwaccm.s.StatFlushPageInvlpg,       STAMTYPE_COUNTER, "/HWACCM/Flush/Page/Invlpg", STAMUNIT_OCCURENCES,    "Nr of occurances");
+    STAM_REG(pVM, &pVM->hwaccm.s.StatFlushTLBWorldSwitch,   STAMTYPE_COUNTER, "/HWACCM/Flush/TLB/Switch",  STAMUNIT_OCCURENCES,    "Nr of occurances");
+    STAM_REG(pVM, &pVM->hwaccm.s.StatNoFlushTLBWorldSwitch, STAMTYPE_COUNTER, "/HWACCM/Flush/TLB/Skipped", STAMUNIT_OCCURENCES,    "Nr of occurances");
-
-    pVM->hwaccm.s.pStatExitReason = 0;

trunk/src/VBox/VMM/HWACCMInternal.h

@@ -308 +308 @@
-    STAMCOUNTER             StatPendingHostIrq;
-
+    STAMCOUNTER             StatFlushPageManual;
+    STAMCOUNTER             StatFlushTLBManual;
+    STAMCOUNTER             StatFlushPageInvlpg;
+    STAMCOUNTER             StatFlushTLBWorldSwitch;
+    STAMCOUNTER             StatNoFlushTLBWorldSwitch;
+    STAMCOUNTER             StatFlushTLBCRxChange;
+
-    STAMCOUNTER             StatSwitchGuestIrq;
-    STAMCOUNTER             StatSwitchToR3;

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

@@ -772 +772 @@
-    /* Make sure we flush the TLB when required. */
-    pVMCB->ctrl.TLBCtrl.n.u1TLBFlush = pVM->hwaccm.s.svm.fForceTLBFlush;
+#ifdef VBOX_WITH_STATISTICS
+    if (pVMCB->ctrl.TLBCtrl.n.u1TLBFlush)
+        STAM_COUNTER_INC(&pVM->hwaccm.s.StatFlushTLBWorldSwitch);
+    else
+        STAM_COUNTER_INC(&pVM->hwaccm.s.StatNoFlushTLBWorldSwitch);
+#endif
-
-    /* In case we execute a goto ResumeExecution later on. */
@@ -1245 +1251 @@
-        rc = SVMR0InterpretInvpg(pVM, CPUMCTX2CORE(pCtx), pVMCB->ctrl.TLBCtrl.n.u32ASID);
-        if (rc == VINF_SUCCESS)
+        {
+            STAM_COUNTER_INC(&pVM->hwaccm.s.StatFlushPageInvlpg);
-            goto ResumeExecution;   /* eip already updated */
+        }
-        break;
-    }
@@ -1283 +1292 @@
-            AssertRC(rc);
-
+            STAM_COUNTER_INC(&pVM->hwaccm.s.StatFlushTLBCRxChange);
+            
-            /** @note Force a TLB flush. SVM requires us to do it manually. */
-            pVM->hwaccm.s.svm.fForceTLBFlush = true;
@@ -1722 +1733 @@
-    AssertMsgReturn(pVMCB, ("Invalid pVMCB\n"), VERR_EM_INTERNAL_ERROR);
-
+    STAM_COUNTER_INC(&pVM->hwaccm.s.StatFlushPageManual);
-    SVMInvlpgA(GCVirt, pVMCB->ctrl.TLBCtrl.n.u32ASID);
-    return VINF_SUCCESS;
@@ -1736 +1748 @@
-    Log2(("SVMR0FlushTLB\n"));
-    pVM->hwaccm.s.svm.fForceTLBFlush = true;
-
+
-    return VINF_SUCCESS;
-}