On Mon, 2009-07-06 at 13:17 +0800, Yolkfull Chow wrote: > Hi Lucas, since the qemu_iotests test suite itself has checked qemu, > qemu-img and qemu-io ELF files before running, I would skip the > duplicated step. Fair enough. Making the check on the autotest module would make things more apparent to the user (we would throw a TestFailure with the reason), but I see your point. > I added 'qemu_path' option for users to specify their alternative PATH > containing these ELF files. > > > + if qemu_path: > > + os.environ['PATH'] += ":" + qemu_path > > + Good. Martin, any comments? The test module and the qemu-iotests suite are looking good, can we commit it? Lucas -- 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