Andreas Färber <afaer...@suse.de> writes: > Hello Anthony/Aurélien, > > This series extends test coverage to all 16 targets. > For now it tests that QOM type changes do not lead to QOM cast assertions. > > v2 extends it to cover virtually all machines (except Xen and pc*-x.y). > Where an fprintf() is touched, use error_report() instead.
Yes, we need such a smoke test for all targets. I toyed with it myself, but I haven't been able to go beyond the crude hackery we discussed about a year ago: https://lists.nongnu.org/archive/html/qemu-devel/2012-08/msg01197.html The problem is that many targets have mandatory options (fun oxymoron), such as -kernel or -pflash. If I understand your approach correctly, you solve it by making these mandatory options optional when qtest_enabled(). My idea was to create suitable dummy images, so we can provide the mandatory options. Guest won't be happy, but that's fine, as this smoke test doesn't want to run any guest code. I like my idea better, because with it we can run unmodified standard code. No testing of qtest_enabled(). However, you've got patches, and I haven't, and that means I like your *patches* infinitely more than mine ;)
