search

WyvernTKC / cpuminer-gr-avx2

Optimised Version of GR miner for RTM
GNU General Public License v2.0
371 stars 193 forks source link

Affinity incorrectly set on > 63 thread CPUs #41

Closed Chipcraft closed 3 years ago

Chipcraft commented 3 years ago

The issue is back in 1.2.1, 1.2.0 was fine in this regard.

Originally posted by @Chipcraft in https://github.com/WyvernTKC/cpuminer-gr-avx2/issues/18#issuecomment-940444342

Giving no "--threads" or "cpu-affinity" parameters results in only the first 32 logical cores being used (i.e. 16C/32T). Adding "--threads 63 --cpu-affinity 0x7FFFFFFFFFFFFFFF" as a parameter works, leaving only one of the logical (SMT thread) unutilized.

Windows 10 Pro.

michal-zurkowski commented 3 years ago

Thanks for reporting. Do you mind contacting me on Discord Delgon (Naen)#1859 ?

Chipcraft commented 3 years ago

It is still broken in the most recent commit. Currently you must (re)set the process affinity to make it work (i.e. open affinity and close it without making any changes).

You are unreachable in Discord, since you are not accepting friend requests.

michal-zurkowski commented 3 years ago

I do not want friend request. Just DM me or reach out to me on RTM (and many other) discords.

It was tested on 32C/64T Threadripper 3975WX and worked like a charm with default affinity flag and without it in config.json

Chipcraft commented 3 years ago

In any case, on a similar TR system and the 1.2.1 release binaries the utilization looks like this:

Without either manually "toggling" the affinity or without including the patch described in issue #42, that is.

121Affi