> >                 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.

It would seem to me that you are throwing away wake ups
needlessly. Is there any reason you cannot put it before the while loop 
or even earlier?

        Regards
                Oliver

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

Reply via email to