<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
Reviewed-by: Tim Rowley <<a href="mailto:timothy.o.rowley@intel.com" class="">timothy.o.rowley@intel.com</a>><br class="">
<div><br class="">
<blockquote type="cite" class="">
<div class="">On Dec 12, 2017, at 5:37 PM, Bruce Cherniak <<a href="mailto:bruce.cherniak@intel.com" class="">bruce.cherniak@intel.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div class="">Environment variable KNOB_MAX_WORKER_THREADS allows the user to override<br class="">
default thread creation and thread binding. Previous commit to adjust<br class="">
linux cpu topology caused setting this KNOB to bind all threads to a single<br class="">
core.<br class="">
<br class="">
This patch restores correct functionality of override.<br class="">
<br class="">
Cc: <<a href="mailto:mesa-stable@lists.freedesktop.org" class="">mesa-stable@lists.freedesktop.org</a>><br class="">
---<br class="">
src/gallium/drivers/swr/rasterizer/core/threads.cpp | 3 +--<br class="">
1 file changed, 1 insertion(+), 2 deletions(-)<br class="">
<br class="">
diff --git a/src/gallium/drivers/swr/rasterizer/core/threads.cpp b/src/gallium/drivers/swr/rasterizer/core/threads.cpp<br class="">
index f4ddc21226..6242cb3fc7 100644<br class="">
--- a/src/gallium/drivers/swr/rasterizer/core/threads.cpp<br class="">
+++ b/src/gallium/drivers/swr/rasterizer/core/threads.cpp<br class="">
@@ -213,8 +213,7 @@ void CalculateProcessorTopology(CPUNumaNodes& out_nodes, uint32_t& out_numThread<br class="">
{<br class="">
for (auto &core : node.cores)<br class="">
{<br class="">
- out_numThreadsPerProcGroup = std::max((size_t)out_numThreadsPerProcGroup,<br class="">
- core.threadIds.size());<br class="">
+ out_numThreadsPerProcGroup += core.threadIds.size();<br class="">
}<br class="">
}<br class="">
<br class="">
-- <br class="">
2.11.0<br class="">
<br class="">
_______________________________________________<br class="">
mesa-stable mailing list<br class="">
<a href="mailto:mesa-stable@lists.freedesktop.org" class="">mesa-stable@lists.freedesktop.org</a><br class="">
https://lists.freedesktop.org/mailman/listinfo/mesa-stable<br class="">
</div>
</div>
</blockquote>
</div>
<br class="">
</body>
</html>