Ján Tomko <jto...@redhat.com> writes:

> Either leave this as-is and add the new feature in a separate test,
> or you can branch the test based on QEMU versions like we do for other
> features:
>
>     DO_TEST_CAPS_VER("pv-spinlock-disabled", "2.7.0");
>     DO_TEST_CAPS_VER("pv-spinlock-disabled", "4.0.0");
>     DO_TEST_CAPS_LATEST("pv-spinlock-disabled");

I think I slightly prefer a new test: this way we can test both 

 <stimer state='on'/>

 and

 <stimer state='on'>
   <direct state='on'/>
 </stimer>

(and it makes no sense to test hv-stimer-direct for old QEMU versions
and 4.1 is "the latest" atm).

Thank you for your review, v4 is comming soon (I promise)!

-- 
Vitaly

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to