KVM: selftests: Generalize check_clocksource() from kvm_clock_test

Browse source

Several existing x86 selftests need to check that the underlying system
clocksource is TSC or based on TSC but every test implements its own
check. As a first step towards unification, extract check_clocksource()
from kvm_clock_test and split it into two functions: arch-neutral
'sys_get_cur_clocksource()' and x86-specific 'sys_clocksource_is_tsc()'.
Fix a couple of pre-existing issues in kvm_clock_test: memory leakage in
check_clocksource() and using TEST_ASSERT() instead of TEST_REQUIRE().
The change also makes the test fail when system clocksource can't be read
from sysfs.

Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Link: https://lore.kernel.org/r/20240109141121.1619463-2-vkuznets@redhat.com
[sean: eliminate if-elif pattern just to set a bool true]
Signed-off-by: Sean Christopherson <seanjc@google.com>

...

This commit is contained in:

Vitaly Kuznetsov 2024-01-09 15:11:17 +01:00 • committed by Sean Christopherson

parent c2a449a30f

commit e440c5f2e3

5 changed files with 40 additions and 37 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

2

tools/testing/selftests/kvm/include/test_util.h

View file

2

tools/testing/selftests/kvm/include/x86_64/processor.h

View file

25

tools/testing/selftests/kvm/lib/test_util.c

View file

10

tools/testing/selftests/kvm/lib/x86_64/processor.c

View file

38

tools/testing/selftests/kvm/x86_64/kvm_clock_test.c

View file