From 86cf50565e955357b779ce9968b6295591ed02e4 Mon Sep 17 00:00:00 2001 From: Wei Liu Date: Tue, 7 May 2024 00:06:32 +0000 Subject: [PATCH] hypervisor: kvm: drop the check for Cap::SignalMsi Per the KVM API document, that capability is only valid with in-kernel irqchip that handles MSIs. Through out the code base, there is no call to KVM_IOCTL_SIGNAL_MSI. Signed-off-by: Wei Liu --- hypervisor/src/kvm/aarch64/mod.rs | 3 --- hypervisor/src/kvm/x86_64/mod.rs | 3 --- 2 files changed, 6 deletions(-) diff --git a/hypervisor/src/kvm/aarch64/mod.rs b/hypervisor/src/kvm/aarch64/mod.rs index 77c76da20..fb5021a62 100644 --- a/hypervisor/src/kvm/aarch64/mod.rs +++ b/hypervisor/src/kvm/aarch64/mod.rs @@ -106,9 +106,6 @@ pub fn is_system_register(regid: u64) -> bool { } pub fn check_required_kvm_extensions(kvm: &Kvm) -> KvmResult<()> { - if !kvm.check_extension(Cap::SignalMsi) { - return Err(KvmError::CapabilityMissing(Cap::SignalMsi)); - } if !kvm.check_extension(Cap::OneReg) { return Err(KvmError::CapabilityMissing(Cap::OneReg)); } diff --git a/hypervisor/src/kvm/x86_64/mod.rs b/hypervisor/src/kvm/x86_64/mod.rs index cb4d069e3..89c9a1a6f 100644 --- a/hypervisor/src/kvm/x86_64/mod.rs +++ b/hypervisor/src/kvm/x86_64/mod.rs @@ -32,9 +32,6 @@ pub use { /// Check KVM extension for Linux /// pub fn check_required_kvm_extensions(kvm: &Kvm) -> KvmResult<()> { - if !kvm.check_extension(Cap::SignalMsi) { - return Err(KvmError::CapabilityMissing(Cap::SignalMsi)); - } if !kvm.check_extension(Cap::TscDeadlineTimer) { return Err(KvmError::CapabilityMissing(Cap::TscDeadlineTimer)); }