diff options
author | FernandoS27 <fsahmkow27@gmail.com> | 2021-04-06 04:23:02 +0200 |
---|---|---|
committer | ReinUsesLisp <reinuseslisp@airmail.cc> | 2021-07-09 00:03:26 +0200 |
commit | a10e112e6436b30c9eb5ca2a82c94f83205bbc34 (patch) | |
tree | c7f0a0fdbcd0d9db3e059af4491dac6a74eb6d8f | |
parent | common/thread_worker: Use unique function (diff) | |
download | yuzu-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 |
-rw-r--r-- | src/common/thread_worker.cpp | 14 | ||||
-rw-r--r-- | src/common/thread_worker.h | 5 |
2 files changed, 18 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 diff --git a/src/common/thread_worker.h b/src/common/thread_worker.h index 7e2b04a07..12bbf5fef 100644 --- a/src/common/thread_worker.h +++ b/src/common/thread_worker.h @@ -11,6 +11,7 @@ #include <vector> #include <queue> +#include "common/common_types.h" #include "common/unique_function.h" namespace Common { @@ -29,6 +30,10 @@ private: std::condition_variable condition; std::condition_variable wait_condition; std::atomic_bool stop{}; + std::atomic<u64> work_scheduled{}; + std::atomic<u64> work_done{}; + std::atomic<u64> workers_stopped{}; + std::atomic<u64> workers_queued{}; }; } // namespace Common |