![]() | ||||||||||
Messages in this thread
|
On Sat, 12 Jan 2002 12:23:09 -0800 Andrew Morton <akpm@zip.com.au> wrote: > And guess what? Nobody has tested the damn thing, so it's going > nowhere. Haven't had latency problems, to be honest. Maybe I should start playing mp3s while I code? 1) conditional_schedule? Hmmm... Why the __set_current_state? I think I prefer an explicit "if (need_schedule()) schedule()", with #define need_schedule() unlikely(current->need_resched) 2) I hate condsched.h: Use sched.h please! 3) Why this: > +#ifndef __LINUX_COMPILER_H > +#include <linux/compiler.h> > +#endif Other than that, I like this patch. Linus? Rusty. -- Anyone who quotes me in their sig is an idiot. -- Rusty Russell. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | |||||||||
| Last update: 2005-03-22 12:15 [from the cache] ©2003-2008 | ||||||||||