aboutsummaryrefslogtreecommitdiff
path: root/src/libutil/thread-pool.cc
diff options
context:
space:
mode:
authorEelco Dolstra <edolstra@gmail.com>2017-09-08 15:31:24 +0200
committerEelco Dolstra <edolstra@gmail.com>2017-09-08 15:31:24 +0200
commitb7376edf06e3836394503e16ec14af23c7615f22 (patch)
tree1ba2185c659f832628580e98073792e02c13be00 /src/libutil/thread-pool.cc
parent6a888ec29a6724f916f96508b3a94a86c643c18e (diff)
ThreadPool: On exception, interrupt the other worker threads
Diffstat (limited to 'src/libutil/thread-pool.cc')
-rw-r--r--src/libutil/thread-pool.cc10
1 files changed, 6 insertions, 4 deletions
diff --git a/src/libutil/thread-pool.cc b/src/libutil/thread-pool.cc
index f43dbe0c3..ce126d36d 100644
--- a/src/libutil/thread-pool.cc
+++ b/src/libutil/thread-pool.cc
@@ -21,7 +21,7 @@ ThreadPool::~ThreadPool()
std::vector<std::thread> workers;
{
auto state(state_.lock());
- state->quit = true;
+ quit = true;
std::swap(workers, state->workers);
}
@@ -36,7 +36,7 @@ ThreadPool::~ThreadPool()
void ThreadPool::enqueue(const work_t & t)
{
auto state(state_.lock());
- if (state->quit)
+ if (quit)
throw ThreadPoolShutDown("cannot enqueue a work item while the thread pool is shutting down");
state->left.push(t);
if (state->left.size() > state->workers.size() && state->workers.size() < maxThreads)
@@ -63,6 +63,8 @@ void ThreadPool::process()
void ThreadPool::workerEntry()
{
+ interruptCheck = [&]() { return (bool) quit; };
+
bool didWork = false;
std::exception_ptr exc;
@@ -80,7 +82,7 @@ void ThreadPool::workerEntry()
if (!state->exception) {
state->exception = exc;
// Tell the other workers to quit.
- state->quit = true;
+ quit = true;
work.notify_all();
} else {
/* Print the exception, since we can't
@@ -100,7 +102,7 @@ void ThreadPool::workerEntry()
/* Wait until a work item is available or another thread
had an exception or we're asked to quit. */
while (true) {
- if (state->quit) {
+ if (quit) {
if (!state->active)
done.notify_one();
return;