Changeset 30231 in vbox

Timestamp:

Jun 16, 2010 9:32:13 AM (15 years ago)

Author:

vboxsync

Message:

Chunk mapping calls can arrive from any thread now because of page fusion (shared page already mapped into the address space, but ring-3
data not updated yet)

File:

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

trunk/src/VBox/VMM/PGMPhys.cpp

@@ -3385 +3385 @@
     if (pVM->pgm.s.ChunkR3Map.c >= pVM->pgm.s.ChunkR3Map.cMax)
         Req.idChunkUnmap = pgmR3PhysChunkFindUnmapCandidate(pVM);
-/** @todo This is wrong. Any thread in the VM process should be able to do this,
- *        there are depenenecies on this.  What currently saves the day is that
- *        we don't unmap anything and that all non-zero memory will therefore
- *        be present when non-EMTs tries to access it.  */
-    rc = VMMR3CallR0(pVM, VMMR0_DO_GMM_MAP_UNMAP_CHUNK, 0, &Req.Hdr);
+
+    /* Must be callable from any thread, so can't use VMMR3CallR0. */
+    rc = SUPR3CallVMMR0Ex(pVM->pVMR0, pVCpu->idCpu, VMMR0_DO_GMM_MAP_UNMAP_CHUNK, 0, &Req.Hdr);
     if (RT_SUCCESS(rc))
     {