[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/9] microvm: add usb support
From: |
Gerd Hoffmann |
Subject: |
Re: [PATCH v2 4/9] microvm: add usb support |
Date: |
Mon, 12 Oct 2020 08:45:36 +0200 |
On Tue, Oct 06, 2020 at 12:03:26PM +0200, Igor Mammedov wrote:
> On Thu, 1 Oct 2020 10:28:29 +0200
> Gerd Hoffmann <kraxel@redhat.com> wrote:
>
> > Wire up "usb=on" machine option, when enabled add
> > a sysbus xhci controller with 8 ports.
>
> first pci, and now usb - not so micro anymore.
Both pcie and usb are optional and disabled by default.
> why not just reuse pci to add usb controller?
> what/how much do we win avoiding PCI?
Didn't benchmark stuff myself, but there are several reports that
the pcie initialization by the linux kernel at boot is noticable
(lots of vmexits for pci config space access).
IIRC this is also the reason why microvm started without pci support in
the first place. So being able to enable usb without having to enable
pcie too looks useful to me.
But, yes, functionality-wise there isn't much of a difference between
"-microvm usb=on" and "-microvm pcie=on -device qemu-xhci".
take care,
Gerd
- [PATCH v2 7/9] tests/acpi: add microvm usb test, (continued)
- [PATCH v2 7/9] tests/acpi: add microvm usb test, Gerd Hoffmann, 2020/10/01
- [PATCH v2 6/9] tests/acpi: add empty tests/data/acpi/microvm/DSDT.{usb, rtc} files, Gerd Hoffmann, 2020/10/01
- [PATCH v2 9/9] tests/acpi: update expected data files, Gerd Hoffmann, 2020/10/01
- [PATCH v2 8/9] tests/acpi: add microvm rtc test, Gerd Hoffmann, 2020/10/01
- [PATCH v2 5/9] tests/acpi: allow updates for expected data files, Gerd Hoffmann, 2020/10/01
- [PATCH v2 1/9] acpi: add aml builder stubs, Gerd Hoffmann, 2020/10/01
- [PATCH v2 2/9] usb/xhci: add include/hw/usb/xhci.h header file, Gerd Hoffmann, 2020/10/01
- [PATCH v2 3/9] usb/xhci: add xhci_sysbus_build_aml() helper, Gerd Hoffmann, 2020/10/01
- [PATCH v2 4/9] microvm: add usb support, Gerd Hoffmann, 2020/10/01