<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 24, 2018 at 4:32 PM Christophe Fergeau <<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Sep 22, 2018 at 12:21:04AM +0300, Yuri Benditovich wrote:<br>
> On Sat, Sep 22, 2018 at 12:14 AM, Yuri Benditovich <<br>
> <a href="mailto:yuri.benditovich@daynix.com" target="_blank">yuri.benditovich@daynix.com</a>> wrote:<br>
> <br>
> > Hi<br>
> ><br>
> > On Fri, Sep 21, 2018 at 9:10 PM, Christophe Fergeau <<a href="mailto:cfergeau@redhat.com" target="_blank">cfergeau@redhat.com</a>><br>
> > wrote:<br>
> ><br>
> >> On Mon, Sep 17, 2018 at 04:22:50PM +0300, Yuri Benditovich wrote:<br>
> >> > Changes from v3:<br>
> >> > Single commit with all new files split to 12 patches<br>
> >> > to make patch review easier<br>
> >><br>
> >> What you did is not splitting at all, adding one new file per commit,<br>
> >> and then all the modifications to the existing code in a final commit<br>
> >> is not what we meant when we asked for split patches.<br>
> >><br>
> >> You asked why this is needed, and actually, it turns out your patches<br>
> >> introduce a regression in virt-viewer even when not exercising at all<br>
> >> any of the new code. The series as it is is not bisectable in any way,<br>
> >> so I basically have to look at these 9000 lines of code to try to figure<br>
> >> out what broke...<br>
> >><br>
> ><br>
> BTW I'm sorry to write it, but for the record the patch is not 9000 lines,<br>
> if is less than 2500 lines including all.<br>
<br>
cd-scsi.c itself which I assume to be new code is 2700 lines, the new<br>
widget is 2000 lines, usb-backend-common.c is 2000 lines. I'm taking the<br>
9000 lines figure from your cover letter by the way..<br>
<a href="https://lists.freedesktop.org/archives/spice-devel/2018-September/045576.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/archives/spice-devel/2018-September/045576.html</a><br>
 26 files changed, 9321 insertions(+), 520 deletions(-)<br>
<br>
<br>
> I believe if you describe the kind of regression I am able to find root<br>
> cause of it quickly.<br>
<br>
I'm sure you can, but if you are the only one who can do that, that's a<br>
maintainance concern. The issue is most likely caused by a leak of the<br>
usb channel (or a leak from it). Start latest virt-viewer to connect to<br>
a VM (probably with USB redir channels configured), exit it, and see it<br>
hangs.<br>
<br></blockquote><div><br></div><div>Exit viewer or exit VM? </div><div>Usually I the test with built-in remote-viewer, running it from .libs directory</div><div>with LD_LIBRARY_PATH=.</div><div><br></div><div>Nothing hangs.</div><div><br></div><div>Let's take it to separate mail thread, may be? Or all this process must be piblic and here?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Christophe<br>
</blockquote></div></div>