Changeset 18957 in vbox for trunk/src/VBox

Timestamp:

Apr 16, 2009 5:03:19 PM (16 years ago)

Author:

vboxsync

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

46039

Message:

Fixed locked array size (was too small) and caused corruption.

Location:

trunk/src/VBox/VMM

Files:

• PGMInternal.h (modified) (1 diff)
• VMMAll/PGMAll.cpp (modified) (1 diff)

trunk/src/VBox/VMM/PGMInternal.h

@@ -2300 +2300 @@
      * The cache size is covering half of the mapping area. */
     RTHCPHYS                        aHCPhysDynPageMapCache[MM_HYPER_DYNAMIC_SIZE >> (PAGE_SHIFT + 1)];
-    uint32_t                        aLockedDynPageMapCache[MM_HYPER_DYNAMIC_SIZE >> (PAGE_SHIFT + 1)];
+    /**
+     * Keep a lock counter for the full (!) mapping area. */
+    uint32_t                        aLockedDynPageMapCache[MM_HYPER_DYNAMIC_SIZE >> (PAGE_SHIFT)];
 
     /** The address of the ring-0 mapping cache if we're making use of it.  */

trunk/src/VBox/VMM/VMMAll/PGMAll.cpp

@@ -2306 +2306 @@
     unsigned iPage;
 
-    AssertCompile(RT_ELEMENTS(pVM->pgm.s.aHCPhysDynPageMapCache) == RT_ELEMENTS(pVM->pgm.s.aLockedDynPageMapCache));
+    AssertCompile(RT_ELEMENTS(pVM->pgm.s.aLockedDynPageMapCache) == 2* RT_ELEMENTS(pVM->pgm.s.aHCPhysDynPageMapCache));
 
     Assert(GCPage >= pVM->pgm.s.pbDynPageMapBaseGC && GCPage < (pVM->pgm.s.pbDynPageMapBaseGC + MM_HYPER_DYNAMIC_SIZE));