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

Timestamp:

Mar 15, 2018 9:15:57 AM (7 years ago)

Author:

vboxsync

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

121295

Message:

VMM/HM: Add nested-guest exit reason stat array for SVM R0 execution.

File:

• trunk/src/VBox/VMM/VMMR3/HM.cpp (modified) (2 diffs)

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

@@ -1043 +1043 @@
 #undef HM_REG_COUNTER
 
+        const char *const *papszDesc = ASMIsIntelCpu() || ASMIsViaCentaurCpu() ? &g_apszVTxExitReasons[0]
+                                                                               : &g_apszAmdVExitReasons[0];
+
+        /*
+         * Guest Exit reason stats.
+         */
         pVCpu->hm.s.paStatExitReason = NULL;
-
         rc = MMHyperAlloc(pVM, MAX_EXITREASON_STAT * sizeof(*pVCpu->hm.s.paStatExitReason), 0 /* uAlignment */, MM_TAG_HM,
                           (void **)&pVCpu->hm.s.paStatExitReason);
-        AssertRC(rc);
-        if (RT_SUCCESS(rc))
-        {
-            const char *const *papszDesc = ASMIsIntelCpu() || ASMIsViaCentaurCpu() ?
-                                           &g_apszVTxExitReasons[0] : &g_apszAmdVExitReasons[0];
-            for (int j = 0; j < MAX_EXITREASON_STAT; j++)
+        AssertRCReturn(rc, rc);
+        for (int j = 0; j < MAX_EXITREASON_STAT; j++)
+        {
+            if (papszDesc[j])
             {
-                if (papszDesc[j])
-                {
-                    rc = STAMR3RegisterF(pVM, &pVCpu->hm.s.paStatExitReason[j], STAMTYPE_COUNTER, STAMVISIBILITY_USED,
-                                         STAMUNIT_OCCURENCES, papszDesc[j], "/HM/CPU%d/Exit/Reason/%02x", i, j);
-                    AssertRC(rc);
-                }
+                rc = STAMR3RegisterF(pVM, &pVCpu->hm.s.paStatExitReason[j], STAMTYPE_COUNTER, STAMVISIBILITY_USED,
+                                     STAMUNIT_OCCURENCES, papszDesc[j], "/HM/CPU%d/Exit/Reason/%02x", i, j);
+                AssertRCReturn(rc, rc);
             }
-            rc = STAMR3RegisterF(pVM, &pVCpu->hm.s.StatExitReasonNpf, STAMTYPE_COUNTER, STAMVISIBILITY_USED, STAMUNIT_OCCURENCES,
-                                 "Nested page fault", "/HM/CPU%d/Exit/Reason/#NPF", i);
-            AssertRC(rc);
-        }
+        }
+        rc = STAMR3RegisterF(pVM, &pVCpu->hm.s.StatExitReasonNpf, STAMTYPE_COUNTER, STAMVISIBILITY_USED, STAMUNIT_OCCURENCES,
+                             "Nested page fault", "/HM/CPU%d/Exit/Reason/#NPF", i);
+        AssertRCReturn(rc, rc);
         pVCpu->hm.s.paStatExitReasonR0 = MMHyperR3ToR0(pVM, pVCpu->hm.s.paStatExitReason);
 # ifdef VBOX_WITH_2X_4GB_ADDR_SPACE
@@ -1072 +1072 @@
 # endif
 
+#ifdef VBOX_WITH_NESTED_HWVIRT
+        /*
+         * Nested-guest Exit reason stats.
+         */
+        pVCpu->hm.s.paStatNestedExitReason = NULL;
+        rc = MMHyperAlloc(pVM, MAX_EXITREASON_STAT * sizeof(*pVCpu->hm.s.paStatNestedExitReason), 0 /* uAlignment */, MM_TAG_HM,
+                          (void **)&pVCpu->hm.s.paStatNestedExitReason);
+        AssertRCReturn(rc, rc);
+        for (int j = 0; j < MAX_EXITREASON_STAT; j++)
+        {
+            if (papszDesc[j])
+            {
+                rc = STAMR3RegisterF(pVM, &pVCpu->hm.s.paStatNestedExitReason[j], STAMTYPE_COUNTER, STAMVISIBILITY_USED,
+                                     STAMUNIT_OCCURENCES, papszDesc[j], "/HM/CPU%d/NestedExit/Reason/%02x", i, j);
+                AssertRC(rc);
+            }
+        }
+        rc = STAMR3RegisterF(pVM, &pVCpu->hm.s.StatNestedExitReasonNpf, STAMTYPE_COUNTER, STAMVISIBILITY_USED,
+                             STAMUNIT_OCCURENCES, "Nested page fault", "/HM/CPU%d/NestedExit/Reason/#NPF", i);
+        AssertRCReturn(rc, rc);
+        pVCpu->hm.s.paStatNestedExitReasonR0 = MMHyperR3ToR0(pVM, pVCpu->hm.s.paStatNestedExitReason);
+# ifdef VBOX_WITH_2X_4GB_ADDR_SPACE
+        Assert(pVCpu->hm.s.paStatNestedExitReasonR0 != NIL_RTR0PTR || !HMIsEnabled(pVM));
+# else
+        Assert(pVCpu->hm.s.paStatNestedExitReasonR0 != NIL_RTR0PTR);
+# endif
+#endif
+
+        /*
+         * Injected events stats.
+         */
         rc = MMHyperAlloc(pVM, sizeof(STAMCOUNTER) * 256, 8, MM_TAG_HM, (void **)&pVCpu->hm.s.paStatInjectedIrqs);
         AssertRCReturn(rc, rc);