]> git.baikalelectronics.ru Git - kernel.git/commit
kvm: x86: replace kvm_spec_ctrl_test_value with runtime test on the host
authorMaxim Levitsky <mlevitsk@redhat.com>
Wed, 8 Jul 2020 11:57:31 +0000 (14:57 +0300)
committerPaolo Bonzini <pbonzini@redhat.com>
Thu, 9 Jul 2020 11:08:38 +0000 (07:08 -0400)
commit78e5777fb05cd5d7abbb8d3e8c31d179b1e02104
treed67740d8f6d03dce2e4ac62920c47be8da1699b5
parent3252581bc2f45201c74e38e049720727aa8ee9d7
kvm: x86: replace kvm_spec_ctrl_test_value with runtime test on the host

To avoid complex and in some cases incorrect logic in
kvm_spec_ctrl_test_value, just try the guest's given value on the host
processor instead, and if it doesn't #GP, allow the guest to set it.

One such case is when host CPU supports STIBP mitigation
but doesn't support IBRS (as is the case with some Zen2 AMD cpus),
and in this case we were giving guest #GP when it tried to use STIBP

The reason why can can do the host test is that IA32_SPEC_CTRL msr is
passed to the guest, after the guest sets it to a non zero value
for the first time (due to performance reasons),
and as as result of this, it is pointless to emulate #GP condition on
this first access, in a different way than what the host CPU does.

This is based on a patch from Sean Christopherson, who suggested this idea.

Fixes: 5a8260c906d8 ("KVM: x86: avoid incorrect writes to host MSR_IA32_SPEC_CTRL")
Cc: stable@vger.kernel.org
Suggested-by: Sean Christopherson <sean.j.christopherson@intel.com>
Signed-off-by: Maxim Levitsky <mlevitsk@redhat.com>
Message-Id: <20200708115731.180097-1-mlevitsk@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/svm/svm.c
arch/x86/kvm/vmx/vmx.c
arch/x86/kvm/x86.c
arch/x86/kvm/x86.h