blender
/
blender
128
396
Fork 569
Code Issues 6.2k Pull Requests 757 Projects 19 Wiki Activity

Fix T60585: threadripper CPU only using 16 threads for e.g. sculpting. 

This reverts the changes from ce927e1 to put the main and job threads on
node 0. The problem is that all threads created as children from these
threads will inherit the NUMA node and so will end up on the same node.
This can be fixed case-by-case by assigning the NUMA node for every child
thread, however this is difficult for external libraries and OpenMP, and
out of our control for plugins like external renderers.
This commit is contained in:
Brecht Van Lommel 2019-03-05 12:43:40 +01:00
parent 8d8d113b73
commit 76608f5ec5
Notes: blender-bot 2023-06-26 11:58:59 +02:00 
Referenced by issue #60585, 2.8 Builds are limited to 16 threads of processing
1 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
source/blender/blenlib/intern/threads.c
View File

@ -888,7 +888,7 @@ static void threadripper_put_process_on_fast_node(void)
if (!is_numa_available) {
return;
}
/* NOTE: Technically, we can use NUMA nodes 0 and 2 and usning both of
/* NOTE: Technically, we can use NUMA nodes 0 and 2 and using both of
* them in the affinity mask will allow OS to schedule threads more
* flexible,possibly increasing overall performance when multiple apps
* are crunching numbers.
@ -923,14 +923,26 @@ static void threadripper_put_thread_on_fast_node(void)
void BLI_thread_put_process_on_fast_node(void)
{
/* Disabled for now since this causes only 16 threads to be used on a
* threadripper for computations like sculpting and fluid sim. The problem
* is that all threads created as children from this thread will inherit
* the NUMA node and so will end up on the same node. This can be fixed
* case-by-case by assigning the NUMA node for every child thread, however
* this is difficult for external libraries and OpenMP, and out of our
* control for plugins like external renderers. */
#if 0
if (check_is_threadripper2_alike_topology()) {
threadripper_put_process_on_fast_node();
}
#endif
}
void BLI_thread_put_thread_on_fast_node(void)
{
/* Disabled for now, see comment above. */
#if 0
if (check_is_threadripper2_alike_topology()) {
threadripper_put_thread_on_fast_node();
}
#endif
}