diff mbox series

KVM: VMX: Also clear SGX EDECCSSA in KVM CPU caps when SGX is disabled

Message ID 20240905120837.579102-1-kai.huang@intel.com (mailing list archive)
State New, archived
Headers show
Series KVM: VMX: Also clear SGX EDECCSSA in KVM CPU caps when SGX is disabled | expand

Commit Message

Huang, Kai Sept. 5, 2024, 12:08 p.m. UTC
When SGX EDECCSSA support was added to KVM in commit 16a7fe3728a8
("KVM/VMX: Allow exposing EDECCSSA user leaf function to KVM guest"), it
forgot to clear the X86_FEATURE_SGX_EDECCSSA bit in KVM CPU caps when
KVM SGX is disabled.  Fix it.

Fixes: 16a7fe3728a8 ("KVM/VMX: Allow exposing EDECCSSA user leaf function to KVM guest")
Signed-off-by: Kai Huang <kai.huang@intel.com>
---
 arch/x86/kvm/vmx/vmx.c | 1 +
 1 file changed, 1 insertion(+)


base-commit: 44518120c4ca569cfb9c6199e94c312458dc1c07

Comments

Sean Christopherson Sept. 10, 2024, 4:56 a.m. UTC | #1
On Fri, 06 Sep 2024 00:08:37 +1200, Kai Huang wrote:
> When SGX EDECCSSA support was added to KVM in commit 16a7fe3728a8
> ("KVM/VMX: Allow exposing EDECCSSA user leaf function to KVM guest"), it
> forgot to clear the X86_FEATURE_SGX_EDECCSSA bit in KVM CPU caps when
> KVM SGX is disabled.  Fix it.

Applied to kvm-x86 vmx.  At some point, we should probably look at using the
kernel's cpufeatures dependency chains, but that's no reason to hold this fix.

Thanks!

[1/1] KVM: VMX: Also clear SGX EDECCSSA in KVM CPU caps when SGX is disabled
      https://github.com/kvm-x86/linux/commit/7efb4d8a392a

--
https://github.com/kvm-x86/linux/tree/next
diff mbox series

Patch

diff --git a/arch/x86/kvm/vmx/vmx.c b/arch/x86/kvm/vmx/vmx.c
index 594db9afbc0f..98d737e0d416 100644
--- a/arch/x86/kvm/vmx/vmx.c
+++ b/arch/x86/kvm/vmx/vmx.c
@@ -7963,6 +7963,7 @@  static __init void vmx_set_cpu_caps(void)
 		kvm_cpu_cap_clear(X86_FEATURE_SGX_LC);
 		kvm_cpu_cap_clear(X86_FEATURE_SGX1);
 		kvm_cpu_cap_clear(X86_FEATURE_SGX2);
+		kvm_cpu_cap_clear(X86_FEATURE_SGX_EDECCSSA);
 	}
 
 	if (vmx_umip_emulated())