From d42bd87f23974164d2539ea85c7b5e6329faf2c2 Mon Sep 17 00:00:00 2001 From: Mathias Agopian Date: Wed, 9 Sep 2009 02:38:13 -0700 Subject: use broadcast() instead of signal() when signaling the condition-variable Thread::RequestExitAndWait() is waiting for we could have several thread waiting on the condition and they all need to wake-up. also added a debug "mTid" field in the class, which contains the tid of the thread (as opposed to pthread_t), this is useful when debugging under gdb for instance. --- libs/utils/Threads.cpp | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'libs/utils/Threads.cpp') diff --git a/libs/utils/Threads.cpp b/libs/utils/Threads.cpp index 6be372c87ea0..ec3db0980d39 100644 --- a/libs/utils/Threads.cpp +++ b/libs/utils/Threads.cpp @@ -655,6 +655,11 @@ int Thread::_threadLoop(void* user) wp weak(strong); self->mHoldSelf.clear(); +#if HAVE_ANDROID_OS + // this is very useful for debugging with gdb + self->mTid = gettid(); +#endif + bool first = true; do { @@ -685,7 +690,7 @@ int Thread::_threadLoop(void* user) self->mExitPending = true; self->mLock.lock(); self->mRunning = false; - self->mThreadExitedCondition.signal(); + self->mThreadExitedCondition.broadcast(); self->mLock.unlock(); break; } @@ -693,7 +698,7 @@ int Thread::_threadLoop(void* user) // Release our strong reference, to let a chance to the thread // to die a peaceful death. strong.clear(); - // And immediately, reacquire a strong reference for the next loop + // And immediately, re-acquire a strong reference for the next loop strong = weak.promote(); } while(strong != 0); -- cgit v1.2.3-59-g8ed1b