<div dir="ltr">It does not exactly removes old bugfix. <div>First version of the code did check for bus.addr only and for example, ignored change of vid/pid without change of bus.addr (AFAIK this is what programmable devices like Cypress do).<div>Previous commit removed check for bus.addr and replaced it with check for vid and pid, effectively causing the bug we are fixing now.</div><div>Now we check all of them and ignore the update event only if there is no change in USB tree.</div><div><br></div><div>I'll mention the old commit in new commit message.</div></div><div>I'll also ask the author of previous commit for comment.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 3, 2017 at 6:31 PM, Christophe Fergeau <span dir="ltr"><<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, Jul 03, 2017 at 02:27:29PM +0300, Yuri Benditovich wrote:<br>
> On Mon, Jul 3, 2017 at 10:16 AM, Christophe Fergeau <<a href="mailto:cfergeau@redhat.com">cfergeau@redhat.com</a>><br>
> wrote:<br>
><br>
> > On Mon, Jul 03, 2017 at 07:48:31AM +0300, Yuri Benditovich wrote:<br>
> > > <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1425961" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/<wbr>show_bug.cgi?id=1425961</a><br>
> > > If attached new device when one device with the same vid<br>
> > > and pid already present, the notification is ignored and<br>
> > > attached device is not redirected (if auto share set) and<br>
> > > not displayed in USB devices widget<br>
> ><br>
> > There apparently were some issues in the past with bus/addr changing<br>
> > when it should not<br>
> > <a href="https://cgit.freedesktop.org/spice/spice-gtk/commit/?id=f9631cd6f8" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/<wbr>spice/spice-gtk/commit/?id=<wbr>f9631cd6f8</a><br>
> ><br>
> > Any idea whether this is no longer needed?<br>
> ><br>
><br>
> There is no additional information about case when the same device comes<br>
> with different bus.addr<br>
> 1. From my point of view this should not be a problem - if new device with<br>
> different bus.addr comes in, the previous one with<br>
> old bus.addr should disappear and be removed anywhere ; new device shall be<br>
> redirected automatically if required.<br>
> If such flow will be identified/reported with UsbDk, we will be able to<br>
> investigate it and solve.<br>
> 2. Whether the configuration with WinUSB is still used by spice-gtk on<br>
> Windows? According to instructions<br>
> on <a href="https://www.spice-space.org/spice-user-manual.html" rel="noreferrer" target="_blank">https://www.spice-space.org/<wbr>spice-user-manual.html</a> , UsbDk should be<br>
> used and WinUSB is not mentioned.<br>
<br>
</div></div>To be honest, I don't know what WinUSB is.. ;) But since this patch is<br>
removing an (old) bugfix, I'd explain in the commit log why it's no<br>
longer necessary to have it.<br>
<span class="HOEnZb"><font color="#888888"><br>
Christophe<br>
</font></span></blockquote></div><br></div>