Re: [RFC PATCH] KVM-test: Add subtest: usb

[Amos Kong <akong@redhat.com>]

----- Original Message -----
> On 07/29/11 06:53, Amos Kong wrote:
> > This test adds a usb storage for the guest, and do some check from
> > monitor and
> > inside the guest.
> > It's not very stable, could you help to review if something is
> > wrong?
> 
> "Not stable" means what exactly?

I found you had added new cmdline support of usb(ahci,usb2) for autotest.
But it's not supported right now.

"12:42:16 INFO | (qemu) *** EHCI support is under development ***"

So I plan to use old options, "-usbdevice disk:format=qcow2:/tmp/usbdevice.qcow2".
guest can identify the new disk.

> > + output = session.get_command_output("lsusb")
> > + #no bus specified, default using "usb.0" for "usb-storage"
> > + if "ID 0000:0000" not in output:
> > + logging.debug(output)
> > + raise error.TestFail("No 'ID 0000:0000' in the output of 'lsusb'")
> 
> You can use "lsusb -v" here, then you get something better to match.
> Also lsusb does some more usb requests then. Should also check for
> errors in the lsusb output.

Thanks for your comments, will send V2 later.

> > + output = session.get_command_output("fdisk -l")
> > + if params.get("fdisk_string") not in output:
> > + logging.debug(output)
> > + raise error.TestFail("Could not realise the usb device")
> > +
> > + error.context("Formating usb disk")
> > + dev_list = session.get_command_output("ls /dev/sd[a-z]")
> > + session.cmd("yes |mkfs %s" % dev_list.split()[-1],
> > + timeout=int(params.get("format_timeout")))
> 
> You can look at /dev/disk/by-path/ to figure which /dev/sd<x> the usb
> flashdrive is.

ok.

> cheers,
> Gerd