Re: [Qemu-devel] EHCI USB regression in 1.2.0 - ehci_state_fetchqtd() asserting

[Shawn Starr]

On Monday, September 24, 2012 11:50:26 AM Hans de Goede wrote:
> Hi,
> 
> On 09/23/2012 08:20 PM, Shawn Starr wrote:
> > On Sunday, September 23, 2012 02:00:10 PM Shawn Starr wrote:
> >> On Sunday, September 23, 2012 12:03:28 PM Hans de Goede wrote:
> >>> Hi,
> > 
> > <snip>
> > 
> >>> spicec is deprecated you really should be using remote-viewer instead.
> >>> That
> >>> will give you an option to select devices.
> > 
> > <snip this out>
> > 
> > The author of spice-gtk told me to downgrade since seems 0.14 broke, got
> > things working now, I do see some buffer overflow errors and drops but no
> > assert triggered.
> > 
> > I have a detailed debug it worked attached via remote-viewer
> > 
> > USB conversation:
> > 
> > qemu-kvm: usb-redir: ep: 00 type: 0 interface: 0
> > qemu-kvm: usb-redir: ep: 80 type: 0 interface: 0
> > qemu-kvm: usb-redir: ep: 87 type: 3 interface: 0
> > qemu-kvm: usb-redir: attaching high speed device 046d:0990 version 0.8
> > class ef
> > qemu-kvm: usb-redir: reset device
> > qemu-kvm: usb-redir: ctrl-out type 0x80 req 0x6 val 0x100 index 0 len 64
> > id
> > 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 18 id 1072685312
> > qemu-kvm: usb-redir: reset device
> > qemu-kvm: usb-redir: set address 1
> > qemu-kvm: usb-redir: ctrl-out type 0x80 req 0x6 val 0x100 index 0 len 18
> > id
> > 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 18 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0x80 req 0x6 val 0x200 index 0 len 255
> > id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 255 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0x80 req 0x6 val 0x200 index 0 len 1433
> > id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 1433 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0x80 req 0x6 val 0x302 index 1033 len
> > 255 id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 18 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0x80 req 0x6 val 0x100 index 0 len 18
> > id
> > 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 18 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0x80 req 0x6 val 0x200 index 0 len 9 id
> > 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 9 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0x80 req 0x6 val 0x200 index 0 len 1433
> > id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 1433 id 1072685312
> > qemu-kvm: usb-redir: set config 1 id 1072685440
> > qemu-kvm: usb-redir: set config status 0 config 1 id 1072685440
> > qemu-kvm: usb-redir: set interface 1 alt 0 id 1072685440
> > qemu-kvm: usb-redir: ep: 00 type: 0 interface: 0
> > qemu-kvm: usb-redir: ep: 80 type: 0 interface: 0
> > qemu-kvm: usb-redir: ep: 87 type: 3 interface: 0
> > qemu-kvm: usb-redir: alt status 0 intf 1 alt 0 id: 1072685440
> > qemu-kvm: usb-redir: set interface 3 alt 0 id 1072685440
> > qemu-kvm: usb-redir: ep: 00 type: 0 interface: 0
> > qemu-kvm: usb-redir: ep: 80 type: 0 interface: 0
> > qemu-kvm: usb-redir: ep: 87 type: 3 interface: 0
> > qemu-kvm: usb-redir: alt status 0 intf 3 alt 0 id: 1072685440
> > qemu-kvm: usb-redir: ctrl-out type 0xa1 req 0x85 val 0x100 index 1024 len
> > 2 id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 2 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0xa1 req 0x86 val 0x100 index 1024 len
> > 1 id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 1 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0xa1 req 0x85 val 0x100 index 1024 len
> > 2 id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 2 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0xa1 req 0x84 val 0x100 index 1024 len
> > 4 id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 4 id 1072685312
> > qemu-kvm: usb-redir: ctrl-out type 0xa1 req 0x82 val 0x100 index 1024 len
> > 4 id 1072685312
> > qemu-kvm: usb-redir: ctrl-in status 0 len 4 id 1072685312
> > <snip>
> > 
> > USB conversation during device usage:
> > 
> > 
> > ... <snip>
> 
> Hmm, no messages like these ones:
> 
> qemu-system-x86_64: usb-redir: interrupt-in status 0 ep 87 len 9 id 1
> 
> qemu-system-x86_64: usb-redir: interrupt-token-in ep 87 status 0 len 9
> 
> ?
> 

Only see that once in the conversation


qemu-kvm: usb-redir: ctrl-in status 0 len 2 id 1072685312

qemu-kvm: usb-redir: set interface 3 alt 0 id 1072685440

qemu-kvm: usb-redir: ep: 00 type: 0 interface: 0

qemu-kvm: usb-redir: ep: 80 type: 0 interface: 0

qemu-kvm: usb-redir: ep: 87 type: 3 interface: 0

qemu-kvm: usb-redir: alt status 0 intf 3 alt 0 id: 1072685440

___ qemu-kvm: usb-redir: interrupt-in status 0 ep 87 len 6 id 0

___ qemu-kvm: usb-redir: interrupt-token-in ep 87 status 0 len 6

qemu-kvm: usb-redir: ctrl-out type 0x21 req 0x1 val 0x200 index 1282 len 2 id 
1072685440

qemu-kvm: usb-redir: ctrl-in status 0 len 2 id 1072685440

> Those are the ones I'm looking for as they indicate that my patch not only
> fixes the assert, but that windows is actually getting interrupt packets
> from the camera ...
> 
> Regards,
> 
> Hans