> this patch is quite trivial.

Patch looks OK (just sets the new usb_driver "owner" field).
Greg, feel free to integrate.


>                 set_current_state (TASK_UNINTERRUPTIBLE);
>                 schedule_timeout (UNLINK_TIMEOUT_JIFFIES);
>                 dbg ("waited for %d urb completions", temp);
> 
> looks suspicious. You are setting the task's state just before you schedule.

Hmm, I seem to recall being told that's the right place to do it.
If that code were to change at all, I'd rather it just switch to
use the <linux/completion.h> idioms -- marginally faster.

- Dave



_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to