Changeset 17643 in vbox

Timestamp:

Mar 10, 2009 4:37:32 PM (16 years ago)

Author:

vboxsync

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

44129

Message:

Some more locking. We really need a better solution.

File:

• trunk/src/VBox/VMM/VMMAll/PGMAllBth.h (modified) (7 diffs)

trunk/src/VBox/VMM/VMMAll/PGMAllBth.h

@@ -988 +988 @@
 # endif /* PGM_SHW_TYPE == PGM_TYPE_AMD64 */
 
+# if defined(IN_RC)
+    /* Make sure the dynamic pPdeDst mapping will not be reused during this function. */
+    PGMDynLockHCPage(pVM, (uint8_t *)pPdeDst);
+# endif
+
     const SHWPDE PdeDst = *pPdeDst;
     if (!PdeDst.n.u1Present)
@@ -1250 +1255 @@
                     LogFlow(("Skipping flush for big page containing %RGv (PD=%X .u=%RX64)-> nothing has changed!\n", GCPtrPage, iPDSrc, PdeSrc.u));
                     STAM_COUNTER_INC(&pVM->pgm.s.CTX_MID_Z(Stat,InvalidatePage4MBPagesSkip));
+# if defined(IN_RC)
+                    /* Make sure the dynamic pPdeDst mapping will not be reused during this function. */
+                    PGMDynUnlockHCPage(pVM, (uint8_t *)pPdeDst);
+# endif
                     return VINF_SUCCESS;
                 }
@@ -1286 +1295 @@
         }
     }
-
+# if defined(IN_RC)
+    /* Make sure the dynamic pPdeDst mapping will not be reused during this function. */
+    PGMDynUnlockHCPage(pVM, (uint8_t *)pPdeDst);
+# endif
     return rc;
 
@@ -1643 +1655 @@
     PPGMPOOLPAGE    pShwPde  = pgmPoolGetPageByHCPhys(pVM, pPdptDst->a[iPdpt].u & X86_PDPE_PG_MASK);
     Assert(pShwPde);
+# endif
+
+# if defined(IN_RC)
+    /* Make sure the dynamic pPdeDst mapping will not be reused during this function. */
+    PGMDynLockHCPage(pVM, (uint8_t *)pPdeDst);
 # endif
 
@@ -1860 +1877 @@
                     LogFlow(("PGM_GCPHYS_2_PTR %RGp (big) failed with %Rrc\n", GCPhys, rc));
             }
+# if defined(IN_RC)
+            /* Make sure the dynamic pPdeDst mapping will not be reused during this function. */
+            PGMDynUnlockHCPage(pVM, (uint8_t *)pPdeDst);
+# endif
             return VINF_SUCCESS;
         }
@@ -1879 +1900 @@
 
     pPdeDst->u = 0;
+
+# if defined(IN_RC)
+    /* Make sure the dynamic pPdeDst mapping will not be reused during this function. */
+    PGMDynUnlockHCPage(pVM, (uint8_t *)pPdeDst);
+# endif
     PGM_INVL_GUEST_TLBS();
     return VINF_PGM_SYNCPAGE_MODIFIED_PDE;
@@ -3126 +3152 @@
     pPdeDst = &pPDDst->a[iPDDst];
 # endif
+
     if (!pPdeDst->n.u1Present)
     {
+# if defined(IN_RC)
+        /* Make sure the dynamic pPdeDst mapping will not be reused during this function. */
+        PGMDynLockHCPage(pVM, (uint8_t *)pPdeDst);
+# endif
         rc = PGM_BTH_NAME(SyncPT)(pVM, iPDSrc, pPDSrc, GCPtrPage);
+# if defined(IN_RC)
+        /* Make sure the dynamic pPdeDst mapping will not be reused during this function. */
+        PGMDynUnlockHCPage(pVM, (uint8_t *)pPdeDst);
+# endif
         AssertRC(rc);
         if (rc != VINF_SUCCESS)