[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tests: minor cleanups in usb-hcd-uhci-test
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH] tests: minor cleanups in usb-hcd-uhci-test |
Date: |
Fri, 7 Oct 2016 10:49:12 +1100 |
User-agent: |
Mutt/1.7.0 (2016-08-17) |
On Thu, Oct 06, 2016 at 04:50:48PM +0200, Laurent Vivier wrote:
> Two minor cleanups:
> - exit gracefully in case on unsupported target,
> - put machine command line in a constant to avoid
> to duplicate it.
>
> Signed-off-by: Laurent Vivier <address@hidden>
Reviewed-by: David Gibson <address@hidden>
I'm not sure if I should take this through my tree or not.
> ---
> tests/usb-hcd-uhci-test.c | 15 +++++++++------
> 1 file changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/tests/usb-hcd-uhci-test.c b/tests/usb-hcd-uhci-test.c
> index 4b951ce..e956b9c 100644
> --- a/tests/usb-hcd-uhci-test.c
> +++ b/tests/usb-hcd-uhci-test.c
> @@ -77,6 +77,9 @@ static void test_usb_storage_hotplug(void)
> int main(int argc, char **argv)
> {
> const char *arch = qtest_get_arch();
> + const char *cmd = "-device piix3-usb-uhci,id=uhci,addr=1d.0"
> + " -drive id=drive0,if=none,file=/dev/null,format=raw"
> + " -device usb-tablet,bus=uhci.0,port=1";
> int ret;
>
> g_test_init(&argc, &argv, NULL);
> @@ -87,13 +90,13 @@ int main(int argc, char **argv)
> qtest_add_func("/uhci/pci/hotplug/usb-storage",
> test_usb_storage_hotplug);
>
> if (strcmp(arch, "i386") == 0 || strcmp(arch, "x86_64") == 0) {
> - qs = qtest_pc_boot("-device piix3-usb-uhci,id=uhci,addr=1d.0"
> - " -drive
> id=drive0,if=none,file=/dev/null,format=raw"
> - " -device usb-tablet,bus=uhci.0,port=1");
> + qs = qtest_pc_boot(cmd);
> } else if (strcmp(arch, "ppc64") == 0) {
> - qs = qtest_spapr_boot("-device piix3-usb-uhci,id=uhci,addr=1d.0"
> - " -drive
> id=drive0,if=none,file=/dev/null,format=raw"
> - " -device usb-tablet,bus=uhci.0,port=1");
> + qs = qtest_spapr_boot(cmd);
> + } else {
> + g_printerr("usb-hcd-uhci-test tests are only "
> + "available on x86 or ppc64\n");
> + exit(EXIT_FAILURE);
> }
> ret = g_test_run();
> qtest_shutdown(qs);
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature