Blender Git Loki
Git Commits -> Revision 826d7ad
Revision 826d7ad by Sergey Sharybin (master) December 27, 2018, 18:12 (GMT) |
Fix T59874: Cycles CPU 25% load only during rendering The issue was introduced by a Threadripper2 commit back in ce927e15e0e3. This boils down to threads inheriting affinity from the parent thread. It is a question how this slipped through the review (we definitely run benchmark round). Quick fix could have been to always set CPU group affinity in Cycles, and it would work for Windows. On other platforms we did not have CPU groups API finished. Ended up making Cycles aware of NUMA topology, so now we bound threads to a specific NUMA node. This required adding an external dependency to Cycles, but made some code there shorter. |
Commit Details:
Full Hash: 826d7adde79216d271b78059c05abd10b7559899
Parent Commit: de14ddf
Lines Changed: +114, -248
1 Deleted Path:
/intern/cycles/util/util_windows.cpp (+0, -103)
9 Modified Paths:
/intern/cycles/util/CMakeLists.txt (+9, -5) (Diff)
/intern/cycles/util/util_system.cpp (+49, -40) (Diff)
/intern/cycles/util/util_system.h (+19, -9) (Diff)
/intern/cycles/util/util_task.cpp (+18, -42) (Diff)
/intern/cycles/util/util_thread.cpp (+4, -15) (Diff)
/intern/cycles/util/util_thread.h (+2, -2) (Diff)
/intern/cycles/util/util_windows.h (+1, -24) (Diff)
/intern/numaapi/include/numaapi.h (+1, -1) (Diff)
/intern/numaapi/source/numaapi_linux.c (+11, -7) (Diff)
/intern/cycles/util/util_system.cpp (+49, -40) (Diff)
/intern/cycles/util/util_system.h (+19, -9) (Diff)
/intern/cycles/util/util_task.cpp (+18, -42) (Diff)
/intern/cycles/util/util_thread.cpp (+4, -15) (Diff)
/intern/cycles/util/util_thread.h (+2, -2) (Diff)
/intern/cycles/util/util_windows.h (+1, -24) (Diff)
/intern/numaapi/include/numaapi.h (+1, -1) (Diff)
/intern/numaapi/source/numaapi_linux.c (+11, -7) (Diff)