[coreboot-gerrit] Change in coreboot[master]: cpu/intel/hyperthreading: Use CPUID leaf 0xb without X2APIC

[Kyösti Mälkki (Code Review)]

Attention is currently required from: Patrick Rudolph.
Kyösti Mälkki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/58383 )

Change subject: cpu/intel/hyperthreading: Use CPUID leaf 0xb without X2APIC
......................................................................

cpu/intel/hyperthreading: Use CPUID leaf 0xb without X2APIC

It is not a requirement to have X2APIC mode enabled to use
CPUID leaf 0xb EDX to detect logical CPU is a hyperthreading
sibling.

Change-Id: I288f2df5a392c396f92bb6d18908df35de55915d
Signed-off-by: Kyösti Mälkki 
---
M src/cpu/intel/common/hyperthreading.c
1 file changed, 4 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/83/58383/1

diff --git a/src/cpu/intel/common/hyperthreading.c b/src/cpu/intel/common/hyperthreading.c
index b9c17b6..2936770 100644
--- a/src/cpu/intel/common/hyperthreading.c
+++ b/src/cpu/intel/common/hyperthreading.c
@@ -23,12 +23,10 @@
    if (!intel_ht_supported())
    	return false;

-	if (is_x2apic_mode()) {
-		if (cpuid_eax(0) >= 0xb) {
-			result = cpuid_ext(0xb, 0);
-			const uint32_t div = 1 << (result.eax & 0x1f);
-			return result.edx % div > 0;
-		}
+	if (cpuid_eax(0) >= 0xb) {
+		result = cpuid_ext(0xb, 0);
+		const uint32_t div = 1 << (result.eax & 0x1f);
+		return result.edx % div > 0;
    }

apic_ids = 1;