Changeset 17431 in vbox

Timestamp:

Mar 6, 2009 2:02:03 AM (16 years ago)

Author:

vboxsync

Message:

GMMR0: Two fixes to the code path freeing pages.

File:

• trunk/src/VBox/VMM/VMMR0/GMMR0.cpp (modified) (5 diffs)

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

@@ -304 +304 @@
 /** @def GMM_PAGE_IS_PRIVATE
  *
- * @returns true if free, false if not.
+ * @returns true if private, false if not.
  * @param   pPage       The GMM page.
  */
@@ -313 +313 @@
 #endif
 
-/** @def GMM_PAGE_IS_FREE
- *
- * @returns true if free, false if not.
+/** @def GMM_PAGE_IS_SHARED
+ *
+ * @returns true if shared, false if not.
  * @param   pPage       The GMM page.
  */
@@ -1521 +1521 @@
 
 /**
- * Allocates one page.
+ * Allocates one private page.
  *
  * Worker for gmmR0AllocatePages.
@@ -1537 +1537 @@
     Assert(pChunk->cFree);
     pChunk->cFree--;
+    pChunk->cPrivate++;
 
     /* unlink the first free page. */
@@ -2260 +2261 @@
     switch (enmAccount)
     {
-        case GMMACCOUNT_BASE:   pGVM->gmm.s.Allocated.cBasePages   -= iPage;
-        case GMMACCOUNT_SHADOW: pGVM->gmm.s.Allocated.cShadowPages -= iPage;
-        case GMMACCOUNT_FIXED:  pGVM->gmm.s.Allocated.cFixedPages  -= iPage;
+        case GMMACCOUNT_BASE:   pGVM->gmm.s.Allocated.cBasePages   -= iPage; break;
+        case GMMACCOUNT_SHADOW: pGVM->gmm.s.Allocated.cShadowPages -= iPage; break;
+        case GMMACCOUNT_FIXED:  pGVM->gmm.s.Allocated.cFixedPages  -= iPage; break;
         default:
             AssertMsgFailedReturn(("enmAccount=%d\n", enmAccount), VERR_INTERNAL_ERROR);