Andreas Färber <afaer...@suse.de> writes: > Am 06.08.2013 10:39, schrieb Markus Armbruster: >> 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. > > Peter rejected having a U-Boot per machine. And from my own arm porting > experiences that would mean having close to one source of U-Boot per > machine since upstreaming works really badly in the embedded world. :(
Mind, I said "dummy images". Just good enough to fool QEMU into loading them. Guest surviving even a single cycle after reset is entirely optional. [...]
