On Wed, Mar 13, 2013 at 10:36 PM, Oliver Neukum <oneukum@xxxxxxx> wrote: > On Wednesday 13 March 2013 22:22:53 Du Xing wrote: >> From: Du Xing duxing2007@xxxxxxxxx >> >> In skel_read,the reader blocked in wait_for_completion before submit bulk in urb. >> Using processed_urb is totally unnecessary, remove it can fix the bug. > > The processed_urb is intended for the case that the interruptible wait in > skel_read was interrupted. So the diagnosis is good, but the fix is dubious. If skel_read is interrupted, the function will exit, so this code won't be run under the situation. Looks the patch is fine. Thanks, -- Ming Lei -- 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