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

[Hans de Goede]

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

?

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