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

Timestamp:

Jun 10, 2016 8:28:24 AM (9 years ago)

Author:

vboxsync

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

107997

Message:

VMM/HM: Use unique, non-zero error codes for u32HMError and avoid dumping duplicate info. in gurus.

File:

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

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

@@ -3261 +3261 @@
                 break;
 
+            /* The guru will dump the HM error and exit history. Nothing extra to report for these errors. */
             case VERR_VMX_INVALID_VMXON_PTR:
-                break;
-
             case VERR_HM_UNSUPPORTED_CPU_FEATURE_COMBO:
             case VERR_VMX_INVALID_GUEST_STATE:
@@ -3272 +3271 @@
             case VERR_SVM_UNEXPECTED_XCPT_EXIT:
             case VERR_VMX_UNEXPECTED_INTERRUPTION_EXIT_TYPE:
-            {
-                LogRel(("HM: CPU[%u] HM error         %#x (%u)\n", i, pVCpu->hm.s.u32HMError, pVCpu->hm.s.u32HMError));
-                LogRel(("HM: CPU[%u] idxExitHistoryFree    %u\n", i, pVCpu->hm.s.idxExitHistoryFree));
-                unsigned const idxLast = pVCpu->hm.s.idxExitHistoryFree > 0 ?
-                                                                            pVCpu->hm.s.idxExitHistoryFree - 1 :
-                                                                            RT_ELEMENTS(pVCpu->hm.s.auExitHistory) - 1;
-                for (unsigned k = 0; k < RT_ELEMENTS(pVCpu->hm.s.auExitHistory); k++)
-                {
-                    LogRel(("HM: CPU[%u] auExitHistory[%2u]   = %#x (%u) %s\n", i, k, pVCpu->hm.s.auExitHistory[k],
-                            pVCpu->hm.s.auExitHistory[k], idxLast == k ? "<-- Last" : ""));
-                }
                 break;
-            }
         }
     }
@@ -3556 +3543 @@
                             idxLast == i ? "<-- Latest exit" : "");
         }
+        pHlp->pfnPrintf(pHlp, "HM error = %#x (%u)\n", pVCpu->hm.s.u32HMError, pVCpu->hm.s.u32HMError);
     }
     else