lkml.org 
[lkml]   [2000]   [Sep]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: test9pre6 usb-storage
I'm the usb-storage maintainer.  Yes, I realize that there is really no
need to reset the state to TASK_RUNNING, but I felt better having those
there. Considering that code is from the reset routines which almost never
get called, I figured it was fine.

Matt

On Sun, Sep 24, 2000 at 03:55:34PM +0100, John Levon wrote:
>
> These chunks :
>
> /* long wait for reset */
> + set_current_state(TASK_UNINTERRUPTIBLE);
> schedule_timeout(HZ*6);
> + set_current_state(TASK_RUNNING);
>
> and
>
> + /* long wait for reset */
> + set_current_state(TASK_UNINTERRUPTIBLE);
> + schedule_timeout(HZ*6);
> + set_current_state(TASK_RUNNING);
>
> seem slightly wrong. Unless someone wants to correct me, there
> should be no need to set TASK_RUNNING after schedule_timeout()
> has been called.
>
> john
>
> p.s. I am not subscribed to linux-usb-devel
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> Please read the FAQ at http://www.tux.org/lkml/

--
Matthew Dharm Home: mdharm-usb@one-eyed-alien.net
Maintainer, Linux USB Mass Storage Driver

Sir, for the hundreth time, we do NOT carry 600-round boxes of belt-fed
suction darts!
-- Salesperson to Greg
User Friendly, 12/30/1997
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2005-03-22 12:38    [W:0.035 / U:6.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site