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

Gerd Hoffmann
2011-08-01T07:33:57Z

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?

> + 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.

> + 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.

cheers,
Gerd