public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: David Ahern <daahern@cisco.com>
To: Erik Rull <erik.rull@rdsoftware.de>
Cc: kvm@vger.kernel.org
Subject: Re: USB EHCI patch for 0.14.0?
Date: Wed, 09 Mar 2011 15:14:05 -0700	[thread overview]
Message-ID: <4D77FBAD.8020703@cisco.com> (raw)
In-Reply-To: <4D77F0E3.6070500@rdsoftware.de>



On 03/09/11 14:28, Erik Rull wrote:
> David Ahern wrote:
>> On 03/09/11 08:40, erik.rull@rdsoftware.de wrote:
>>> But some things are not working and cause my Windows guest to stop
>>> booting
>>> or getting slowed down:
>>> -device usb-tablet
>>> -device usb-mouse
>>> do not really work. (I have connected a PS/2 mouse to have no
>>> interference
>>> with the rest of the USB system that works fine without the patch)
>>> If I add them to the command line, windows does not boot up (it hangs
>>> before the GUI comes up with ~ 12% CPU time on the host side)
>>> If I add them at runtime via the qemu console it has no influence to the
>>> guest - I still see no possibility grabbing the mouse to the client
>>
>> Did those work with the previous qemu-kvm releases you tested?
> 
> I've tried it long time ago with kvm-88, but there it was working but
> extremely slow. (At least the mouse was reacting there)

I thought you tried the EHCI patch against a recent qemu-kvm version --
like December 2010 or January 2011.


> Hm, okay.
> As far as I understood it, the auto-add feature should be similar to the
> USB 1.1, right? It seems to work basically but not fully - the system is
> somehow slowed down, maybe the polling timer is too fast? (I will play a
> little bit with that and review and compare as well the auto routine)
> 
> And the usb-tablet should be an uhci-emulated component, that should
> then not interfere with the ehci-emulation, right?

My proposal from July 2010 was to have emulated devices state their
version and have host devices try EHCI then UHCI. This means that the
tablet device is attached to UHCI and a host USB key is attached to
EHCI. Like this:

info usb
  Device 0.1, Port 1, Speed 12 Mb/s, Product QEMU USB Tablet
  Device 1.1, Port , Speed 480 Mb/s, Product DT 101 II
(qemu)

That's what the patch I sent does.

EHCI does have a lot higher frame rate and in its current form does have
a noticeable impact on CPU usage when devices are connected and one of
the lists is activated. If the OS deactivates the controller when there
is nothing to do (e.g., not actively talking to the device), cpu usage
goes down.

I just tried a few scenarios with the X and V versions of those GUI
based guests and an external key attached to EHCI bus worked fine and
the usb tablet also worked fine.

I did notice some differences in command syntax. For instance, my
scripts still use the older -usbdevice tablet syntax and I did not see
the USB stall message. Switching to '-device usb-tablet' did generate
the message at boot (though overall it seems to be harmless).

Nothing fancy with the setup -- ide drive, virtio or e1000 networking,
ac97 sound, no-hpet and usb tablet devices.

In the time it took to write this response about 900MB was transferred
to the usb key at about 1.7-1.8MB/sec rate.

David

> 
> If you have any additional hints where to start digging, just let me know.
> 
> Best regards,
> 
> Erik
> 

  reply	other threads:[~2011-03-09 22:13 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-09 15:40 USB EHCI patch for 0.14.0? erik.rull
2011-03-09 16:12 ` David Ahern
2011-03-09 21:28   ` Erik Rull
2011-03-09 22:14     ` David Ahern [this message]
2011-03-09 22:48       ` Erik Rull
  -- strict thread matches above, loose matches on Subject: below --
2011-03-14  8:38 erik.rull
2011-03-17 18:05 ` Erik Rull
2011-03-11 15:18 erik.rull
2011-03-11 15:39 ` Markus Armbruster
2011-03-11 16:01   ` David Ahern
2011-03-11 16:17     ` Markus Armbruster
2011-03-11 16:34       ` David Ahern
2011-03-11 16:24   ` Erik Rull
2011-03-11 16:46     ` Markus Armbruster
2011-03-11 16:56       ` Erik Rull
2011-03-11 17:11         ` David Ahern
2011-03-11 17:31           ` David Ahern
2011-04-11  9:40             ` ya su
2011-04-11 13:23               ` David Ahern
2011-04-11 16:32                 ` Jan Kiszka
2011-04-11 16:46                 ` Jan Kiszka
2011-04-11 17:53                   ` David Ahern
2011-04-11 20:07                     ` Jan Kiszka
2011-04-11 21:10                       ` Alexander Graf
2011-03-11 15:57 ` David Ahern
2011-03-11 16:32   ` Erik Rull
2011-03-08 16:23 Erik Rull
2011-03-08 18:54 ` David Ahern
2011-06-06 11:33   ` André Weidemann
2011-06-06 13:18     ` David Ahern

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D77FBAD.8020703@cisco.com \
    --to=daahern@cisco.com \
    --cc=erik.rull@rdsoftware.de \
    --cc=kvm@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox