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

Timestamp:

Aug 28, 2012 2:47:29 PM (12 years ago)

Author:

vboxsync

Message:

VMM: Must flush changes pending in the handy page array before freeing memory. There may be requests to free shared pages in there, esp. in a guest shutdown scenario.

File:

• trunk/src/VBox/VMM/VMMR3/PGMPhys.cpp (modified) (1 diff)

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

@@ -2059 +2059 @@
 
 #ifdef VBOX_WITH_PAGE_SHARING
-    /* Clear all registered shared modules. */
+    /*
+     * Clear all registered shared modules.
+     */
     pgmR3PhysAssertSharedPageChecksums(pVM);
     rc = GMMR3ResetSharedModules(pVM);
     AssertRC(rc);
+
+    /*
+     * Flush the handy pages updates to make sure no shared pages are hiding
+     * in there.  (No unlikely if the VM shuts down, apparently.)
+     */
+    rc = VMMR3CallR0(pVM, VMMR0_DO_PGM_FLUSH_HANDY_PAGES, 0, NULL);
 #endif