Messages in this thread Patch in this message | | | Date | Wed, 25 May 2005 15:51:30 +0200 | From | Ingo Molnar <> | Subject | Re: [patch] remove set_tsk_need_resched() from init_idle() |
| |
* Andrew Morton <akpm@osdl.org> wrote:
> > --- linux/arch/x86_64/kernel/process.c.orig > > +++ linux/arch/x86_64/kernel/process.c > > @@ -162,6 +162,8 @@ EXPORT_SYMBOL_GPL(cpu_idle_wait); > > */ > > void cpu_idle (void) > > { > > + set_tsk_need_resched(current); > > + > > /* endless idle loop with no priority at all */ > > while (1) { > > while (!need_resched()) { > > ia64 needed this treatment also to fix a hang during boot. u o me 3 hrs. > > Are all the other architectures busted as well?
oh damn, they are indeed, because they need to hit schedule() at least once.
The patch below should address this problem for all architectures, by doing an explicit schedule() in the init code before calling into cpu_idle(). It's a replacement for the following patch:
sched-remove-set_tsk_need_resched-from-init_idle.patch
Ingo
--
This patch tweaks idle thread setup semantics a bit: instead of setting NEED_RESCHED in init_idle(), we do an explicit schedule() before calling into cpu_idle().
This patch, while having no negative side-effects, enables wider use of cond_resched()s. (which might happen in the stock kernel too, but it's particulary important for voluntary-preempt)
Signed-off-by: Ingo Molnar <mingo@elte.hu> Acked-by: Nick Piggin <nickpiggin@yahoo.com.au> Signed-off-by: Andrew Morton <akpm@osdl.org>
--- linux/kernel/sched.c.orig +++ linux/kernel/sched.c @@ -4163,6 +4163,14 @@ void show_state(void) read_unlock(&tasklist_lock); } +/** + * init_idle - set up an idle thread for a given CPU + * @idle: task in question + * @cpu: cpu the idle task belongs to + * + * NOTE: this function does not set the idle thread's NEED_RESCHED + * flag, to make booting more robust. + */ void __devinit init_idle(task_t *idle, int cpu) { runqueue_t *rq = cpu_rq(cpu); @@ -4180,7 +4188,6 @@ void __devinit init_idle(task_t *idle, i #if defined(CONFIG_SMP) && defined(__ARCH_WANT_UNLOCKED_CTXSW) idle->oncpu = 1; #endif - set_tsk_need_resched(idle); spin_unlock_irqrestore(&rq->lock, flags); /* Set the preempt count _outside_ the spinlocks! */ --- linux/init/main.c.orig +++ linux/init/main.c @@ -383,6 +383,13 @@ static void noinline rest_init(void) numa_default_policy(); unlock_kernel(); preempt_enable_no_resched(); + + /* + * The boot idle thread must execute schedule() + * at least one to get things moving: + */ + schedule(); + cpu_idle(); } - 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/
| |