From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Srivatsa S. Bhat" Subject: [RESEND PATCH 2/2] mn10300/CPU hotplug: Add missing call to notify_cpu_starting() Date: Wed, 16 May 2012 00:32:37 +0530 Message-ID: <20120515190228.22459.47763.stgit@srivatsabhat> References: <20120515190115.22459.93544.stgit@srivatsabhat> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Cc: akpm@linux-foundation.org, mingo@kernel.org, a.p.zijlstra@chello.nl, mpatocka@redhat.com, stepanm@codeaurora.org, tglx@linutronix.de, James.Bottomley@hansenpartnership.com, deller@gmx.de, dhowells@redhat.com, yasutake.koichi@jp.panasonic.com, eike-kernel@sf-tec.de, linux-parisc@vger.kernel.org, linux-kernel@vger.kernel.org, srivatsa.bhat@linux.vnet.ibm.com, linux-am33-list@redhat.com To: torvalds@linux-foundation.org Return-path: In-Reply-To: <20120515190115.22459.93544.stgit@srivatsabhat> List-ID: List-Id: linux-parisc.vger.kernel.org The scheduler depends on receiving the CPU_STARTING notification, without which we end up into a lot of trouble. So add the missing call to notify_cpu_starting() in the bringup code. Signed-off-by: Srivatsa S. Bhat --- arch/mn10300/kernel/smp.c | 9 ++++++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/arch/mn10300/kernel/smp.c b/arch/mn10300/kernel/smp.c index 910dddf..9cd69ad 100644 --- a/arch/mn10300/kernel/smp.c +++ b/arch/mn10300/kernel/smp.c @@ -24,6 +24,7 @@ #include #include #include +#include #include #include #include @@ -38,7 +39,6 @@ #include "internal.h" #ifdef CONFIG_HOTPLUG_CPU -#include #include static unsigned long sleep_mode[NR_CPUS]; @@ -874,10 +874,13 @@ static void __init smp_online(void) cpu = smp_processor_id(); - local_irq_enable(); + notify_cpu_starting(cpu); + ipi_call_lock(); set_cpu_online(cpu, true); - smp_wmb(); + ipi_call_unlock(); + + local_irq_enable(); } /**