qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Shawn Starr <shawn.starr@rogers.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 11:50:26 +0200	[thread overview]
Message-ID: <50602CE2.5080501@redhat.com> (raw)
In-Reply-To: <32908230.8fxjKQhQHT@segfault.sh0n.net>

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

  parent reply	other threads:[~2012-09-24  9:49 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 [this message]
2012-09-24 14:20                       ` Shawn Starr
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=50602CE2.5080501@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=gerd@kraxel.org \
    --cc=qemu-devel@nongnu.org \
    --cc=shawn.starr@rogers.com \
    /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).