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

2012-10-08 Thread Shawn Starr
On Monday, October 08, 2012 11:32:54 PM Hans de Goede wrote: > Hi, > > On 10/08/2012 10:18 PM, Shawn Starr wrote: > > On Monday, October 08, 2012 10:10:49 PM Hans de Goede wrote: > >> Hi, > >> > >> On 10/08/2012 06:11 PM, Johannes Stezenbach wrote: > >> > >> > >> > >> > I also wonder if this

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

2012-10-08 Thread Hans de Goede
Hi, On 10/08/2012 10:18 PM, Shawn Starr wrote: On Monday, October 08, 2012 10:10:49 PM Hans de Goede wrote: Hi, On 10/08/2012 06:11 PM, Johannes Stezenbach wrote: > I also wonder if this is not a generic problem for all emulated hw if the driver uses some timeout? Yes it likely is, be

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

2012-10-08 Thread Shawn Starr
On Monday, October 08, 2012 10:10:49 PM Hans de Goede wrote: > Hi, > > On 10/08/2012 06:11 PM, Johannes Stezenbach wrote: > > > > > I also wonder if this is not a generic problem > > > > for all emulated hw if the driver uses some timeout? > > Yes it likely is, because the emulated timer int

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

2012-10-08 Thread Hans de Goede
Hi, On 10/08/2012 06:11 PM, Johannes Stezenbach wrote: > I also wonder if this is not a generic problem for all emulated hw if the driver uses some timeout? Yes it likely is, because the emulated timer interrupt will try to make up for time lost when the vm was not running (in either hyperv

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

2012-10-08 Thread Johannes Stezenbach
Hi, On Mon, Oct 08, 2012 at 05:03:24PM +0200, Hans de Goede wrote: > On 10/08/2012 04:49 PM, Johannes Stezenbach wrote: > >On Mon, Oct 08, 2012 at 03:51:08PM +0200, Hans de Goede wrote: > >>the real problem is the > >>"ehci warning: guest updated active QH" message, which most likely indicates > >

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

2012-10-08 Thread Hans de Goede
Hi, On 10/08/2012 04:49 PM, Johannes Stezenbach wrote: Hi, On Mon, Oct 08, 2012 at 03:51:08PM +0200, Hans de Goede wrote: On 10/08/2012 03:01 PM, Johannes Stezenbach wrote: There will always be a race between the call to USBDEVFS_DISCARDURB and the URB completing. IMHO the handling in usb_h

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

2012-10-08 Thread Johannes Stezenbach
Hi, On Mon, Oct 08, 2012 at 03:51:08PM +0200, Hans de Goede wrote: > On 10/08/2012 03:01 PM, Johannes Stezenbach wrote: > > > >There will always be a race between the call to USBDEVFS_DISCARDURB > >and the URB completing. IMHO the handling in usb_host_stop_n_free_iso() > >is buggy. How about dro

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

2012-10-08 Thread Hans de Goede
Hi, On 10/08/2012 03:01 PM, Johannes Stezenbach wrote: Hi Hans, On Mon, Oct 08, 2012 at 01:27:28PM +0200, Hans de Goede wrote: On 10/02/2012 05:26 PM, Shawn Starr wrote: Reopening this issue with usb-host stalling now ehci warning: guest updated active QH USBDEVFS_DISCARDURB: Invalid argume

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

2012-10-08 Thread Johannes Stezenbach
Hi Hans, On Mon, Oct 08, 2012 at 01:27:28PM +0200, Hans de Goede wrote: > On 10/02/2012 05:26 PM, Shawn Starr wrote: > > > >Reopening this issue with usb-host stalling now > > > >ehci warning: guest updated active QH > >USBDEVFS_DISCARDURB: Invalid argument > >USBDEVFS_DISCARDURB: Invalid argument

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

2012-10-08 Thread Hans de Goede
Hi, On 10/02/2012 05:26 PM, Shawn Starr wrote: On Monday, September 24, 2012 10:38:57 AM Shawn Starr wrote: On Monday, September 24, 2012 04:36:15 PM Hans de Goede wrote: Hi, Hello, Reopening this issue with usb-host stalling now ehci warning: guest updated active QH USBDEVFS_DISCARDURB:

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

2012-10-02 Thread Shawn Starr
On Monday, September 24, 2012 10:38:57 AM Shawn Starr wrote: > On Monday, September 24, 2012 04:36:15 PM Hans de Goede wrote: > > Hi, > > Hello, Reopening this issue with usb-host stalling now ehci warning: guest updated active QH USBDEVFS_DISCARDURB: Invalid argument USBDEVFS_DISCARDURB: Inval

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

2012-09-24 Thread Shawn Starr
On Monday, September 24, 2012 04:36:15 PM Hans de Goede wrote: > Hi, > > On 09/24/2012 04:20 PM, Shawn Starr wrote: > > 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 Shaw

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

2012-09-24 Thread Hans de Goede
Hi, On 09/24/2012 04:20 PM, Shawn Starr wrote: 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, s

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

2012-09-24 Thread Shawn Starr
On Monday, September 24, 2012 11:52:36 AM Hans de Goede wrote: > Hi, > > On 09/23/2012 08:52 PM, Shawn Starr wrote: > > I see why there are USB errors, I am seeing corrupt video frames with > > webcam attached via spice-usbredirection, this is not seen with host-usb > > attachment. > What errors

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

2012-09-24 Thread 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, > > > > > > > >>> spicec is deprecat

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

2012-09-24 Thread Hans de Goede
Hi, On 09/23/2012 08:52 PM, Shawn Starr wrote: I see why there are USB errors, I am seeing corrupt video frames with webcam attached via spice-usbredirection, this is not seen with host-usb attachment. What errors exactly are you talking about I did not notice any error messages in your logs.

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

2012-09-24 Thread 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, spicec is deprecated you really should be using remote-viewer instead. That will give you an option to select devic

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

2012-09-23 Thread Shawn Starr
On Sunday, September 23, 2012 02:20:09 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, > > > > > > spicec is deprecated you really should be using remote-viewer instead. > > > That >

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

2012-09-23 Thread Shawn Starr
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, > > > > spicec is deprecated you really should be using remote-viewer instead. > > That > > will give you an option to select devices. > The author of spice-gt

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

2012-09-23 Thread Shawn Starr
On Sunday, September 23, 2012 12:03:28 PM Hans de Goede wrote: > Hi, > > On 09/21/2012 08:46 PM, Shawn Starr wrote: > > On Friday, September 21, 2012 07:35:42 PM Hans de Goede wrote: > >> Hi, > >> > >> On 09/21/2012 05:39 PM, Shawn Starr wrote: > >>> On Friday, September 21, 2012 02:19:08 PM Hans

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

2012-09-23 Thread Hans de Goede
Hi, On 09/21/2012 08:46 PM, Shawn Starr wrote: On Friday, September 21, 2012 07:35:42 PM Hans de Goede wrote: Hi, On 09/21/2012 05:39 PM, Shawn Starr wrote: On Friday, September 21, 2012 02:19:08 PM Hans de Goede wrote: Hi, Ok, so your using host redirection here, see the device usb-hos

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

2012-09-21 Thread Shawn Starr
On Friday, September 21, 2012 07:35:42 PM Hans de Goede wrote: > Hi, > > On 09/21/2012 05:39 PM, Shawn Starr wrote: > > On Friday, September 21, 2012 02:19:08 PM Hans de Goede wrote: > >> Hi, > > > > > > > >> Ok, so your using host redirection here, see the device usb-host > >> > >>> In additi

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

2012-09-21 Thread Hans de Goede
Hi, On 09/21/2012 05:39 PM, Shawn Starr wrote: On Friday, September 21, 2012 02:19:08 PM Hans de Goede wrote: Hi, Ok, so your using host redirection here, see the device usb-host In addition, if i disable usb-redirection device the cam works as it did before (although like before, the vi

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

2012-09-21 Thread Shawn Starr
On Friday, September 21, 2012 02:19:08 PM Hans de Goede wrote: > Hi, > > Ok, so your using host redirection here, see the device usb-host > > > In addition, if i disable usb-redirection device the cam works as it did > > before (although like before, the video image is upside down in some > > a

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

2012-09-21 Thread Hans de Goede
Hi, On 09/21/2012 02:04 AM, Shawn Starr wrote: On Thursday, September 20, 2012 07:29:08 PM Shawn Starr wrote: On Thursday, September 20, 2012 05:37:44 PM Hans de Goede wrote: Hi, On 09/19/2012 06:42 PM, Shawn Starr wrote: Hello QMU folks, The latest EHCI patches and or USB redirection ones

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

2012-09-20 Thread Shawn Starr
On Thursday, September 20, 2012 07:29:08 PM Shawn Starr wrote: > On Thursday, September 20, 2012 05:37:44 PM Hans de Goede wrote: > > Hi, > > > > On 09/19/2012 06:42 PM, Shawn Starr wrote: > > > Hello QMU folks, > > > > > > The latest EHCI patches and or USB redirection ones have caused a > > > r

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

2012-09-20 Thread Shawn Starr
On Thursday, September 20, 2012 05:37:44 PM Hans de Goede wrote: > Hi, > > On 09/19/2012 06:42 PM, Shawn Starr wrote: > > Hello QMU folks, > > > > The latest EHCI patches and or USB redirection ones have caused a > > regression. Using the (legacy) qemu-kvm git master repository which does > > not

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

2012-09-20 Thread Shawn Starr
On Thursday, September 20, 2012 05:37:44 PM Hans de Goede wrote: > Hi, > > Thanks for reporting this. This is caused by a recent change to > fix a memory leak inside the ehci codes interrupt ep handling, together > with: > > // TODO Windows does not seem to ever set the MULT field > > The a

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

2012-09-20 Thread Hans de Goede
Hi, On 09/19/2012 06:42 PM, Shawn Starr wrote: Hello QMU folks, The latest EHCI patches and or USB redirection ones have caused a regression. Using the (legacy) qemu-kvm git master repository which does not have these patches (not sure which patch is causing assert specifically yet). Using a

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

2012-09-19 Thread Shawn Starr
On Wednesday, September 19, 2012 12:42:01 PM Shawn Starr wrote: > Hello QMU folks, > Here's the commandline used: /usr/bin/qemu-kvm -name Windows -S -M pc-1.2 -cpu Penryn,+osxsave,+xsave, +pdcm,+xtpr,+tm2,+est,+smx,+vmx,+ds_cpl,+monitor,+dtes64,+pbe,+tm,+ht,+ss, +acpi,+ds,+vme -enable-kvm -m 102

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

2012-09-19 Thread Shawn Starr
Hello QMU folks, The latest EHCI patches and or USB redirection ones have caused a regression. Using the (legacy) qemu-kvm git master repository which does not have these patches (not sure which patch is causing assert specifically yet). Using a Logitech QuickCam Pro 9000 and starting a Window