Re: [Qemu-devel] [5050] uhci: rewrite UHCI emulator, fully async operation with multiple outstanding transactions ( Max Krasnyansky)

[Aurelien Jarno]

On Thu, Aug 21, 2008 at 11:07:03PM -0700, Max Krasnyansky wrote:
> Aurelien Jarno wrote:
> > I am using QEMU, from the SVN, without KQEMU. The problem does not
> > depend of the console used (SDL, VNC or curses). The guest kernel 
> > outputs the following errors:
> > usb 1-2: device not accepting address 2, error -71
> > usb 1-2: new full speed USB device using uhci_hcd and address 3
> > usb 1-2: device not accepting address 3, error -71
> > usb 1-2: new full speed USB device using uhci_hcd and address 4
> > usb 1-2: device not accepting address 4, error -71
> > usb 1-2: new full speed USB device using uhci_hcd and address 5
> > usb 1-2: device not accepting address 5, error -71
> > 
> > You can download the image from:
> > http://people.debian.org/~aurel32/qemu/amd64/
> > 
> > It is reproducible with the following command line:
> > qemu-system-x86_64 -hda debian_etch_amd64_small.qcow -usbdevice tablet      
> >                                     
> > 
> > The error can also be reproduced with -usbdevice mouse or -usbdevice 
> > disk:image.
> Turns out I managed to screw up transaction length handling for control
> transfers in certain scenarious. Looks like XP and newer kernels are not
> sensitive to that.
> 
> I just sent out a patch that fixes the regression.
> 
> [PATCH] uhci: Fixed length handling for SETUP and OUT tokens
> 
> Please confirm that it fixes your setup. I tested XP, your Debian image and
> Fedora 9.

Yes, I confirm it works with my setup. Thanks for your reactivity.


-- 
  .''`.  Aurelien Jarno             | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   address@hidden         | address@hidden
   `-    people.debian.org/~aurel32 | www.aurel32.net