Re: [PATCH 2/3] qemu: add support for qemu-xhci USB controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2017-04-20 at 15:44 +0200, Pavel Hrdina wrote:
[...]
> @@ -3464,7 +3464,7 @@
>          "ehci", "ich9-ehci1", "ich9-uhci1", "ich9-uhci2", "ich9-uhci3",
>          "vt82c686b-uhci", "pci-ohci", "nec-xhci", "qusb1" (xen pvusb
>          with qemu backend, version 1.1) or "qusb2" (xen pvusb with qemu
> -        backend, version 2.0). Additionally,
> +        backend, version 2.0), qemu-xhci. Additionally,

Add double quotes around qemu-xhci to conform with the rest
of the documentation. Additionally, having

  "nec-xhci", "qusb1" or "qusb2", "qemu-xhci"

looks weird, I think you should change it to

  "nec-xhci", "qusb1", "qusb2" or "qemu-xhci"

I won't get into the whole Oxford comma argument ;)

[...]
> @@ -0,0 +1,15 @@
> +<domain type='qemu'>
> +  <name>QEMUGuest1</name>
> +  <uuid>c7a5fdbd-edaf-9455-926a-d65c16db1809</uuid>
> +  <memory unit='KiB'>219136</memory>
> +  <currentMemory unit='KiB'>219136</currentMemory>

I like having minimal input files for test cases!

You can make them even smaller, without reducing the test
surface, by getting rid of the <currentMemory> element...

[...]
> new file mode 100644
> index 0000000000..78d13e0c1e
> --- /dev/null
> +++ b/tests/qemuxml2argvdata/qemuxml2argv-usb-controller-qemu-xhci-unavailable.xml

... and you can get rid of this file altogether by using...

> @@ -1416,6 +1416,10 @@ mymain(void)
>      DO_TEST_PARSE_ERROR("usb-controller-xhci-limit",
>              QEMU_CAPS_CHARDEV, QEMU_CAPS_NODEFCONFIG, QEMU_CAPS_PIIX3_USB_UHCI,
>              QEMU_CAPS_NEC_USB_XHCI, QEMU_CAPS_NEC_USB_XHCI_PORTS);
> +    DO_TEST("usb-controller-qemu-xhci", QEMU_CAPS_DEVICE_QEMU_XHCI);
> +    DO_TEST_FAILURE("usb-controller-qemu-xhci-unavailable", NONE);

... "usb-controller-qemu-xhci" again for this test case.


ACK with these issues fixed.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux