public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Nikola Ciprich <nikola.ciprich@linuxbox.cz>
To: linux-kernel@vger.kernel.org
Cc: nikola.ciprich@linuxbox.cz, stable@kernel.org,
	hirofuchi@users.sourceforge.net, arjan.mels@gmx.net, rjw@sisk.pl,
	greg@kroah.com, stern@rowland.harvard.edu,
	Stanislav SCHATTKE <schattke@linuxbox.cz>
Subject: [regression?] d199c96d41d80a567493e12b8e96ea056a1350c1 breaks usbip
Date: Sun, 1 May 2011 20:10:30 +0200	[thread overview]
Message-ID: <20110501181030.GA9106@nik-comp.lan> (raw)

[-- Attachment #1: Type: text/plain, Size: 3546 bytes --]

Hello,
we've hit regression introduced by d199c96d41d80a567493e12b8e96ea056a1350c1:

Author: Alan Stern <stern@rowland.harvard.edu>
Date:   Mon Jan 31 10:56:37 2011 -0500

    USB: prevent buggy hubs from crashing the USB stack
    
    If anyone comes across a high-speed hub that (by mistake or by design)
    claims to have no Transaction Translators, plugging a full- or
    low-speed device into it will cause the USB stack to crash.  This
    patch (as1446) prevents the problem by ignoring such devices, since
    the kernel has no way to communicate with them.
    
    Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
    Tested-by: Perry Neben <neben@vmware.com>
    CC: <stable@kernel.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

since this commit, trying to use staging/usbip spits message about missing TT:

[  388.893134] usbip_common_mod: module is from the staging directory, the quality is unknown, you have been warned.
[  388.898154] usbip_common_mod: usbip common driver1.0
[  388.905360] vhci_hcd: module is from the staging directory, the quality is unknown, you have been warned.
[  388.909991] vhci_hcd: vhci_hcd, 1.0
[  388.912370] usbip: proving...
[  388.912379] vhci_hcd vhci_hcd: USB/IP Virtual Host Contoroller
[  388.913052] vhci_hcd vhci_hcd: new USB bus registered, assigned bus number 2
[  388.932339] usb usb2: New USB device found, idVendor=1d6b, idProduct=0002
[  388.932344] usb usb2: New USB device strings: Mfr=3, Product=2, SerialNumber=1
[  388.932348] usb usb2: Product: USB/IP Virtual Host Contoroller
[  388.932351] usb usb2: Manufacturer: Linux 2.6.32lb.38 vhci_hcd
[  388.932354] usb usb2: SerialNumber: vhci_hcd
[  388.932531] usb usb2: configuration #1 chosen from 1 choice
[  388.932578] hub 2-0:1.0: USB hub found
[  388.932587] hub 2-0:1.0: 8 ports detected
[  478.379036] usbip: rhport(0) sockfd(3) devid(131075) speed(2)
[  478.379214] usbip: changed 1
[  478.500048] usbip: changed 0
[  478.608075] usb 2-1: new full speed USB device using vhci_hcd and address 2
[  478.608079] usb 2-1: parent hub has no TT
[  478.720074] usb 2-1: new full speed USB device using vhci_hcd and address 3
[  478.720078] usb 2-1: parent hub has no TT
[  478.832086] usb 2-1: new full speed USB device using vhci_hcd and address 4
[  478.832091] usb 2-1: parent hub has no TT
[  478.944076] usb 2-1: new full speed USB device using vhci_hcd and address 5
[  478.944080] usb 2-1: parent hub has no TT
[  478.944087] hub 2-0:1.0: unable to enumerate USB device on port 1

I'm not sure whether this is bug in d199c96d41d80a567493e12b8e96ea056a1350c1, I guess it might be usbip problem.
Since d199c96d41d80a567493e12b8e96ea056a1350c1 already got into -stable, usbip stopped working in 2.6.32.x (and I guess also stable and longterm kernels).

I also tried 2.6.39-rc5-git4 but usbip seems to be even more broken there. Here's dmesg trying to initialize usbip (with debugging enabled).
http://nelide.cz/nik/2.6.39-rc5-git4-usbip-dmesg.txt
(it's long, so I've uploaded it to web).

Could somebody please have a look at this? Should more information or testing be needed, I'll be glad to assist.

Cheers!

nik


-- 
-------------------------------------
Ing. Nikola CIPRICH
LinuxBox.cz, s.r.o.
28. rijna 168, 709 01 Ostrava

tel.:   +420 596 603 142
fax:    +420 596 621 273
mobil:  +420 777 093 799

www.linuxbox.cz

mobil servis: +420 737 238 656
email servis: servis@linuxbox.cz
-------------------------------------

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

             reply	other threads:[~2011-05-01 18:11 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-01 18:10 Nikola Ciprich [this message]
2011-05-01 19:27 ` [regression?] d199c96d41d80a567493e12b8e96ea056a1350c1 breaks usbip Greg KH
2011-05-02 14:09   ` Alan Stern
2011-05-02 16:59     ` Nikola Ciprich
2011-05-02 20:41       ` Nikola Ciprich

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=20110501181030.GA9106@nik-comp.lan \
    --to=nikola.ciprich@linuxbox.cz \
    --cc=arjan.mels@gmx.net \
    --cc=greg@kroah.com \
    --cc=hirofuchi@users.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rjw@sisk.pl \
    --cc=schattke@linuxbox.cz \
    --cc=stable@kernel.org \
    --cc=stern@rowland.harvard.edu \
    /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