Re: 2.4.6 possible problem

From: Richard B. Johnson (
Date: Tue Jul 17 2001 - 14:36:24 EST

On Tue, 17 Jul 2001, Linus Torvalds wrote:

> In article <>,
> Richard B. Johnson <> wrote:
> >
> > ticks = 1 * HZ; /* For 1 second */
> > while((ticks = interruptible_sleep_on_timeout(&wqhead, ticks)) > 0)
> > ;
> Don't do this.
> Imagine what happens if a signal comes in and wakes you up? The signal
> will continue to be pending, which will make your "sleep loop" be a busy
> loop as you can never go to sleep interruptibly with a pending signal.
> In short: if you have to wait for a certain time or for a certain event,
> you MUST NOT USE a interruptible sleep.
> If it is ok to return early due to signals or similar (which is nice -
> you can allow people to kill the process), then you use an interruptible
> sleep, but then you mustn't have the above kind of loop.
> Linus

Okay, then ../linux/drivers/net/8139too.c (line 2239) should be fixed
because that's where it came from.

Dick Johnson

Penguin : Linux version 2.4.1 on an i686 machine (799.53 BogoMips).

    I was going to compile a list of innovations that could be
    attributed to Microsoft. Once I realized that Ctrl-Alt-Del
    was handled in the BIOS, I found that there aren't any.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

This archive was generated by hypermail 2b29 : Mon Jul 23 2001 - 21:00:09 EST