From: Bjørn Mork <bjorn@xxxxxxx> Date: Mon, 3 Sep 2012 10:26:18 +0200 > Suspending an open usbnet device results in constant > rescheduling of usbnet_bh. > > commit 65841fd5 "usbnet: handle remote wakeup asap" > refactored the usbnet_bh code to allow sharing the > urb allocate and submit code with usbnet_resume. In > this process, a test for, and immediate return on, > ENOLINK from rx_submit was unintentionally dropped. > > The rx queue will not grow if rx_submit fails, > making usbnet_bh reschedule itself. This results > in a softirq storm if the error is persistent. > rx_submit translates the usb_submit_urb error > EHOSTUNREACH into ENOLINK, so this is an expected > and persistent error for a suspended device. The > old code tested for this condition and avoided > rescheduling. Putting this test back. > > Cc: <stable@xxxxxxxxxxxxxxx> # v3.5 > Cc: Ming Lei <ming.lei@xxxxxxxxxxxxx> > Cc: Oliver Neukum <oneukum@xxxxxxx> > Signed-off-by: Bjørn Mork <bjorn@xxxxxxx> Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
