[PATCH v2 0/2] KVM: selftests: Report enable_pmu module value when test is skipped

Like Xu posted 2 patches 2 years, 6 months ago
tools/testing/selftests/kvm/include/kvm_util_base.h        | 1 +
tools/testing/selftests/kvm/lib/kvm_util.c                 | 5 +++++
tools/testing/selftests/kvm/x86_64/pmu_event_filter_test.c | 1 +
tools/testing/selftests/kvm/x86_64/vmx_pmu_caps_test.c     | 1 +
4 files changed, 8 insertions(+)
[PATCH v2 0/2] KVM: selftests: Report enable_pmu module value when test is skipped
Posted by Like Xu 2 years, 6 months ago
Adequate info can help developers quickly distinguish whether the cause
is a code flaw or a platform limitation when a test fails or is skipped,
and this minor patch-set is doing a little work in that direction.

Previous:
https://lore.kernel.org/kvm/20230214084920.59787-1-likexu@tencent.com/

V1 -> V2 Changelog:
- Apply TEST_REQUIRE() to x86_64/vmx_pmu_caps_test as well;
- Put TEST_REQUIRE() at the very top; (Sean)

Like Xu (2):
  KVM: selftests: Add a helper to read kvm boolean module parameters
  KVM: selftests: Report enable_pmu module value when test is skipped

 tools/testing/selftests/kvm/include/kvm_util_base.h        | 1 +
 tools/testing/selftests/kvm/lib/kvm_util.c                 | 5 +++++
 tools/testing/selftests/kvm/x86_64/pmu_event_filter_test.c | 1 +
 tools/testing/selftests/kvm/x86_64/vmx_pmu_caps_test.c     | 1 +
 4 files changed, 8 insertions(+)


base-commit: 13738a3647368f7f600b30d241779bcd2a3ebbfd
-- 
2.39.2
Re: [PATCH v2 0/2] KVM: selftests: Report enable_pmu module value when test is skipped
Posted by Sean Christopherson 2 years, 5 months ago
On Mon, Mar 13, 2023, Like Xu wrote:
> Adequate info can help developers quickly distinguish whether the cause
> is a code flaw or a platform limitation when a test fails or is skipped,
> and this minor patch-set is doing a little work in that direction.
> 
> Previous:
> https://lore.kernel.org/kvm/20230214084920.59787-1-likexu@tencent.com/
> 
> V1 -> V2 Changelog:
> - Apply TEST_REQUIRE() to x86_64/vmx_pmu_caps_test as well;
> - Put TEST_REQUIRE() at the very top; (Sean)

Ah fudge, I missed this in my todo list and applied v1.  I'll probably just fixup
and force push.  One way or another, I'll take care of it.