sched, sparc32: Update scheduler comments in copy_thread()
authorIngo Molnar <mingo@kernel.org>
Tue, 4 Aug 2015 07:42:54 +0000 (09:42 +0200)
committerIngo Molnar <mingo@kernel.org>
Tue, 4 Aug 2015 07:48:12 +0000 (09:48 +0200)
There's no finish_arch_switch() anymore in the latest scheduler tree.
Also update some other details.

Cc: David S. Miller <davem@davemloft.net>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: linux-kernel@vger.kernel.org
Cc: sparclinux@vger.kernel.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
arch/sparc/kernel/process_32.c

index 50e7b626afe86493c54fc686d900ce8806bb2c77..c5113c7ce2fd4cd03628867b4eb13f4064dd8c18 100644 (file)
@@ -333,11 +333,11 @@ int copy_thread(unsigned long clone_flags, unsigned long sp,
        childregs = (struct pt_regs *) (new_stack + STACKFRAME_SZ);
 
        /*
-        * A new process must start with interrupts closed in 2.5,
-        * because this is how Mingo's scheduler works (see schedule_tail
-        * and finish_arch_switch). If we do not do it, a timer interrupt hits
-        * before we unlock, attempts to re-take the rq->lock, and then we die.
-        * Thus, kpsr|=PSR_PIL.
+        * A new process must start with interrupts disabled, see schedule_tail()
+        * and finish_task_switch(). (If we do not do it and if a timer interrupt
+        * hits before we unlock and attempts to take the rq->lock, we deadlock.)
+        *
+        * Thus, kpsr |= PSR_PIL.
         */
        ti->ksp = (unsigned long) new_stack;
        p->thread.kregs = childregs;