diff options
| author | Gerd Hoffmann <kraxel@redhat.com> | 2023-10-17 13:28:07 +0200 |
|---|---|---|
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2023-10-25 09:52:06 +0000 |
| commit | 170d4ce8e90abb1eff03852940a69c9d17f8afe5 (patch) | |
| tree | 41dae72b491c829fed6e2d49d78422edd3c735c1 /OvmfPkg/AcpiPlatformDxe | |
| parent | d85bf54b7f462eb0297351b5f8dfde09adf617fb (diff) | |
| download | edk2-170d4ce8e90abb1eff03852940a69c9d17f8afe5.tar.gz edk2-170d4ce8e90abb1eff03852940a69c9d17f8afe5.tar.bz2 edk2-170d4ce8e90abb1eff03852940a69c9d17f8afe5.zip | |
UefiCpuPkg/BaseXApicX2ApicLib: fix CPUID_V2_EXTENDED_TOPOLOGY detection
Checking the max cpuid leaf is not enough to figure whenever
CPUID_V2_EXTENDED_TOPOLOGY is supported. Intel SDM says:
Software must detect the presence of CPUID leaf 1FH by verifying
(a) the highest leaf index supported by CPUID is >= 1FH, and
(b) CPUID.1FH:EBX[15:0] reports a non-zero value.
The same is true for CPUID leaf 0BH.
This patch adds the EBX check to GetProcessorLocation2ByApicId(). The
patch also fixes the existing check in GetProcessorLocationByApicId() to
be in line with the spec by looking at bits 15:0. The comments are
updated with a quote from the Intel SDM.
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2241388
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Message-Id: <20231017112807.1244254-1-kraxel@redhat.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Diffstat (limited to 'OvmfPkg/AcpiPlatformDxe')
0 files changed, 0 insertions, 0 deletions