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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to