Subject: kern/32354: Recent patch breaks usb for some device/controller
To: None <kern-bug-people@netbsd.org, gnats-admin@netbsd.org,>
From: Mark A. Wicks <mwicks@gaspra.kettering.edu>
List: netbsd-bugs
Date: 12/21/2005 15:35:00
>Number:         32354
>Category:       kern
>Synopsis:       Recent patch breaks usb for some device/controller
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    kern-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Dec 21 15:35:00 +0000 2005
>Originator:     Mark A. Wicks
>Release:        NetBSD 3.0 (and current)
>Organization:

>Environment:
System: NetBSD gaspra.kettering.edu 3.0 NetBSD 3.0 (GASPRA - Customized) #3: Tue Dec 20 16:28:11 EST 2005 mwicks@gaspra.kettering.edu:/var/tmp/build/obj/sys/arch/i386/compile/GASPRA-3.0 i386
Architecture: i386
Machine: i386
>Description:
	A recent patch to usb_subr.c (revision 1.122.2.1 in netbsd-3
or revision 1.130 in current) breaks certain USB device/controller
combinations.  The patch submitter noted that he expected certain
buggy devices would break, but I have a scanner that works on some
computers and not others with this patch.  I don't think the scanner
itself is the problem.

Here are the symptoms: The "scanimage" command from sane-backends
hangs. The command "scanimage -L" completes the first time, but hangs
the second time. I can provide additional information if somebody
tells me what information would be useful.

>How-To-Repeat:
Here's the hardware combination that doesn't work:

uhci0 at pci0 dev 16 function 0: VIA Technologies VT83C572 USB Controller (rev. 0x81)
usb0 at uhci0: USB revision 1.0

Controller /dev/usb4:
addr 1: high speed, self powered, config 1, EHCI root hub(0x0000), VIA Technologie(0x1106), rev 1.00
 port 4 addr 2: high speed, self powered, config 1, EPSON Scanner(0x011e), EPSON(0x04b8), rev 1.00

But on another computer with

uhci0 at pci0 dev 7 function 2: Intel 82371AB (PIIX4) USB Host Controller (rev. 0x01)

everything works perfectly.

>Fix:
   Reverse the patch to usb_subr.c (1.122.2.1 in netbsd-3 or 1.130 in -current)

>Unformatted:
 combinations