KVM: selftests: Have perf_test_util signal when to stop vCPUs

Browse source

Signal that a test run is complete through perf_test_args instead of
having tests open code a similar solution. Ensure that the field resets
to false at the beginning of a test run as the structure is reused
between test runs, eliminating a couple of bugs:

access_tracking_perf_test hangs indefinitely on a subsequent test run,
as 'done' remains true. The bug doesn't amount to much right now, as x86
supports a single guest mode. However, this is a precondition of
enabling the test for other architectures with >1 guest mode, like
arm64.

memslot_modification_stress_test has the exact opposite problem, where
subsequent test runs complete immediately as 'run_vcpus' remains false.

Co-developed-by: Sean Christopherson <seanjc@google.com>
Signed-off-by: Sean Christopherson <seanjc@google.com>
[oliver: added commit message, preserve spin_wait_for_next_iteration()]
Signed-off-by: Oliver Upton <oliver.upton@linux.dev>
Reviewed-by: Gavin Shan <gshan@redhat.com>
Signed-off-by: Marc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/20221118211503.4049023-2-oliver.upton@linux.dev

...

This commit is contained in:

Oliver Upton 2022-11-18 21:15:02 +00:00 • committed by Marc Zyngier

parent 30a0b95b13

commit 9ec1eb1bcc

4 changed files with 8 additions and 12 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

8

tools/testing/selftests/kvm/access_tracking_perf_test.c

View file

3

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

View file

3

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

View file

6

tools/testing/selftests/kvm/memslot_modification_stress_test.c

View file