arch, vmm: Clear AMX CPUID bits if the feature is not enabled

Fixes: #5833

Signed-off-by: Bo Chen <chen.bo@intel.com>
This commit is contained in:
Bo Chen 2023-10-11 15:06:27 -07:00 committed by Bo Chen
parent 7dd260f82f
commit 0b4c153d4d
4 changed files with 16 additions and 0 deletions

View File

@ -37,6 +37,9 @@ const TSC_DEADLINE_TIMER_ECX_BIT: u8 = 24; // tsc deadline timer ecx bit.
const HYPERVISOR_ECX_BIT: u8 = 31; // Hypervisor ecx bit. const HYPERVISOR_ECX_BIT: u8 = 31; // Hypervisor ecx bit.
const MTRR_EDX_BIT: u8 = 12; // Hypervisor ecx bit. const MTRR_EDX_BIT: u8 = 12; // Hypervisor ecx bit.
const INVARIANT_TSC_EDX_BIT: u8 = 8; // Invariant TSC bit on 0x8000_0007 EDX const INVARIANT_TSC_EDX_BIT: u8 = 8; // Invariant TSC bit on 0x8000_0007 EDX
const AMX_BF16: u8 = 22; // AMX tile computation on bfloat16 numbers
const AMX_TILE: u8 = 24; // AMX tile load/store instructions
const AMX_INT8: u8 = 25; // AMX tile computation on 8-bit integers
// KVM feature bits // KVM feature bits
#[cfg(feature = "tdx")] #[cfg(feature = "tdx")]
@ -151,6 +154,7 @@ pub struct CpuidConfig {
pub kvm_hyperv: bool, pub kvm_hyperv: bool,
#[cfg(feature = "tdx")] #[cfg(feature = "tdx")]
pub tdx: bool, pub tdx: bool,
pub amx: bool,
} }
#[derive(Debug)] #[derive(Debug)]
@ -640,6 +644,12 @@ pub fn generate_common_cpuid(
// Update some existing CPUID // Update some existing CPUID
for entry in cpuid.as_mut_slice().iter_mut() { for entry in cpuid.as_mut_slice().iter_mut() {
match entry.function { match entry.function {
// Clear AMX related bits if the AMX feature is not enabled
0x7 => {
if !config.amx && entry.index == 0 {
entry.edx &= !(1 << AMX_BF16 | 1 << AMX_TILE | 1 << AMX_INT8)
}
}
0xd => 0xd =>
{ {
#[cfg(feature = "tdx")] #[cfg(feature = "tdx")]

View File

@ -745,6 +745,7 @@ impl CpuManager {
kvm_hyperv: self.config.kvm_hyperv, kvm_hyperv: self.config.kvm_hyperv,
#[cfg(feature = "tdx")] #[cfg(feature = "tdx")]
tdx, tdx,
amx: self.config.features.amx,
}, },
) )
.map_err(Error::CommonCpuId)? .map_err(Error::CommonCpuId)?

View File

@ -1665,6 +1665,7 @@ impl Vmm {
let common_cpuid = { let common_cpuid = {
#[cfg(feature = "tdx")] #[cfg(feature = "tdx")]
let tdx = vm_config.lock().unwrap().is_tdx_enabled(); let tdx = vm_config.lock().unwrap().is_tdx_enabled();
let amx = vm_config.lock().unwrap().cpus.features.amx;
let phys_bits = let phys_bits =
vm::physical_bits(&hypervisor, vm_config.lock().unwrap().cpus.max_phys_bits); vm::physical_bits(&hypervisor, vm_config.lock().unwrap().cpus.max_phys_bits);
arch::generate_common_cpuid( arch::generate_common_cpuid(
@ -1675,6 +1676,7 @@ impl Vmm {
kvm_hyperv: vm_config.lock().unwrap().cpus.kvm_hyperv, kvm_hyperv: vm_config.lock().unwrap().cpus.kvm_hyperv,
#[cfg(feature = "tdx")] #[cfg(feature = "tdx")]
tdx, tdx,
amx,
}, },
) )
.map_err(|e| { .map_err(|e| {
@ -1866,6 +1868,7 @@ impl Vmm {
kvm_hyperv: vm_config.cpus.kvm_hyperv, kvm_hyperv: vm_config.cpus.kvm_hyperv,
#[cfg(feature = "tdx")] #[cfg(feature = "tdx")]
tdx: vm_config.is_tdx_enabled(), tdx: vm_config.is_tdx_enabled(),
amx: vm_config.cpus.features.amx,
}, },
) )
.map_err(|e| { .map_err(|e| {

View File

@ -2409,6 +2409,7 @@ impl Snapshottable for Vm {
#[cfg(all(feature = "kvm", target_arch = "x86_64"))] #[cfg(all(feature = "kvm", target_arch = "x86_64"))]
let common_cpuid = { let common_cpuid = {
let amx = self.config.lock().unwrap().cpus.features.amx;
let phys_bits = physical_bits( let phys_bits = physical_bits(
&self.hypervisor, &self.hypervisor,
self.config.lock().unwrap().cpus.max_phys_bits, self.config.lock().unwrap().cpus.max_phys_bits,
@ -2421,6 +2422,7 @@ impl Snapshottable for Vm {
kvm_hyperv: self.config.lock().unwrap().cpus.kvm_hyperv, kvm_hyperv: self.config.lock().unwrap().cpus.kvm_hyperv,
#[cfg(feature = "tdx")] #[cfg(feature = "tdx")]
tdx: tdx_enabled, tdx: tdx_enabled,
amx,
}, },
) )
.map_err(|e| { .map_err(|e| {