On 2/10/22 11:56 AM, Paolo Bonzini wrote:
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,
Thank you for explaining the discrepancy. Would you like take
this through your tree? This patch depends on
commit: de1956f48543e90f94b1194395f33140898b39b2
thanks,
-- Shuah
