From: "Michael Chan" <[EMAIL PROTECTED]>
Date: Wed, 24 Aug 2005 11:33:19 -0700

> On Wed, 2005-08-24 at 15:52 -0400, Jeff Garzik wrote:
> 
> > 2) Would prefer to follow the generic net stack and other areas of the 
> > kernel, for the last piece of quoted code.  net stack used to loop on 
> > schedule_timeout(1) in dev_close() [net/core/dev.c], which has now been 
> > updated to loop on msleep(1).  As the code comment there notes, we're 
> > not in a hurry here.
> > 
> 
> I see other drivers doing yield() also. But if you prefer, I can change
> it to msleep().

There are people walking the entire kernel tree converting things
to msleep(), so you might want to just beat them to it since it's
inevitable. :-)

It takes care of all of the salient issues of setting and restoring
the task state, the ordering of the operations, etc.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to