diff options
author | Mike Klein <mtklein@chromium.org> | 2017-09-01 11:53:16 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-09-01 16:19:47 +0000 |
commit | 022cfa258df280d914cfff4dc1466a3fa909ce9c (patch) | |
tree | 063fc22439e0eb99210a51d8521b73cdcad2c6a4 /tools/flags/SkCommonFlags.cpp | |
parent | d41dc171496e20e6918909b4a34776eada4b7862 (diff) |
Add FIFO thread pool, use it for most things.
We like a LIFO default thread pool in tools like DM for better
memory/time locality... the bots use less memory this way, and
generally run faster.
But most use cases want a FIFO queue, so that they can get going
on the next parts of early work while later work is still running.
This splits the implementation into one using SkTArray and pop_back
for LIFO, and a new one using std::deque and pop_front for FIFO.
Change-Id: Ief203b6869a00f1f8084019431a781d15fc63750
Reviewed-on: https://skia-review.googlesource.com/41849
Commit-Queue: Mike Klein <mtklein@chromium.org>
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Yuqian Li <liyuqian@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>
Diffstat (limited to 'tools/flags/SkCommonFlags.cpp')
-rw-r--r-- | tools/flags/SkCommonFlags.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/flags/SkCommonFlags.cpp b/tools/flags/SkCommonFlags.cpp index 0986d2b57c..7884bef30e 100644 --- a/tools/flags/SkCommonFlags.cpp +++ b/tools/flags/SkCommonFlags.cpp @@ -131,6 +131,6 @@ bool CollectImages(SkCommandLineFlags::StringArray images, SkTArray<SkString>* o SkExecutor* GpuExecutorForTools() { static std::unique_ptr<SkExecutor> gGpuExecutor = (0 != FLAGS_gpuThreads) - ? SkExecutor::MakeThreadPool(FLAGS_gpuThreads) : nullptr; + ? SkExecutor::MakeFIFOThreadPool(FLAGS_gpuThreads) : nullptr; return gGpuExecutor.get(); } |