diff options
author | Angel Pons <th3fanbus@gmail.com> | 2021-01-04 17:56:44 +0100 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2021-02-12 19:29:07 +0000 |
commit | 11235d68750ff2274e21571210f95c018a295a36 (patch) | |
tree | f187ce9da1a67078d74aea48cbe9760d1122f265 /tests/commonlib | |
parent | a5541225531a8b4ab12b01587b5cb8ba6c5168e9 (diff) |
cpu/intel/haswell/acpi.c: Correct `get_cores_per_package`
CPUID result does not change when HyperThreading is disabled on
HT-enabled CPUs, which breaks `generate_cpu_entries`. Use MSR 0x35
instead, which returns the currently-enabled core and thread count.
Also rename the function to `get_logical_cores_per_package, which is
more accurate. Based on commit 920d2b77f2 (cpu/intel/206ax/acpi.c: Fix
get_cores_per_package). The MSR definition is the same for Sandy Bridge
and Haswell.
Change-Id: I5e1789d3037780b4285c9e367ff0e2b0d4365b39
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49099
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'tests/commonlib')
0 files changed, 0 insertions, 0 deletions