qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Shawn Starr <shawn.starr@rogers.com>
To: Hans de Goede <hdegoede@redhat.com>
Cc: qemu-devel@nongnu.org, gerd@kraxel.org
Subject: Re: [Qemu-devel] EHCI USB regression in 1.2.0 - ehci_state_fetchqtd() asserting
Date: Mon, 24 Sep 2012 10:20:26 -0400	[thread overview]
Message-ID: <2726385.4Q3zJ6aFgT@segfault.sh0n.net> (raw)
In-Reply-To: <50602CE2.5080501@redhat.com>

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

  reply	other threads:[~2012-09-24 14:21 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-19 16:42 [Qemu-devel] EHCI USB regression in 1.2.0 - ehci_state_fetchqtd() asserting Shawn Starr
2012-09-19 16:53 ` Shawn Starr
2012-09-20 15:37 ` Hans de Goede
2012-09-20 19:08   ` Shawn Starr
2012-09-20 23:28   ` Shawn Starr
     [not found]   ` <6177450.mT8I4ey0nz@segfault.sh0n.net>
2012-09-21  0:04     ` Shawn Starr
2012-09-21 12:19       ` Hans de Goede
2012-09-21 15:39         ` Shawn Starr
2012-09-21 17:35           ` Hans de Goede
2012-09-21 18:46             ` Shawn Starr
2012-09-23 10:03               ` Hans de Goede
2012-09-23 18:00                 ` Shawn Starr
2012-09-23 18:20                   ` Shawn Starr
2012-09-23 18:52                     ` Shawn Starr
2012-09-24  9:52                       ` Hans de Goede
2012-09-24 14:24                         ` Shawn Starr
2012-09-24  9:50                     ` Hans de Goede
2012-09-24 14:20                       ` Shawn Starr [this message]
2012-09-24 14:36                         ` Hans de Goede
2012-09-24 14:38                           ` Shawn Starr
2012-10-02 15:26                             ` Shawn Starr
2012-10-08 11:27                               ` Hans de Goede
2012-10-08 13:01                                 ` Johannes Stezenbach
2012-10-08 13:51                                   ` Hans de Goede
2012-10-08 14:49                                     ` Johannes Stezenbach
2012-10-08 15:03                                       ` Hans de Goede
2012-10-08 16:11                                         ` Johannes Stezenbach
2012-10-08 20:10                                           ` Hans de Goede
2012-10-08 20:18                                             ` Shawn Starr
2012-10-08 21:32                                               ` Hans de Goede
2012-10-08 21:37                                                 ` Shawn Starr

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2726385.4Q3zJ6aFgT@segfault.sh0n.net \
    --to=shawn.starr@rogers.com \
    --cc=gerd@kraxel.org \
    --cc=hdegoede@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).