Re: Stable update of linux-2.6
- To: dann frazier <dannf@dannf.org>
- Cc: Ben Hutchings <ben@decadent.org.uk>, stable@kernel.org, Debian kernel maintainers <debian-kernel@lists.debian.org>, debian-release@lists.debian.org, "Adam D. Barratt" <adam@adam-barratt.org.uk>, greg@kroah.com, paulmck@linux.vnet.ibm.com, mikey@neuling.org, benh@kernel.crashing.org, anton@samba.org
- Subject: Re: Stable update of linux-2.6
- From: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
- Date: Sun, 3 Apr 2011 13:21:03 +0530
- Message-id: <20110403075103.GB4822@linux.vnet.ibm.com>
- Reply-to: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
- In-reply-to: <20110402172303.GB2611@dannf.org>
- References: <1301254282.26693.661.camel@localhost> <1301437578.12508.562.camel@hathi.jungle.funky-badger.org> <1301440319.10056.13.camel@localhost> <20110330084109.GA8255@linux.vnet.ibm.com> <20110402172303.GB2611@dannf.org>
* dann frazier <dannf@dannf.org> [2011-04-02 11:23:03]:
> 2.6.32.36 also fails to build on powerpc/SMP:
>
> CC arch/powerpc/kernel/crash.o
> arch/powerpc/kernel/crash.c: In function 'crash_kexec_wait_realmode':
> arch/powerpc/kernel/crash.c:176: error: 'paca' undeclared (first use in this function)
> arch/powerpc/kernel/crash.c:176: error: (Each undeclared identifier is reported only once
> arch/powerpc/kernel/crash.c:176: error: for each function it appears in.)
> make[1]: *** [arch/powerpc/kernel/crash.o] Error 1
> make: *** [arch/powerpc/kernel] Error 2
Hi Dann,
Can you please try the following patch, which adds the changes
introduced by Kumar Gala to the commit b3df895aeb to my previous patch.
powerpc: Fix default_machine_crash_shutdown #ifdef build failure
Introducing #ifdef to fix the build failure caused by
crash_kexec_wait_realmode(), with powerpc build with !SMP.
Reported-by: Ben Hutchings <ben@decadent.org.uk>
Reported-by: dann frazier <dannf@dannf.org>
Signed-off-by: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
cc: Michael Neuling <mikey@neuling.org>
cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
cc: Anton Blanchard <anton@samba.org>
cc: Kumar Gala <galak@kernel.crashing.org>
--
arch/powerpc/kernel/crash.c | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/powerpc/kernel/crash.c b/arch/powerpc/kernel/crash.c
index fe02e71..175d1a7 100644
--- a/arch/powerpc/kernel/crash.c
+++ b/arch/powerpc/kernel/crash.c
@@ -163,6 +163,7 @@ static void crash_kexec_prepare_cpus(int cpu)
}
/* wait for all the CPUs to hit real mode but timeout if they don't come in */
+#ifdef CONFIG_PPC_STD_MMU_64
static void crash_kexec_wait_realmode(int cpu)
{
unsigned int msecs;
@@ -187,6 +188,7 @@ static void crash_kexec_wait_realmode(int cpu)
}
mb();
}
+#endif
/*
* This function will be called by secondary cpus or by kexec cpu
@@ -445,7 +447,9 @@ void default_machine_crash_shutdown(struct pt_regs *regs)
crash_kexec_prepare_cpus(crashing_cpu);
cpu_set(crashing_cpu, cpus_in_crash);
crash_kexec_stop_spus();
+#if defined(CONFIG_PPC_STD_MMU_64) && defined(CONFIG_SMP)
crash_kexec_wait_realmode(crashing_cpu);
+#endif
if (ppc_md.kexec_cpu_down)
ppc_md.kexec_cpu_down(1, 0);
}
Kamalesh
Reply to: