KVM: selftests: Improve error handling when opening files

[PATCH 0/4] KVM: selftests: Improve error handling when opening files

Sean Christopherson posted 4 patches 7 months ago

Download series mbox

.../selftests/kvm/access_tracking_perf_test.c |  7 ++----
.../testing/selftests/kvm/include/kvm_util.h  |  1 +
.../selftests/kvm/include/x86/processor.h     |  6 ++++-
tools/testing/selftests/kvm/lib/kvm_util.c    | 23 +++++++++++++++----
.../testing/selftests/kvm/lib/x86/processor.c | 10 --------
.../vmx_exception_with_invalid_guest_state.c  |  2 +-
6 files changed, 28 insertions(+), 21 deletions(-)

Expand all Fold all

[PATCH 0/4] KVM: selftests: Improve error handling when opening files

Posted by Sean Christopherson 7 months ago

Improve selftests' error reporting when opening a file fails, e.g. so that
failure to access a module param spits out a message about KVM not being
loaded, instead of a cryptic message about a param not being supported.

Sean Christopherson (4):
  KVM: selftests: Verify KVM is loaded when getting a KVM module param
  KVM: selftests: Add __open_path_or_exit() variant to provide extra
    help info
  KVM: selftests: Play nice with EACCES errors in open_path_or_exit()
  KVM: selftests: Print a more helpful message for EACCESS in access
    tracking test

 .../selftests/kvm/access_tracking_perf_test.c |  7 ++----
 .../testing/selftests/kvm/include/kvm_util.h  |  1 +
 .../selftests/kvm/include/x86/processor.h     |  6 ++++-
 tools/testing/selftests/kvm/lib/kvm_util.c    | 23 +++++++++++++++----
 .../testing/selftests/kvm/lib/x86/processor.c | 10 --------
 .../vmx_exception_with_invalid_guest_state.c  |  2 +-
 6 files changed, 28 insertions(+), 21 deletions(-)


base-commit: 7ef51a41466bc846ad794d505e2e34ff97157f7f
-- 
2.49.0.1112.g889b7c5bd8-goog

Re: [PATCH 0/4] KVM: selftests: Improve error handling when opening files

Posted by Sean Christopherson 5 months, 3 weeks ago

On Fri, 16 May 2025 14:59:05 -0700, Sean Christopherson wrote:
> Improve selftests' error reporting when opening a file fails, e.g. so that
> failure to access a module param spits out a message about KVM not being
> loaded, instead of a cryptic message about a param not being supported.
> 
> Sean Christopherson (4):
>   KVM: selftests: Verify KVM is loaded when getting a KVM module param
>   KVM: selftests: Add __open_path_or_exit() variant to provide extra
>     help info
>   KVM: selftests: Play nice with EACCES errors in open_path_or_exit()
>   KVM: selftests: Print a more helpful message for EACCESS in access
>     tracking test
> 
> [...]

Applied to kvm-x86 selftests, thanks!

[1/4] KVM: selftests: Verify KVM is loaded when getting a KVM module param
      https://github.com/kvm-x86/linux/commit/fcab107abe1a
[2/4] KVM: selftests: Add __open_path_or_exit() variant to provide extra help info
      https://github.com/kvm-x86/linux/commit/6e1cce7cda1b
[3/4] KVM: selftests: Play nice with EACCES errors in open_path_or_exit()
      https://github.com/kvm-x86/linux/commit/ba300a728f6f
[4/4] KVM: selftests: Print a more helpful message for EACCESS in access tracking test
      https://github.com/kvm-x86/linux/commit/71443210e26d

--
https://github.com/kvm-x86/kvm-unit-tests/tree/next