summaryrefslogtreecommitdiffstats
path: root/src/common/thread_worker.cpp
diff options
context:
space:
mode:
authorFernandoS27 <fsahmkow27@gmail.com>2021-04-06 04:23:02 +0200
committerReinUsesLisp <reinuseslisp@airmail.cc>2021-07-09 00:03:26 +0200
commita10e112e6436b30c9eb5ca2a82c94f83205bbc34 (patch)
treec7f0a0fdbcd0d9db3e059af4491dac6a74eb6d8f /src/common/thread_worker.cpp
parentcommon/thread_worker: Use unique function (diff)
downloadyuzu-a10e112e6436b30c9eb5ca2a82c94f83205bbc34.tar
yuzu-a10e112e6436b30c9eb5ca2a82c94f83205bbc34.tar.gz
yuzu-a10e112e6436b30c9eb5ca2a82c94f83205bbc34.tar.bz2
yuzu-a10e112e6436b30c9eb5ca2a82c94f83205bbc34.tar.lz
yuzu-a10e112e6436b30c9eb5ca2a82c94f83205bbc34.tar.xz
yuzu-a10e112e6436b30c9eb5ca2a82c94f83205bbc34.tar.zst
yuzu-a10e112e6436b30c9eb5ca2a82c94f83205bbc34.zip
Diffstat (limited to 'src/common/thread_worker.cpp')
-rw-r--r--src/common/thread_worker.cpp14
1 files changed, 13 insertions, 1 deletions
diff --git a/src/common/thread_worker.cpp b/src/common/thread_worker.cpp
index f4d8bb0f0..fd130dfb4 100644
--- a/src/common/thread_worker.cpp
+++ b/src/common/thread_worker.cpp
@@ -8,9 +8,17 @@
namespace Common {
ThreadWorker::ThreadWorker(std::size_t num_workers, const std::string& name) {
+ workers_queued.store(static_cast<u64>(num_workers), std::memory_order_release);
const auto lambda = [this, thread_name{std::string{name}}] {
Common::SetCurrentThreadName(thread_name.c_str());
+ // TODO(Blinkhawk): Change the design, this is very prone to data races
+ // Wait for first request
+ {
+ std::unique_lock lock{queue_mutex};
+ condition.wait(lock, [this] { return stop || !requests.empty(); });
+ }
+
while (!stop) {
UniqueFunction<void> task;
{
@@ -26,7 +34,9 @@ ThreadWorker::ThreadWorker(std::size_t num_workers, const std::string& name) {
requests.pop();
}
task();
+ work_done++;
}
+ workers_stopped++;
wait_condition.notify_all();
};
for (size_t i = 0; i < num_workers; ++i) {
@@ -49,13 +59,15 @@ void ThreadWorker::QueueWork(UniqueFunction<void> work) {
{
std::unique_lock lock{queue_mutex};
requests.emplace(std::move(work));
+ work_scheduled++;
}
condition.notify_one();
}
void ThreadWorker::WaitForRequests() {
std::unique_lock lock{queue_mutex};
- wait_condition.wait(lock, [this] { return stop || requests.empty(); });
+ wait_condition.wait(
+ lock, [this] { return workers_stopped >= workers_queued || work_done >= work_scheduled; });
}
} // namespace Common