[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 8/8] usb-hid: Allow connecting to a USB-2 device
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 8/8] usb-hid: Allow connecting to a USB-2 device |
Date: |
Wed, 07 Nov 2012 10:47:53 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121016 Thunderbird/16.0.1 |
Il 06/11/2012 15:08, Hans de Goede ha scritto:
> Our ehci code has is capable of significantly lowering the wakeup rate
> for the hcd emulation while the device is idle. It is possible to add
> similar code ot the uhci emulation, but that simply is not there atm,
> and there is no reason why a (virtual) usb device can not be a USB-2 device.
>
> Making usb-hid devices connect to the emulated ehci controller instead
> of the emulated uhci controller on vms which have both lowers the cpuload
> for a fully idle vm from 20% to 2-3% (on my laptop).
You need this to be dependent on the machine version. Otherwise the USB
paths may change and migration will break.
Paolo
> Signed-off-by: Hans de Goede <address@hidden>
> ---
> hw/usb/dev-hid.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/hw/usb/dev-hid.c b/hw/usb/dev-hid.c
> index 69f89ff..96ba0c0 100644
> --- a/hw/usb/dev-hid.c
> +++ b/hw/usb/dev-hid.c
> @@ -97,7 +97,7 @@ static const USBDescIface desc_iface_mouse = {
> .bEndpointAddress = USB_DIR_IN | 0x01,
> .bmAttributes = USB_ENDPOINT_XFER_INT,
> .wMaxPacketSize = 4,
> - .bInterval = 0x0a,
> + .bInterval = 8,
> },
> },
> };
> @@ -127,7 +127,7 @@ static const USBDescIface desc_iface_tablet = {
> .bEndpointAddress = USB_DIR_IN | 0x01,
> .bmAttributes = USB_ENDPOINT_XFER_INT,
> .wMaxPacketSize = 8,
> - .bInterval = 0x0a,
> + .bInterval = 8,
> },
> },
> };
> @@ -158,7 +158,7 @@ static const USBDescIface desc_iface_keyboard = {
> .bEndpointAddress = USB_DIR_IN | 0x01,
> .bmAttributes = USB_ENDPOINT_XFER_INT,
> .wMaxPacketSize = 8,
> - .bInterval = 0x0a,
> + .bInterval = 8,
> },
> },
> };
> @@ -224,6 +224,7 @@ static const USBDesc desc_mouse = {
> .iSerialNumber = STR_SERIALNUMBER,
> },
> .full = &desc_device_mouse,
> + .high = &desc_device_mouse,
> .str = desc_strings,
> };
>
> @@ -237,6 +238,7 @@ static const USBDesc desc_tablet = {
> .iSerialNumber = STR_SERIALNUMBER,
> },
> .full = &desc_device_tablet,
> + .high = &desc_device_tablet,
> .str = desc_strings,
> };
>
> @@ -250,6 +252,7 @@ static const USBDesc desc_keyboard = {
> .iSerialNumber = STR_SERIALNUMBER,
> },
> .full = &desc_device_keyboard,
> + .high = &desc_device_keyboard,
> .str = desc_strings,
> };
>
>
- [Qemu-devel] usb: Move interrupt handling from poll to async handling, Hans de Goede, 2012/11/06
- [Qemu-devel] [PATCH 2/8] usb-redir: Store interrupt receiving status in the bufp-queue, Hans de Goede, 2012/11/06
- [Qemu-devel] [PATCH 1/8] usb-redir: Split usb_handle_interrupt_data into separate in/out functions, Hans de Goede, 2012/11/06
- [Qemu-devel] [PATCH 6/8] hid: Change idle handling to use a timer, Hans de Goede, 2012/11/06
- [Qemu-devel] [PATCH 5/8] ehci: Lower timer freq when there are no iso packets in the periodic schedule, Hans de Goede, 2012/11/06
- [Qemu-devel] [PATCH 4/8] usb-redir: Handle interrupt packets async, Hans de Goede, 2012/11/06
- [Qemu-devel] [PATCH 7/8] usb-hid: Move from NAK/polling to async packet handling, Hans de Goede, 2012/11/06
- [Qemu-devel] [PATCH 8/8] usb-hid: Allow connecting to a USB-2 device, Hans de Goede, 2012/11/06
- [Qemu-devel] [PATCH 3/8] usb-redir: Only add actually in flight packets to the in flight queue, Hans de Goede, 2012/11/06
- Re: [Qemu-devel] usb: Move interrupt handling from poll to async handling, Gerd Hoffmann, 2012/11/06