QThreadPool: fix race at time of thread expiry.
authorDavid Faure <david.faure@kdab.com>
Mon, 24 Jun 2013 17:20:24 +0000 (19:20 +0200)
committerThe Qt Project <gerrit-noreply@qt-project.org>
Thu, 31 Oct 2013 16:45:58 +0000 (17:45 +0100)
commita9b6a78e54670a70b96c122b10ad7bd64d166514
tree35cb2b3068ddbdabcc69221ccc4ca6498b6cb559
parent94fd108ea42a99dacefa819bc3fd4363fb95e886
QThreadPool: fix race at time of thread expiry.

The current synchronization mechanism was racy: decrementing waitingThreads
and then hoping that the wakeOne will wake a thread before its expiry
timeout happens. In other words, on timeout, a just-assigned task would
never run. And then no other task would run, if maxThreadCount is reached.

Fixed by using a queue of waiting threads (rather than just a count), and by
moving the wait condition into the thread itself, so we know precisely
which one we're waking up, and we can remove it from the set of waiting threads
before waking it up, and therefore it can determine on wakeup whether it
has work to do (caller removed it from the queue) or it expired (it's still
in the queue). This is reliable, whereas the return value from QWaitCondition::wait
isn't reliable, when the main thread has already decided that this thread
has work to do.

Task-number: QTBUG-3786
Change-Id: I1eac5d6c309daed7f483ac7a8074297bfda6ee32
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
src/corelib/thread/qthreadpool.cpp
src/corelib/thread/qthreadpool_p.h
tests/auto/corelib/thread/qthreadpool/tst_qthreadpool.cpp