Hello build bot (Jenkins), Patrick Rudolph, Angel Pons, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/42413
to look at the new patch set (#6).
Change subject: cpu/intel/206ax/acpi.c: Fix get_cores_per_package ......................................................................
cpu/intel/206ax/acpi.c: Fix get_cores_per_package
Current implementation uses CPUID 0Bh function that returns the number of logical cores of requested level. The problem with this approach is that this value doesn't change when HyperThreading is disabled (it's in the Intel docs), so it breaks generate_cpu_entries().
- Use MSR 0x35 instead, which returns the correct number of logical processors with and without HT.
- Rename the function to get_logical_cores_per_package, which is more accurate.
Tested on ThinkPad X220 with and without HT.
Related to CB:29669.
Change-Id: Ib32c2d40408cfa42ca43ab42ed661c168e579ada Signed-off-by: Evgeny Zinoviev me@ch1p.io --- M src/cpu/intel/model_206ax/acpi.c 1 file changed, 4 insertions(+), 14 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/13/42413/6