Changeset 23792 in vbox for trunk/src

Timestamp:

Oct 15, 2009 7:27:40 AM (16 years ago)

Author:

vboxsync

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

53530

Message:

Similar unexpected return value hack for nested paging (guest SMP).

File:

• trunk/src/VBox/VMM/VMMR0/PGMR0.cpp (modified) (1 diff)

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

@@ -258 +258 @@
     if (rc == VINF_PGM_SYNCPAGE_MODIFIED_PDE)
         rc = VINF_SUCCESS;
+    else
+    /* Note: hack alert for difficult to reproduce problem. */
+    if (    rc == VERR_PAGE_TABLE_NOT_PRESENT           /* seen with UNI & SMP */
+        ||  rc == VERR_PAGE_DIRECTORY_PTR_NOT_PRESENT   /* seen with SMP */
+        ||  rc == VERR_PAGE_MAP_LEVEL4_NOT_PRESENT)     /* precaution */
+    {
+        Log(("WARNING: Unexpected VERR_PAGE_TABLE_NOT_PRESENT (%d) for page fault at %RGp error code %x (rip=%RGv)\n", rc, pvFault, uErr, pRegFrame->rip));
+        /* Some kind of inconsistency in the SMP case; it's safe to just execute the instruction again; not sure about single VCPU VMs though. */
+        rc = VINF_SUCCESS;
+    }
+
     STAM_STATS({ if (!pVCpu->pgm.s.CTX_SUFF(pStatTrap0eAttribution))
                     pVCpu->pgm.s.CTX_SUFF(pStatTrap0eAttribution) = &pVCpu->pgm.s.StatRZTrap0eTime2Misc; });