Re: [patch 05/18] net/s2io: replace schedule_timeout() with msleep()

[Nishanth Aravamudan <nacc@xxxxxxxxxx>]

On Sun, Oct 31, 2004 at 06:09:34AM -0500, Jeff Garzik wrote:
> janitor@xxxxxxxxxxxxxx wrote:
> >@@ -2846,11 +2838,10 @@ static int s2io_ethtool_idnic(struct net
> >             sp->id_timer.data = (unsigned long) sp;
> >     }
> >     mod_timer(&sp->id_timer, jiffies);
> >-    set_current_state(TASK_INTERRUPTIBLE);
> >     if (data)
> >-            schedule_timeout(data * HZ);
> >+            msleep_interruptible(data * 1000);
> 
> 
> clearly wants ssleep() here

Even though ssleep() is in TASK_UNINTERRUPTIBLE? In cases where the
existing code used TASK_INTERRUPTIBLE, but was sleeping in terms of
seconds, I left it as msleep_interruptible, since ssleep_interruptible()
doesn't exist.

-Nish