2015-12-17 13:42-0600, Andrew Jones: > On Thu, Dec 17, 2015 at 06:53:40PM +0100, Radim Krčmář wrote: >> It's not easy to distinguish successful unit-test from failed QEMU, so >> we check for presence of the needed feature before hand. >> >> Signed-off-by: Radim Krčmář <rkrcmar@xxxxxxxxxx> >> --- >> v2: remove "> /dev/null" as check doesn't print the output anymore >> diff --git a/x86/unittests.cfg b/x86/unittests.cfg >> @@ -182,3 +182,4 @@ arch = x86_64 >> file = hyperv_synic.flat >> smp = 2 >> extra_params = -cpu kvm64,hv_synic -device hyperv-testdev >> +check = echo quit | $qemu -cpu kvm64,hv_synic -device hyperv-testdev -monitor stdio > > Let's make sure $QEMU==$qemu in contexts where unittests.cfg is used, and > then document (in the unittests.cfg header) that $QEMU may be used in the > check lines. I'd just get rid of $qemu, we don't gain anything by having them both. Documenting it is a good idea. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html