On 2/10/22 19:02, Shuah Khan wrote:
I am fine with the change itself. For this patch: Reviewed-by: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> However, are we missing a vmx_pi_mmio_test and that test needs to be added. Just in case the test didn't make it into the 6a58150859fd and the intent was to add it - hence the Makefile addition? This can be addressed in another patch. Just want to make sure we aren't missing a test.
This was probably a rebase resolution issue. The vmx_pi_mmio_test fails in the mainline tree, and is lingering in my tree together with the fix because somebody promised a better fix for it. I included the TEST_GEN_PROGS_x86_64 line by mistake when rebasing Peter Gonda's test below vmx_pi_mmio_test.
Thanks for the fix, Acked-by: Paolo Bonzini <pbonzini@xxxxxxxxxx> Paolo