Re: [PATCH v2 7/7] KVM: selftests: x86: Use TAP interface in the userspace_msr_exit test

From: Thomas Huth
Date: Thu Feb 08 2024 - 14:34:40 EST

On 26/01/2024 20.32, Sean Christopherson wrote:
On Thu, Oct 05, 2023, Thomas Huth wrote:
Use the kselftest_harness.h interface in this test to get TAP
output, so that it is easier for the user to see what the test
is doing.

Note: We're not using the KVM_ONE_VCPU_TEST() macro here (but the
generic TEST() macro from kselftest_harness.h) since each of the
tests needs a different guest code function.

I would much rather we add a KVM framework that can deal with this, i.e. build
something that is flexible from the get-go. Allowing tests to set the entry point
after vCPU is fairly straightforward (patch below, compile tested only on x86).

With that, my vote would be to have KVM_ONE_VCPU_TEST_SUITE() *always* pass NULL
for the entry point, and instead always require sub-tests to pass the guest code
to KVM_ONE_VCPU_TEST(). I think having the sub-test explicitly specify its guest
code will be helpful for developers reading the code.

Yes, I agree that sounds quite a bit nicer. I'll give it a try...