Thanks to visit codestin.com
Credit goes to github.com

Skip to content

arch: Handle amx related features in subleaf 0x1 of leaf 0x7 - #7496

Closed
olivereanderson wants to merge 1 commit into
cloud-hypervisor:mainfrom
olivereanderson:leaf-0x7-subleaf-1-amx-masking
Closed

olivereanderson wants to merge 1 commit into
cloud-hypervisor:mainfrom
olivereanderson:leaf-0x7-subleaf-1-amx-masking

Conversation

@olivereanderson

Copy link
Copy Markdown
Contributor

The Intel Granite Rapids processors include more AMX related features that are advertised in leaf 0x7 subleaf 0x1.

If the VM is not configured to support AMX (the default) then these feature bits need to be masked out. This is addressed by this PR.

@olivereanderson
olivereanderson requested a review from a team as a code owner November 21, 2025 14:35
The Intel Granite Rapids processors include more AMX related features
that are advertised in leaf 0x7 subleaf 0x1. If the VM is not
configured to support AMX (the default) then these feature bits need
to be masked out. This is addressed by this PR.

Signed-off-by: Oliver Anderson <[email protected]>
On-behalf-of: SAP <[email protected]>
@olivereanderson
olivereanderson force-pushed the leaf-0x7-subleaf-1-amx-masking branch from ffee01e to 9681f22 Compare November 21, 2025 14:39

@tpressure tpressure left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@olivereanderson I think this is still not enough. Looking at the most recent spec, we probably also need to wipe leaf 0x1d (tile information leaf) and leaf 0x1e (TMUL information main leaf) including all their subleafs :(

It is really unfortunate that Intel spread this information across so many leafs.

@phip1611 phip1611 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will approve when our CPUID leaf expert @tpressure gives a 👍🏻

@olivereanderson

Copy link
Copy Markdown
Contributor Author

Closed in favour of #7508

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants