From: Anthony Liguori <anthony@codemonkey.ws>
To: Max Krasnyansky <maxk@kernel.org>
Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: Re: [PATCH 2/5] husb: support for USB host device auto connect.
Date: Thu, 14 Aug 2008 15:21:50 -0500 [thread overview]
Message-ID: <48A493DE.40506@codemonkey.ws> (raw)
In-Reply-To: <48A489D3.5070900@kernel.org>
Max Krasnyansky wrote:
> Anthony Liguori wrote:
>
>> Max Krasnyansky wrote:
>>
>>> Anyway, it's implemented using a periodic timer that scans host devices
>>> and grabs those that match the filter. Timer is started when the first
>>> filter is added.
>>>
>>>
>> Again, there has to be a way to get notified of usb device add/remove in
>> Linux.
>>
>
> Yes the best way is probably registering with HAL via dbus.
>
I was afraid you would say that :-)
> Do you mind if we do something like that incrementally ?
>
Yeah, I don't really want to have QEMU depend on dbus so in this case,
polling would be better.
> ie Merge the current patch then change polling to HAL registration.
> That would simplify testing/patch stacking for me. Existing users are
> not affected anyway since the timer is activated only when new filter is added.
>
Regards,
Anthony Liguori
WARNING: multiple messages have this Message-ID (diff)
From: Anthony Liguori <anthony@codemonkey.ws>
To: Max Krasnyansky <maxk@kernel.org>
Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: [Qemu-devel] Re: [PATCH 2/5] husb: support for USB host device auto connect.
Date: Thu, 14 Aug 2008 15:21:50 -0500 [thread overview]
Message-ID: <48A493DE.40506@codemonkey.ws> (raw)
In-Reply-To: <48A489D3.5070900@kernel.org>
Max Krasnyansky wrote:
> Anthony Liguori wrote:
>
>> Max Krasnyansky wrote:
>>
>>> Anyway, it's implemented using a periodic timer that scans host devices
>>> and grabs those that match the filter. Timer is started when the first
>>> filter is added.
>>>
>>>
>> Again, there has to be a way to get notified of usb device add/remove in
>> Linux.
>>
>
> Yes the best way is probably registering with HAL via dbus.
>
I was afraid you would say that :-)
> Do you mind if we do something like that incrementally ?
>
Yeah, I don't really want to have QEMU depend on dbus so in this case,
polling would be better.
> ie Merge the current patch then change polling to HAL registration.
> That would simplify testing/patch stacking for me. Existing users are
> not affected anyway since the timer is activated only when new filter is added.
>
Regards,
Anthony Liguori
next prev parent reply other threads:[~2008-08-14 20:22 UTC|newest]
Thread overview: 59+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-14 4:22 [PATCH 0/5] Various USB fixes and improvements Max Krasnyansky
2008-08-14 4:22 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 4:22 ` [PATCH 1/5] husb: support for USB host device auto disconnect Max Krasnyansky
2008-08-14 4:22 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 16:28 ` Anthony Liguori
2008-08-14 16:28 ` [Qemu-devel] " Anthony Liguori
2008-08-14 19:26 ` Max Krasnyansky
2008-08-14 19:26 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 21:41 ` Max Krasnyansky
2008-08-14 21:41 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 4:22 ` [PATCH 2/5] husb: support for USB host device auto connect Max Krasnyansky
2008-08-14 4:22 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 16:41 ` Anthony Liguori
2008-08-14 16:41 ` [Qemu-devel] " Anthony Liguori
2008-08-14 19:38 ` Max Krasnyansky
2008-08-14 19:38 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 20:21 ` Anthony Liguori [this message]
2008-08-14 20:21 ` Anthony Liguori
2008-08-14 20:34 ` Max Krasnyansky
2008-08-14 20:34 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 20:41 ` Anthony Liguori
2008-08-14 20:41 ` [Qemu-devel] " Anthony Liguori
2008-08-14 21:14 ` François Revol
2008-08-15 7:46 ` Guido Günther
2008-08-15 7:46 ` Guido Günther
2008-08-15 18:24 ` Max Krasnyansky
2008-08-15 18:31 ` Javier Guerra
2008-08-18 18:21 ` Max Krasnyansky
2008-08-18 18:52 ` Javier Guerra
2008-08-18 18:56 ` Jamie Lokier
2008-08-18 18:56 ` Jamie Lokier
2008-08-17 7:52 ` Avi Kivity
2008-08-18 18:46 ` Max Krasnyansky
2008-08-18 14:11 ` Anthony Liguori
2008-08-18 18:16 ` Max Krasnyansky
2008-08-14 4:22 ` [PATCH 3/5] usb: generic packet handler cleanup and documentation Max Krasnyansky
2008-08-14 4:22 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 4:22 ` [PATCH 4/5] uhci: rewrite UHCI emulator, fully async operation with multiple outstanding transactions Max Krasnyansky
2008-08-14 4:22 ` [Qemu-devel] " Max Krasnyansky
2008-08-14 17:51 ` Anthony Liguori
2008-08-14 17:51 ` [Qemu-devel] " Anthony Liguori
2008-08-14 19:49 ` Max Krasnyansky
2008-08-14 19:49 ` [Qemu-devel] " Max Krasnyansky
2008-10-11 23:54 ` [Qemu-devel] " Juergen Lock
2008-10-15 19:54 ` Max Krasnyansky
2008-10-15 22:05 ` andrzej zaborowski
2008-10-16 21:25 ` Juergen Lock
2008-08-14 4:22 ` [PATCH 5/5] husb: rewrite Linux host USB layer, fully async operation Max Krasnyansky
2008-08-14 4:22 ` [Qemu-devel] " Max Krasnyansky
2008-08-15 14:24 ` Paul Brook
2008-08-15 14:24 ` Paul Brook
2008-08-15 19:04 ` Max Krasnyansky
2008-08-15 19:53 ` Paul Brook
2008-08-15 19:53 ` Paul Brook
2008-08-18 18:40 ` Max Krasnyansky
2008-08-14 17:55 ` [PATCH 0/5] Various USB fixes and improvements Anthony Liguori
2008-08-14 17:55 ` [Qemu-devel] " Anthony Liguori
2008-08-14 19:55 ` Max Krasnyansky
2008-08-14 19:55 ` [Qemu-devel] " Max Krasnyansky
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=48A493DE.40506@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=kvm@vger.kernel.org \
--cc=maxk@kernel.org \
--cc=qemu-devel@nongnu.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.