From: Bjørn Mork <bj...@mork.no> 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: <sta...@vger.kernel.org> # v3.5 > Cc: Ming Lei <ming....@canonical.com> > Cc: Oliver Neukum <oneu...@suse.de> > Signed-off-by: Bjørn Mork <bj...@mork.no> Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
