KVM: x86: Fix MSR_IA32_PERF_CAPABILITIES writes check and vmx_pmu_msrs_test

[PATCH v2 0/2] KVM: x86: Fix MSR_IA32_PERF_CAPABILITIES writes check and vmx_pmu_msrs_test

Posted by Vitaly Kuznetsov 4 years, 6 months ago

This is a continuation of "KVM: selftests: Avoid KVM_SET_CPUID2 after 
KVM_RUN in vmx_pmu_msrs_test" work. Instead of fixing the immediate issue,
drop incorrect check in KVM which was making the result of host initiated
writes to MSR_IA32_PERF_CAPABILITIES dependent on guest visible CPUIDs and
the corresponding tests in vmx_pmu_msrs_test, this will also make the issue
reported by kernel test robot to go away.

Vitaly Kuznetsov (2):
  KVM: selftests: vmx_pmu_msrs_test: Drop tests mangling guest visible
    CPUIDs
  KVM: x86: Drop guest CPUID check for host initiated writes to
    MSR_IA32_PERF_CAPABILITIES

 arch/x86/kvm/x86.c                              |  2 +-
 .../selftests/kvm/x86_64/vmx_pmu_msrs_test.c    | 17 -----------------
 2 files changed, 1 insertion(+), 18 deletions(-)

-- 
2.33.1

Re: [PATCH v2 0/2] KVM: x86: Fix MSR_IA32_PERF_CAPABILITIES writes check and vmx_pmu_msrs_test

Posted by Paolo Bonzini 4 years, 6 months ago

On 12/16/21 17:52, Vitaly Kuznetsov wrote:
> This is a continuation of "KVM: selftests: Avoid KVM_SET_CPUID2 after
> KVM_RUN in vmx_pmu_msrs_test" work. Instead of fixing the immediate issue,
> drop incorrect check in KVM which was making the result of host initiated
> writes to MSR_IA32_PERF_CAPABILITIES dependent on guest visible CPUIDs and
> the corresponding tests in vmx_pmu_msrs_test, this will also make the issue
> reported by kernel test robot to go away.
> 
> Vitaly Kuznetsov (2):
>    KVM: selftests: vmx_pmu_msrs_test: Drop tests mangling guest visible
>      CPUIDs
>    KVM: x86: Drop guest CPUID check for host initiated writes to
>      MSR_IA32_PERF_CAPABILITIES
> 
>   arch/x86/kvm/x86.c                              |  2 +-
>   .../selftests/kvm/x86_64/vmx_pmu_msrs_test.c    | 17 -----------------
>   2 files changed, 1 insertion(+), 18 deletions(-)
> 

Queued, thanks,

Paolo