Changeset 19674 in vbox

Timestamp:

May 14, 2009 7:57:59 AM (16 years ago)

Author:

vboxsync

Message:

Use the pgm lock during pgmPoolAlloc.

File:

• trunk/src/VBox/VMM/VMMAll/PGMAllPool.cpp (modified) (5 diffs)

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

@@ -4213 +4213 @@
      *  Assert(!(pVM->pgm.s.fGlobalSyncFlags & PGM_GLOBAL_SYNC_CLEAR_PGM_POOL)); */
 
+    pgmLock(pVM);
+
 #ifdef PGMPOOL_WITH_CACHE
     if (pPool->fCacheEnabled)
@@ -4219 +4221 @@
         if (RT_SUCCESS(rc2))
         {
+            pgmUnlock(pVM);
             STAM_PROFILE_ADV_STOP(&pPool->StatAlloc, a);
             LogFlow(("pgmPoolAlloc: cached returns %Rrc *ppPage=%p:{.Key=%RHp, .idx=%d}\n", rc2, *ppPage, (*ppPage)->Core.Key, (*ppPage)->idx));
@@ -4236 +4239 @@
         if (RT_FAILURE(rc))
         {
+            pgmUnlock(pVM);
             Log(("pgmPoolAlloc: returns %Rrc (Free)\n", rc));
             STAM_PROFILE_ADV_STOP(&pPool->StatAlloc, a);
@@ -4281 +4285 @@
         pPage->iNext = pPool->iFreeHead;
         pPool->iFreeHead = pPage->idx;
+        pgmUnlock(pVM);
         STAM_PROFILE_ADV_STOP(&pPool->StatAlloc, a);
         Log(("pgmPoolAlloc: returns %Rrc (Insert)\n", rc3));
@@ -4304 +4309 @@
 
     *ppPage = pPage;
+    pgmUnlock(pVM);
     LogFlow(("pgmPoolAlloc: returns %Rrc *ppPage=%p:{.Key=%RHp, .idx=%d, .fCached=%RTbool, .fMonitored=%RTbool}\n",
              rc, pPage, pPage->Core.Key, pPage->idx, pPage->fCached, pPage->fMonitored));