Changeset 21555 in vbox for trunk/src/VBox/Runtime/r0drv

Timestamp:

Jul 13, 2009 4:51:25 PM (16 years ago)

Author:

vboxsync

Message:

semspinmutex-r0drv-generic.c: better fix.

File:

• trunk/src/VBox/Runtime/r0drv/generic/semspinmutex-r0drv-generic.c (modified) (4 diffs)

trunk/src/VBox/Runtime/r0drv/generic/semspinmutex-r0drv-generic.c

@@ -79 +79 @@
      * This is NIL if the semaphore is not owned by anyone. */
     RTNATIVETHREAD volatile hOwner;
-    /** Number of threads waiting for the lock. */
+    /** Number of threads that are fighting for the lock. */
     int32_t volatile        cLockers;
     /** The semaphore to block on. */
@@ -126 +126 @@
     pThis->fFlags    = fFlags;
     pThis->hOwner    = NIL_RTNATIVETHREAD;
-    pThis->cLockers  = -1;
+    pThis->cLockers  = 0;
     rc = RTSemEventCreate(&pThis->hEventSem);
     if (RT_SUCCESS(rc))
@@ -401 +401 @@
     cLockers = ASMAtomicDecS32(&pThis->cLockers);
     rtSemSpinMutexLeave(&State);
-    if (cLockers >= 0)
+    if (cLockers > 0)
     {
         int rc = RTSemEventSignal(pThis->hEventSem);
@@ -423 +423 @@
 
     /* No destruction races allowed! */
-    AssertMsg(   pThis->cLockers  == -1
+    AssertMsg(   pThis->cLockers  == 0
               && pThis->hOwner    == NIL_RTNATIVETHREAD,
               ("pThis=%p cLockers=%d hOwner=%p\n", pThis, pThis->cLockers, pThis->hOwner));