Changeset 60899 in vbox for trunk/src/VBox

Timestamp:

May 9, 2016 3:44:49 PM (9 years ago)

Author:

vboxsync

Message:

PGMAllBth.h: Improved assertion so it's possible to tell if it's valid.

File:

• trunk/src/VBox/VMM/VMMAll/PGMAllBth.h (modified) (2 diffs)

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

@@ -1070 +1070 @@
 #   endif
 #   ifdef VBOX_STRICT
-                RTGCPHYS GCPhys2;
-                uint64_t fPageGst;
+                RTGCPHYS GCPhys2 = RTGCPHYS_MAX;
+                uint64_t fPageGst = UINT64_MAX;
                 if (!pVM->pgm.s.fNestedPaging)
                 {
@@ -1081 +1081 @@
                 rc = PGMShwGetPage(pVCpu, pvFault, &fPageShw, NULL);
                 AssertMsg((RT_SUCCESS(rc) && (fPageShw & X86_PTE_RW)) || pVM->cCpus > 1 /* new monitor can be installed/page table flushed between the trap exit and PGMTrap0eHandler */,
-                          ("rc=%Rrc fPageShw=%RX64\n", rc, fPageShw));
+                          ("rc=%Rrc fPageShw=%RX64 GCPhys2=%RGp fPageGst=%RX64\n", rc, fPageShw, GCPhys2, fPageGst)); /* Bogus? Can we end up mapping MMIO here for instance, or is that handled above already?  */
 #   endif /* VBOX_STRICT */
                 STAM_STATS({ pVCpu->pgm.s.CTX_SUFF(pStatTrap0eAttribution) = &pVCpu->pgm.s.CTX_SUFF(pStats)->StatRZTrap0eTime2OutOfSyncHndObs; });