[FIX] Detect like cores by looking at scaling_max_freq instead of cpuinfo_max_freq #8370
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
On my AMD 3950X,
/sys/devices/systen/cpu/cpu*/cpufreq/cpuinfo_max_freq
is not the same across all cores. If I look atscaling_max_freq
instead, then I see that all cores are the same frequency (as expected). This probably still won't work correctly across all platforms, but I don't think there is an easier, quick way to do the detection. Maybe we should think about using pytorch's cpuinfo library (https://github.com/pytorch/cpuinfo).@tqchen @FrozenGene