[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH for-1.6? v2 00/21] qtest: Test all targets

From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH for-1.6? v2 00/21] qtest: Test all targets
Date: Tue, 06 Aug 2013 10:39:53 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux)

Andreas Färber <address@hidden> 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:

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 ;)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]