Changeset 80 in vbox for trunk/src/VBox/VMM/PGMPhys.cpp

Timestamp:

Jan 17, 2007 8:45:10 AM (18 years ago)

Author:

vboxsync

Message:

PGMR3PhysGrowRange: a request made from another thread may end up in EMT after somebody else has already allocated the range

File:

• trunk/src/VBox/VMM/PGMPhys.cpp (modified) (2 diffs)

trunk/src/VBox/VMM/PGMPhys.cpp

@@ -304 +304 @@
      */
     pgmLock(pVM);
+
     PPGMRAMRANGE pRam = CTXSUFF(pVM->pgm.s.pRamRanges);
     while (pRam)
@@ -311 +312 @@
             &&  (pRam->fFlags & MM_RAM_FLAGS_DYNAMIC_ALLOC))
         {
+            bool     fRangeExists = false;
+            unsigned off = (GCPhys - pRam->GCPhys) >> PGM_DYNAMIC_CHUNK_SHIFT;
+
+            /** @note A request made from another thread may end up in EMT after somebody else has already allocated the range. */
+            if (pRam->pavHCChunkHC[off])
+                fRangeExists = true;
+
             pgmUnlock(pVM);
+            if (fRangeExists)
+                return VINF_SUCCESS;
             return pgmr3PhysGrowRange(pVM, GCPhys);
         }