Changeset 38944 in vbox for trunk/src/VBox

Timestamp:

Oct 5, 2011 2:03:53 PM (13 years ago)

Author:

vboxsync

Message:

PDMCritSectLeave: Indicate whether it was a nested exit or a real one.

File:

• trunk/src/VBox/VMM/VMMAll/PDMAllCritSect.cpp (modified) (6 diffs)

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

@@ -491 +491 @@
  * Leaves a critical section entered with PDMCritSectEnter().
  *
+ * @returns Indication whether we really exited the critical section.
+ * @retval  VINF_SUCCESS if we really exited.
+ * @retval  VINF_SEM_NESTED if we only reduced the nesting count.
+ * @retval  VERR_NOT_OWNER if you somehow ignore release assertions.
+ *
  * @param   pCritSect           The PDM critical section to leave.
  */
-VMMDECL(void) PDMCritSectLeave(PPDMCRITSECT pCritSect)
+VMMDECL(int) PDMCritSectLeave(PPDMCRITSECT pCritSect)
 {
     AssertMsg(pCritSect->s.Core.u32Magic == RTCRITSECT_MAGIC, ("%p %RX32\n", pCritSect, pCritSect->s.Core.u32Magic));
@@ -500 +505 @@
     /* Check for NOP sections before asserting ownership. */
     if (pCritSect->s.Core.fFlags & RTCRITSECT_FLAGS_NOP)
-        return;
+        return VINF_SUCCESS;
 
     /*
@@ -506 +511 @@
      */
     RTNATIVETHREAD const hNativeSelf = pdmCritSectGetNativeSelf(pCritSect);
-    AssertReleaseMsgReturnVoid(pCritSect->s.Core.NativeThreadOwner == hNativeSelf,
-                               ("%p %s: %p != %p; cLockers=%d cNestings=%d\n", pCritSect, R3STRING(pCritSect->s.pszName),
-                                pCritSect->s.Core.NativeThreadOwner, hNativeSelf,
-                                pCritSect->s.Core.cLockers, pCritSect->s.Core.cNestings));
+    AssertReleaseMsgReturn(pCritSect->s.Core.NativeThreadOwner == hNativeSelf,
+                           ("%p %s: %p != %p; cLockers=%d cNestings=%d\n", pCritSect, R3STRING(pCritSect->s.pszName),
+                            pCritSect->s.Core.NativeThreadOwner, hNativeSelf,
+                            pCritSect->s.Core.cLockers, pCritSect->s.Core.cNestings),
+                           VERR_NOT_OWNER);
     Assert(pCritSect->s.Core.cNestings >= 1);
 
@@ -521 +527 @@
         ASMAtomicDecS32(&pCritSect->s.Core.cLockers);
         Assert(pCritSect->s.Core.cLockers >= 0);
-        return;
+        return VINF_SEM_NESTED;
     }
 
@@ -595 +601 @@
             ASMAtomicWriteHandle(&pCritSect->s.Core.NativeThreadOwner, NIL_RTNATIVETHREAD);
             if (ASMAtomicCmpXchgS32(&pCritSect->s.Core.cLockers, -1, 0))
-                return;
+                return VINF_SUCCESS;
 
             /* darn, someone raced in on us. */
@@ -620 +626 @@
     }
 #endif /* IN_RING0 || IN_RC */
+
+    return VINF_SUCCESS;
 }