diff options
author | Sean Christopherson <seanjc@google.com> | 2022-05-27 15:13:03 -0700 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2022-06-11 11:47:28 -0400 |
commit | 3ea9b809650b4eda5d4ae18ed7bb080e499af154 (patch) | |
tree | 69a88487730d38d154c7e7fea7af84973b8ac713 /tools/testing/selftests/kvm/x86_64/tsc_scaling_sync.c | |
parent | KVM: selftests: Return an 'unsigned int' from kvm_check_cap() (diff) | |
download | linux-dev-3ea9b809650b4eda5d4ae18ed7bb080e499af154.tar.xz linux-dev-3ea9b809650b4eda5d4ae18ed7bb080e499af154.zip |
KVM: selftests: Add kvm_has_cap() to provide syntactic sugar
Add kvm_has_cap() to wrap kvm_check_cap() and return a bool for the use
cases where the caller only wants check if a capability is supported,
i.e. doesn't care about the value beyond whether or not it's non-zero.
The "check" terminology is somewhat ambiguous as the non-boolean return
suggests that '0' might mean "success", i.e. suggests that the ioctl uses
the 0/-errno pattern. Provide a wrapper instead of trying to find a new
name for the raw helper; the "check" terminology is derived from the name
of the ioctl, so using e.g. "get" isn't a clear win.
Signed-off-by: Sean Christopherson <seanjc@google.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'tools/testing/selftests/kvm/x86_64/tsc_scaling_sync.c')
-rw-r--r-- | tools/testing/selftests/kvm/x86_64/tsc_scaling_sync.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/kvm/x86_64/tsc_scaling_sync.c b/tools/testing/selftests/kvm/x86_64/tsc_scaling_sync.c index 4a962952212e..fcc713ff75ff 100644 --- a/tools/testing/selftests/kvm/x86_64/tsc_scaling_sync.c +++ b/tools/testing/selftests/kvm/x86_64/tsc_scaling_sync.c @@ -93,7 +93,7 @@ static void *run_vcpu(void *_cpu_nr) int main(int argc, char *argv[]) { - if (!kvm_check_cap(KVM_CAP_VM_TSC_CONTROL)) { + if (!kvm_has_cap(KVM_CAP_VM_TSC_CONTROL)) { print_skip("KVM_CAP_VM_TSC_CONTROL not available"); exit(KSFT_SKIP); } |