getDefaultThreadCount faulty

[ThomasTheGerman]

cpu-core-counts/windows/ThreadCount-Win7.cpp

Line 69 in 7c2329a

DWORD getDefaultThreadCount() {

As mentioned in this reddit thread, the logic in this function doesn't make any sense. For modern AMD Ryzen CPUs, it should return the logical value as well, just like on modern Intel CPUs.

[pagefault]

It's not that simple. It's highly game-dependent. Have a look at this:

https://gpuopen.com/learn/cpu-core-count-detection-windows/

https://gpuopen.com/wp-content/uploads/2018/05/gdc_2018_sponsored_optimizing_for_ryzen.pdf (page 25)

[tannisroot]

Even the article you linked states that

the vast majority of multithreaded games and applications work and scale really well when managing an active thread pool up to the number of logical cores that the processor supports

So it makes no sense for it to default to behavior that is only going to be beneficial for "a small number of games".
Also, evidenced by Cyberpunk, it's clear game developers do not and are not going to profile this and will release a product that is going to run poorly on modern AMD cpus for no reason.

[ThomasTheGerman]

Default behavior should be to utilize the processor fully, SMT can be disabled in cases where performance regressions occur, which are rare.

SMT Regressions in rare edge cases aren't unique to AMD either, so the function as it is right now makes no sense in multiple ways.