Fix extended topology enumeration leaf by tpressure · Pull Request #7087 · cloud-hypervisor/cloud-hypervisor · GitHub
Skip to content

Fix extended topology enumeration leaf#7087

Merged
russell-islam merged 1 commit intocloud-hypervisor:mainfrom
tpressure:cpuid_fix
May 22, 2025
Merged

Fix extended topology enumeration leaf#7087
russell-islam merged 1 commit intocloud-hypervisor:mainfrom
tpressure:cpuid_fix

Conversation

@tpressure
Copy link
Copy Markdown
Member

When booting a Linux guest in SMP configuration, the following kernel warning can be observed:

[Firmware Bug]: CPUID leaf 0xb subleaf 1 APIC ID mismatch 1 != 0

The reason is that we announce the presence of the extended topology leaf, but fail to announce the x2apic ID in EDX.

I found this issue because I am currently working on x86_64 support for >255 vCPUs where this fix is absolutely needed in order to convince Linux to boot the additional processors above 255.

@tpressure tpressure requested a review from a team as a code owner May 21, 2025 15:15
When booting a Linux guest in SMP configuration,
the following kernel warning can be observed:

[Firmware Bug]: CPUID leaf 0xb subleaf 1 APIC ID mismatch 1 != 0

The reason is that we announce the presence of the extended topology
leaf, but fail to announce the x2apic ID in EDX.

Signed-off-by: Thomas Prescher <thomas.prescher@cyberus-technology.de>
On-behalf-of: SAP thomas.prescher@sap.com
@blitz
Copy link
Copy Markdown
Contributor

blitz commented May 21, 2025

@tpressure
Copy link
Copy Markdown
Member Author

@russell-islam russell-islam added this pull request to the merge queue May 21, 2025
Merged via the queue into cloud-hypervisor:main with commit d172a5d May 22, 2025
39 of 40 checks passed
@likebreath likebreath added the bug-fix Bug fix to include in release notes label May 22, 2025
@likebreath likebreath moved this from 🆕 New to ✅ Done in Cloud Hypervisor Roadmap May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Bug fix to include in release notes

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

6 participants