Changeset 20753 in vbox

Timestamp:

Jun 21, 2009 11:11:24 PM (15 years ago)

Author:

vboxsync

Message:

VMM: Added PDMR3CritSectLeaveAll for bulk unlocking during guru meditation.

Location:

trunk

Files:

• include/VBox/pdmcritsect.h (modified) (1 diff)
• src/VBox/VMM/PDMCritSect.cpp (modified) (2 diffs)
• src/VBox/VMM/VMMAll/PDMAllCritSect.cpp (modified) (1 diff)
• src/VBox/VMM/VMMGuruMeditation.cpp (modified) (3 diffs)

trunk/include/VBox/pdmcritsect.h

@@ -71 +71 @@
 VMMDECL(void)       PDMCritSectFF(PVMCPU pVCpu);
 VMMR3DECL(uint32_t) PDMR3CritSectCountOwned(PVM pVM, char *pszNames, size_t cbNames);
+VMMR3DECL(void)     PDMR3CritSectLeaveAll(PVM pVM);
 
 /** @} */

trunk/src/VBox/VMM/PDMCritSect.cpp

@@ -26 +26 @@
 #define LOG_GROUP LOG_GROUP_PDM//_CRITSECT
 #include "PDMInternal.h"
-#include <VBox/pdm.h>
+#include <VBox/pdmcritsect.h>
 #include <VBox/mm.h>
 #include <VBox/vm.h>
@@ -476 +476 @@
     return cCritSects;
 }
+
+
+/**
+ * Leave all critical sections the calling thread owns.
+ *
+ * @param   pVM         The VM handle.
+ */
+void PDMR3CritSectLeaveAll(PVM pVM)
+{
+    RTNATIVETHREAD const hNativeSelf = RTThreadNativeSelf();
+
+    RTCritSectEnter(&pVM->pdm.s.MiscCritSect);
+    for (PPDMCRITSECTINT pCur = pVM->pdm.s.pCritSects;
+         pCur;
+         pCur = pCur->pNext)
+    {
+        while (     pCur->Core.NativeThreadOwner == hNativeSelf
+               &&   pCur->Core.cNestings > 0)
+            PDMCritSectLeave((PPDMCRITSECT)pCur);
+    }
+    RTCritSectLeave(&pVM->pdm.s.MiscCritSect);
+}
+

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

@@ -26 +26 @@
 #define LOG_GROUP LOG_GROUP_PDM//_CRITSECT
 #include "PDMInternal.h"
-#include <VBox/pdm.h>
+#include <VBox/pdmcritsect.h>
 #include <VBox/mm.h>
 #include <VBox/vm.h>

trunk/src/VBox/VMM/VMMGuruMeditation.cpp

@@ -26 +26 @@
 #include <VBox/vmm.h>
 #include <VBox/pdmapi.h>
+#include <VBox/pdmcritsect.h>
 #include <VBox/trpm.h>
 #include <VBox/dbgf.h>
@@ -213 +214 @@
 
     /* Release owned locks to make sure other VCPUs can continue in case they were waiting for one. */
+#if 1
+    PDMR3CritSectLeaveAll(pVM);
+#else
     MMR3ReleaseOwnedLocks(pVM);
     PGMR3ReleaseOwnedLocks(pVM);
@@ -218 +222 @@
     IOMR3ReleaseOwnedLocks(pVM);
     EMR3ReleaseOwnedLocks(pVM);
+#endif
 
     /*