Changeset 8669 in vbox for trunk

Timestamp:

May 7, 2008 3:51:13 PM (17 years ago)

Author:

vboxsync

Message:

Use the IN_TREE flag instead of the state just to be on the safe side.

File:

• trunk/src/VBox/Runtime/common/misc/thread.cpp (modified) (1 diff)

trunk/src/VBox/Runtime/common/misc/thread.cpp

@@ -527 +527 @@
      * Remove it from the tree and mark it as dead.
      *
-     * Thread that has seen rtThreadTerminate and entered the final TERMINATED
-     * state does not need removing, probably no thread should require removing
-     * here. However, be careful making sure that cRefs isn't 0 if we do or we'll
-     * blow up because the strict locking code will be calling us back.
-     */
-    if (pThread->enmState != RTTHREADSTATE_TERMINATED)
+     * Threads that has seen rtThreadTerminate and should already have been
+     * removed from the tree. There is probably no thread that  should
+     * require removing here. However, be careful making sure that cRefs
+     * isn't 0 if we do or we'll blow up because the strict locking code
+     * will be calling us back.
+     */
+    if (ASMBitTest(&pThread->fIntFlags, RTTHREADINT_FLAG_IN_TREE_BIT))
     {
         ASMAtomicIncU32(&pThread->cRefs);