On Wed, Feb 01, 2012 at 03:28:34PM -0500, Paul Gortmaker wrote: > On 12-02-01 01:55 PM, Paul E. McKenney wrote: > > The variable cpus_in_crash gets a defined-but-not-used warning when > > CONFIG_SMP=n, so fix by placing the variable under ifdef. > > I think Ben already merged a similar fix. > > http://patchwork.ozlabs.org/patch/136549/ > > P.
Linus's tree does now have cpus_in_crash, so agreed. Sorry for the noise! Thanx, Paul > > Signed-off-by: Paul E. McKenney <paul...@linux.vnet.ibm.com> > > > > diff --git a/arch/powerpc/kernel/crash.c b/arch/powerpc/kernel/crash.c > > index 28be345..eb3384e 100644 > > --- a/arch/powerpc/kernel/crash.c > > +++ b/arch/powerpc/kernel/crash.c > > @@ -46,7 +46,9 @@ > > > > /* This keeps a track of which one is the crashing cpu. */ > > int crashing_cpu = -1; > > +#ifdef CONFIG_SMP > > static atomic_t cpus_in_crash; > > +#endif /* #ifdef CONFIG_SMP */ > > static int time_to_dump; > > > > #define CRASH_HANDLER_MAX 3 > > > _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev