[v2,0/2] KVM: x86: don't announce KVM_CAP_HYPERV_ENLIGHTENED_VMCS when it is unavailable

Message

Vitaly Kuznetsov Aug. 28, 2019, 7:59 a.m. UTC

It was discovered that hyperv_cpuid test now fails on AMD as it tries to
enable KVM_CAP_HYPERV_ENLIGHTENED_VMCS which is (wrongfully) reported as
available.

Changes since v1:
- This is a v2 for '[PATCH 0/3] KVM: x86: fix a couple of issues with
 Enlightened VMCS enablement' renamed as the first patch of the series
 was already merged.
- Added Jim's Reviewed-by: to PATCH1
- Added missing break in PATCH2 [Jim Mattson, Sean Christopherson] 

Vitaly Kuznetsov (2):
  KVM: x86: svm: remove unneeded nested_enable_evmcs() hook
  KVM: x86: announce KVM_CAP_HYPERV_ENLIGHTENED_VMCS support only when
    it is available

 arch/x86/kvm/svm.c | 9 +--------
 arch/x86/kvm/x86.c | 4 +++-
 2 files changed, 4 insertions(+), 9 deletions(-)

Comments

Paolo Bonzini Sept. 17, 2019, 1:29 p.m. UTC | #1

On 28/08/19 09:59, Vitaly Kuznetsov wrote:
> It was discovered that hyperv_cpuid test now fails on AMD as it tries to
> enable KVM_CAP_HYPERV_ENLIGHTENED_VMCS which is (wrongfully) reported as
> available.
> 
> Changes since v1:
> - This is a v2 for '[PATCH 0/3] KVM: x86: fix a couple of issues with
>  Enlightened VMCS enablement' renamed as the first patch of the series
>  was already merged.
> - Added Jim's Reviewed-by: to PATCH1
> - Added missing break in PATCH2 [Jim Mattson, Sean Christopherson] 
> 
> Vitaly Kuznetsov (2):
>   KVM: x86: svm: remove unneeded nested_enable_evmcs() hook
>   KVM: x86: announce KVM_CAP_HYPERV_ENLIGHTENED_VMCS support only when
>     it is available
> 
>  arch/x86/kvm/svm.c | 9 +--------
>  arch/x86/kvm/x86.c | 4 +++-
>  2 files changed, 4 insertions(+), 9 deletions(-)
> 

Queued, thanks.

Paolo