[Spice-devel] [PATCH spice-gtk] win-usb-dev: fix device arrival event logic

Christophe Fergeau cfergeau at redhat.com
Mon Jul 3 15:31:20 UTC 2017


On Mon, Jul 03, 2017 at 02:27:29PM +0300, Yuri Benditovich wrote:
> On Mon, Jul 3, 2017 at 10:16 AM, Christophe Fergeau <cfergeau at redhat.com>
> wrote:
> 
> > On Mon, Jul 03, 2017 at 07:48:31AM +0300, Yuri Benditovich wrote:
> > > https://bugzilla.redhat.com/show_bug.cgi?id=1425961
> > > If attached new device when one device with the same vid
> > > and pid already present, the notification is ignored and
> > > attached device is not redirected (if auto share set) and
> > > not displayed in USB devices widget
> >
> > There apparently were some issues in the past with bus/addr changing
> > when it should not
> > https://cgit.freedesktop.org/spice/spice-gtk/commit/?id=f9631cd6f8
> >
> > Any idea whether this is no longer needed?
> >
> 
> There is no additional information about case when the same device comes
> with different bus.addr
> 1. From my point of view this should not be a problem - if new device with
> different bus.addr comes in, the previous one with
> old bus.addr should disappear and be removed anywhere ; new device shall be
> redirected automatically if required.
> If such flow will be identified/reported with UsbDk, we will be able to
> investigate it and solve.
> 2. Whether the configuration with WinUSB is still used by spice-gtk on
> Windows? According to instructions
> on https://www.spice-space.org/spice-user-manual.html , UsbDk should be
> used and WinUSB is not mentioned.

To be honest, I don't know what WinUSB is.. ;) But since this patch is
removing an (old) bugfix, I'd explain in the commit log why it's no
longer necessary to have it.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170703/9a97dabe/attachment.sig>


More information about the Spice-devel mailing list