On Thu, Oct 06, 2005 at 04:07:47PM -0700, Pete Zaitcev wrote: > On Thu, 6 Oct 2005 16:43:44 -0400 (EDT), Alan Stern <[EMAIL PROTECTED]> wrote: > > > This patch (as576) removes some mistaken tests for disconnection from the > > HID driver. -EILSEQ refers to an arbitrary low-level protocol error, not > > necessarily a disconnection. Also, a completion routine will never see a > > status of -EPERM; that's used only to indicate a failure during > > usb_submit_urb. > > > --- usb-2.6.orig/drivers/usb/input/hid-core.c > > +++ usb-2.6/drivers/usb/input/hid-core.c > > @@ -923,9 +923,7 @@ static void hid_irq_in(struct urb *urb, > > break; > > case -ECONNRESET: /* unlink */ > > case -ENOENT: > > - case -EPERM: > > case -ESHUTDOWN: /* unplug */ > > - case -EILSEQ: /* unplug timeout on uhci */ > > return; > > case -ETIMEDOUT: /* NAK */ > > break; > > No freaking way, man. Is there a particular scenario which this patch > fixes? Because that code is there for a reason. You cannot just do > it because some retarded document in Documentation/ suggests that it > might be a good idea. > > I have a bug assigned (bz#167070), which is, unfortunately, private. > But the summary goes like this > > HID driver completion handler prints -84(-EILSEQ) error messages > when a USB keyboard or mouse was unplugged. > > When a USB keyboard or mouse was unplugged, > HID driver completion handler hid_irq_in() > (@drivers/usb/input/hid-core.c) > continuously prints an error message "input irq status -84 received". > Once the USB was unplugged, the machine just hungs up. > > By the way, I have already found the patch to fix this > problem at http://lkml.org/lkml/2005/1/27/176. > The problem was settled by applying this patch to > drivers/usb/input/hid-core.c. > > Attached is a patch we may be shipping as a hotfix for RHEL 4 U2 and > which is going to be shipped in U3 by normal means.
Why add the error count? Mainline doesn't do it that way. > I can guarantee you Vojtech and Greg see the same with SLES. Hm, we never added that patch to SLES, I guess no one reported it... thanks, greg k-h ------------------------------------------------------- This SF.Net email is sponsored by: Power Architecture Resource Center: Free content, downloads, discussions, and more. http://solutions.newsforge.com/ibmarch.tmpl _______________________________________________ [email protected] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
