All of lore.kernel.org
 help / color / mirror / Atom feed
From: Erik Rull <erik.rull@rdsoftware.de>
To: David Ahern <daahern@cisco.com>
Cc: kvm@vger.kernel.org
Subject: Re: USB EHCI patch for 0.14.0?
Date: Wed, 09 Mar 2011 23:48:59 +0100	[thread overview]
Message-ID: <4D7803DB.6050106@rdsoftware.de> (raw)
In-Reply-To: <4D77FBAD.8020703@cisco.com>

David Ahern wrote:
>
>
> On 03/09/11 14:28, Erik Rull wrote:
>> David Ahern wrote:
>> 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.

Yes, this time I patched against 0.14.0, I misunderstood your question, sorry.
My last attempt before that was kvm-88.

>> 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.

Thanks for the explanation!

> 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

I will give it another try tomorrow. I use
-usb -device usb-tablet -device usb-host
The last parameter for the auto-add to the guest
I will also try the old parameter syntax.
What does the no-hpet option do? Might this be somehow significant 
regarding the performance?

Best regards,

Erik

P.S.
When EHCI was active today without tablet and without auto-add I got a 
transfer rate of ~5MByte/sec from USB key to the HDD.

  reply	other threads:[~2011-03-09 22:49 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
2011-03-09 22:48       ` Erik Rull [this message]
  -- 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=4D7803DB.6050106@rdsoftware.de \
    --to=erik.rull@rdsoftware.de \
    --cc=daahern@cisco.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.