From 709f5bdc4a2bf31f422f5cf60797224c0463c10a Mon Sep 17 00:00:00 2001 From: Bruce Cherniak Date: Tue, 12 Dec 2017 17:18:23 -0600 Subject: [PATCH] swr: Fix KNOB_MAX_WORKER_THREADS thread creation override. Environment variable KNOB_MAX_WORKER_THREADS allows the user to override default thread creation and thread binding. Previous commit to adjust linux cpu topology caused setting this KNOB to bind all threads to a single core. This patch restores correct functionality of override. Cc: Reviewed-by: Tim Rowley --- src/gallium/drivers/swr/rasterizer/core/threads.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/threads.cpp b/src/gallium/drivers/swr/rasterizer/core/threads.cpp index f4ddc212264..6242cb3fc7c 100644 --- a/src/gallium/drivers/swr/rasterizer/core/threads.cpp +++ b/src/gallium/drivers/swr/rasterizer/core/threads.cpp @@ -213,8 +213,7 @@ void CalculateProcessorTopology(CPUNumaNodes& out_nodes, uint32_t& out_numThread { for (auto &core : node.cores) { - out_numThreadsPerProcGroup = std::max((size_t)out_numThreadsPerProcGroup, - core.threadIds.size()); + out_numThreadsPerProcGroup += core.threadIds.size(); } } -- 2.30.2