Changeset 79539 in vbox for trunk/src

Timestamp:

Jul 5, 2019 7:21:31 AM (6 years ago)

Author:

vboxsync

Message:

VMM/HMSVMR0: Nested SVM: bugref:7243 Add missing exception stat counters.

File:

• trunk/src/VBox/VMM/VMMR0/HMSVMR0.cpp (modified) (7 diffs)

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

@@ -7615 +7615 @@
     HMSVM_VALIDATE_EXIT_HANDLER_PARAMS(pVCpu, pSvmTransient);
     HMSVM_ASSERT_NOT_IN_NESTED_GUEST(&pVCpu->cpum.GstCtx);
+    STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestUD);
 
     /* Paranoia; Ensure we cannot be called as a result of event delivery. */
@@ -7661 +7662 @@
     HMSVM_VALIDATE_EXIT_HANDLER_PARAMS(pVCpu, pSvmTransient);
     HMSVM_CPUMCTX_IMPORT_STATE(pVCpu, HMSVM_CPUMCTX_EXTRN_ALL);
+    STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestMF);
 
     PCPUMCTX pCtx  = &pVCpu->cpum.GstCtx;
@@ -7702 +7704 @@
     HMSVM_CPUMCTX_IMPORT_STATE(pVCpu, HMSVM_CPUMCTX_EXTRN_ALL);
     HMSVM_CHECK_EXIT_DUE_TO_EVENT_DELIVERY(pVCpu, pSvmTransient);
+    STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestDB);
 
     if (RT_UNLIKELY(pVCpu->hm.s.Event.fPending))
@@ -7758 +7761 @@
     HMSVM_VALIDATE_EXIT_HANDLER_PARAMS(pVCpu, pSvmTransient);
     HMSVM_CHECK_EXIT_DUE_TO_EVENT_DELIVERY(pVCpu, pSvmTransient);
+    STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestAC);
 
     SVMEVENT Event;
@@ -7779 +7783 @@
     HMSVM_CPUMCTX_IMPORT_STATE(pVCpu, HMSVM_CPUMCTX_EXTRN_ALL);
     HMSVM_CHECK_EXIT_DUE_TO_EVENT_DELIVERY(pVCpu, pSvmTransient);
+    STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestBP);
 
     PCPUMCTX pCtx = &pVCpu->cpum.GstCtx;
@@ -7877 +7882 @@
     HMSVM_VALIDATE_EXIT_HANDLER_PARAMS(pVCpu, pSvmTransient);
     HMSVM_CHECK_EXIT_DUE_TO_EVENT_DELIVERY(pVCpu, pSvmTransient);
+    STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestGP);
 
     PCSVMVMCB pVmcb = hmR0SvmGetCurrentVmcb(pVCpu);
@@ -7937 +7943 @@
     }
 
+#ifdef VBOX_WITH_STATISTICS
+    switch (uVector)
+    {
+        case SVM_EXIT_XCPT_DE:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestDE);      break;
+        /*   SVM_EXIT_XCPT_DB: */          STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestDB);      break;
+        /*   SVM_EXIT_XCPT_NMI: */         /* Handled elsewhere. */
+        /*   SVM_EXIT_XCPT_BP: */          STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestBP);      break;
+        case SVM_EXIT_XCPT_OF:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestOF);      break;
+        case SVM_EXIT_XCPT_BR:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestBR);      break;
+        /*   SVM_EXIT_XCPT_UD: */          STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestUD);      break;
+        case SVM_EXIT_XCPT_NM:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestOF);      break;
+        case SVM_EXIT_XCPT_DF:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestDF);      break;
+        case SVM_EXIT_XCPT_CO_SEG_OVERRUN: /* Legacy */                                         break;
+        case SVM_EXIT_XCPT_TS:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestTS);      break;
+        case SVM_EXIT_XCPT_NP:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestNP);      break;
+        case SVM_EXIT_XCPT_SS:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestSS);      break;
+        case SVM_EXIT_XCPT_GP:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestGP);      break;
+        /*   SVM_EXIT_XCPT_PF: */          STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestPF);      break;
+        case SVM_EXIT_XCPT_15:             /* Reserved.      */
+        /*   SVM_EXIT_XCPT_MF: */          STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestMF);      break;
+        /*   SVM_EXIT_XCPT_AC: */          STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestAC);      break;
+        case SVM_EXIT_XCPT_MC:             /* Machine-check exceptions shouldn't happen. */     break;
+        case SVM_EXIT_XCPT_XF:             STAM_COUNTER_INC(&pVCpu->hm.s.StatExitGuestXF);      break;
+    }
+#endif
+
     hmR0SvmSetPendingEvent(pVCpu, &Event, 0 /* GCPtrFaultAddress */);
     return VINF_SUCCESS;