Changeset 37419 in vbox for trunk/src/VBox/Runtime/generic

Timestamp:

Jun 11, 2011 8:25:37 PM (14 years ago)

Author:

vboxsync

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

72218

Message:

PDM/IPRT CritSect: Introduced the NOP critical section for simplifying locking in IOM and TM. (Revisiting device emulation locking, making it more fine grained over time.)

File:

• trunk/src/VBox/Runtime/generic/critsect-generic.cpp (modified) (6 diffs)

trunk/src/VBox/Runtime/generic/critsect-generic.cpp

@@ -52 +52 @@
                              const char *pszNameFmt, ...)
 {
-    AssertReturn(!(fFlags & ~(RTCRITSECT_FLAGS_NO_NESTING | RTCRITSECT_FLAGS_NO_LOCK_VAL | RTCRITSECT_FLAGS_BOOTSTRAP_HACK)),
+    AssertReturn(!(fFlags & ~(RTCRITSECT_FLAGS_NO_NESTING | RTCRITSECT_FLAGS_NO_LOCK_VAL | RTCRITSECT_FLAGS_BOOTSTRAP_HACK | RTCRITSECT_FLAGS_NOP)),
                  VERR_INVALID_PARAMETER);
 
@@ -66 +66 @@
     int rc = VINF_SUCCESS;
 #ifdef RTCRITSECT_STRICT
-    if (!(fFlags & RTCRITSECT_FLAGS_BOOTSTRAP_HACK))
+    if (!(fFlags & (RTCRITSECT_FLAGS_BOOTSTRAP_HACK | RTCRITSECT_FLAGS_NOP)))
     {
         if (!pszNameFmt)
@@ -111 +111 @@
     AssertPtrReturn(pCritSect, RTLOCKVAL_SUB_CLASS_INVALID);
     AssertReturn(pCritSect->u32Magic == RTCRITSECT_MAGIC, RTLOCKVAL_SUB_CLASS_INVALID);
+    AssertReturn(!(pCritSect->fFlags & RTCRITSECT_FLAGS_NOP), RTLOCKVAL_SUB_CLASS_INVALID);
     return RTLockValidatorRecExclSetSubClass(pCritSect->pValidatorRec, uSubClass);
 #else
@@ -122 +123 @@
     Assert(pCritSect);
     Assert(pCritSect->u32Magic == RTCRITSECT_MAGIC);
-    RTNATIVETHREAD  NativeThreadSelf = RTThreadNativeSelf();
+    /*AssertReturn(pCritSect->u32Magic == RTCRITSECT_MAGIC, VERR_SEM_DESTROYED);*/
+
+    /*
+     * Return straight away if NOP.
+     */
+    if (pCritSect->fFlags & RTCRITSECT_FLAGS_NOP)
+        return VINF_SUCCESS;
 
     /*
      * Try take the lock. (cLockers is -1 if it's free)
      */
+    RTNATIVETHREAD NativeThreadSelf = RTThreadNativeSelf();
     if (!ASMAtomicCmpXchgS32(&pCritSect->cLockers, 0, -1))
     {
@@ -186 +194 @@
 DECL_FORCE_INLINE(int) rtCritSectEnter(PRTCRITSECT pCritSect, PCRTLOCKVALSRCPOS pSrcPos)
 {
-    Assert(pCritSect);
-    Assert(pCritSect->u32Magic == RTCRITSECT_MAGIC);
+    AssertPtr(pCritSect);
+    AssertReturn(pCritSect->u32Magic == RTCRITSECT_MAGIC, VERR_SEM_DESTROYED);
+
+    /*
+     * Return straight away if NOP.
+     */
+    if (pCritSect->fFlags & RTCRITSECT_FLAGS_NOP)
+        return VINF_SUCCESS;
+
+    /*
+     * How is calling and is the order right?
+     */
     RTNATIVETHREAD  NativeThreadSelf = RTThreadNativeSelf();
-
-    /* If the critical section has already been destroyed, then inform the caller. */
-    if (pCritSect->u32Magic != RTCRITSECT_MAGIC)
-        return VERR_SEM_DESTROYED;
-
-#ifdef RTCRITSECT_STRICT
-    RTTHREAD hThreadSelf = pCritSect->pValidatorRec
-                         ? RTThreadSelfAutoAdopt()
-                         : RTThreadSelf();
-    int      rc9;
+#ifdef RTCRITSECT_STRICT
+    RTTHREAD        hThreadSelf = pCritSect->pValidatorRec
+                                ? RTThreadSelfAutoAdopt()
+                                : RTThreadSelf();
+    int             rc9;
     if (pCritSect->pValidatorRec) /* (bootstap) */
     {
@@ -305 +318 @@
 {
     /*
-     * Assert ownership and so on.
+     * Assert sanity and check for NOP.
      */
     Assert(pCritSect);
     Assert(pCritSect->u32Magic == RTCRITSECT_MAGIC);
+    if (pCritSect->fFlags & RTCRITSECT_FLAGS_NOP)
+        return VINF_SUCCESS;
+
+    /*
+     * Assert ownership and so on.
+     */
     Assert(pCritSect->cNestings > 0);
     Assert(pCritSect->cLockers >= 0);