From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Christian Kujau <lists@nerdbynature.de>
Cc: linuxppc-dev@lists.ozlabs.org, LKML <linux-kernel@vger.kernel.org>
Subject: Re: error: 'cpus_in_crash' defined but not used
Date: Wed, 25 Jan 2012 10:18:40 +1100 [thread overview]
Message-ID: <1327447120.19850.46.camel@pasglop> (raw)
In-Reply-To: <alpine.DEB.2.01.1201201642580.2895@trent.utfs.org>
On Fri, 2012-01-20 at 16:45 -0800, Christian Kujau wrote:
> > On Tue, 17 Jan 2012 at 20:13, Christian Kujau wrote:
> > >
> > > compiling today's git (mainline, a25a2b8) on powerpc32 gives:
> > >
> > > CC arch/powerpc/kernel/crash.o
> > > cc1: warnings being treated as errors
> > > /usr/local/src/linux-2.6-git/arch/powerpc/kernel/crash.c:49: error:
> > > ‘cpus_in_crash’ defined but not used
> > > make[2]: *** [arch/powerpc/kernel/crash.o] Error 1
> > > make[1]: *** [arch/powerpc/kernel] Error 2
> > > make: *** [sub-make] Error 2
>
> This still happens in 3.3.0-rc1 and is fixed by the patch below.
>
> .config is here: http://nerdbynature.de/bits/3.2.0/
>
> Thoughts?
Applying now, thanks.
Cheers,
Ben.
> Thanks,
> Christian.
>
> > I could not find cpus_in_crash anywhere in the sourcetree, except for
> > arch/powerpc/kernel/crash.c. Moving the definition into the CONFIG_SMP
> > ifdef helps on my UP system, of course - not sure about other machines
> > though:
> >
> > diff --git a/arch/powerpc/kernel/crash.c b/arch/powerpc/kernel/crash.c
> > index 28be345..abef751 100644
> > --- a/arch/powerpc/kernel/crash.c
> > +++ b/arch/powerpc/kernel/crash.c
> > @@ -46,7 +46,6 @@
> >
> > /* This keeps a track of which one is the crashing cpu. */
> > int crashing_cpu = -1;
> > -static atomic_t cpus_in_crash;
> > static int time_to_dump;
> >
> > #define CRASH_HANDLER_MAX 3
> > @@ -66,6 +65,7 @@ static int handle_fault(struct pt_regs *regs)
> >
> > #ifdef CONFIG_SMP
> >
> > +static atomic_t cpus_in_crash;
> > void crash_ipi_callback(struct pt_regs *regs)
> > {
> > static cpumask_t cpus_state_saved = CPU_MASK_NONE;
> >
> >
> > Christian.
> > --
> > BOFH excuse #272:
> >
> > Netscape has crashed
> > --
> > 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/
> >
>
WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Christian Kujau <lists@nerdbynature.de>
Cc: LKML <linux-kernel@vger.kernel.org>, linuxppc-dev@lists.ozlabs.org
Subject: Re: error: 'cpus_in_crash' defined but not used
Date: Wed, 25 Jan 2012 10:18:40 +1100 [thread overview]
Message-ID: <1327447120.19850.46.camel@pasglop> (raw)
In-Reply-To: <alpine.DEB.2.01.1201201642580.2895@trent.utfs.org>
On Fri, 2012-01-20 at 16:45 -0800, Christian Kujau wrote:
> > On Tue, 17 Jan 2012 at 20:13, Christian Kujau wrote:
> > >
> > > compiling today's git (mainline, a25a2b8) on powerpc32 gives:
> > >
> > > CC arch/powerpc/kernel/crash.o
> > > cc1: warnings being treated as errors
> > > /usr/local/src/linux-2.6-git/arch/powerpc/kernel/crash.c:49: error:
> > > ‘cpus_in_crash’ defined but not used
> > > make[2]: *** [arch/powerpc/kernel/crash.o] Error 1
> > > make[1]: *** [arch/powerpc/kernel] Error 2
> > > make: *** [sub-make] Error 2
>
> This still happens in 3.3.0-rc1 and is fixed by the patch below.
>
> .config is here: http://nerdbynature.de/bits/3.2.0/
>
> Thoughts?
Applying now, thanks.
Cheers,
Ben.
> Thanks,
> Christian.
>
> > I could not find cpus_in_crash anywhere in the sourcetree, except for
> > arch/powerpc/kernel/crash.c. Moving the definition into the CONFIG_SMP
> > ifdef helps on my UP system, of course - not sure about other machines
> > though:
> >
> > diff --git a/arch/powerpc/kernel/crash.c b/arch/powerpc/kernel/crash.c
> > index 28be345..abef751 100644
> > --- a/arch/powerpc/kernel/crash.c
> > +++ b/arch/powerpc/kernel/crash.c
> > @@ -46,7 +46,6 @@
> >
> > /* This keeps a track of which one is the crashing cpu. */
> > int crashing_cpu = -1;
> > -static atomic_t cpus_in_crash;
> > static int time_to_dump;
> >
> > #define CRASH_HANDLER_MAX 3
> > @@ -66,6 +65,7 @@ static int handle_fault(struct pt_regs *regs)
> >
> > #ifdef CONFIG_SMP
> >
> > +static atomic_t cpus_in_crash;
> > void crash_ipi_callback(struct pt_regs *regs)
> > {
> > static cpumask_t cpus_state_saved = CPU_MASK_NONE;
> >
> >
> > Christian.
> > --
> > BOFH excuse #272:
> >
> > Netscape has crashed
> > --
> > 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/
> >
>
next prev parent reply other threads:[~2012-01-24 23:18 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-18 4:13 error: ‘cpus_in_crash’ defined but not used Christian Kujau
2012-01-18 5:13 ` error: 'cpus_in_crash' " Christian Kujau
2012-01-21 0:45 ` Christian Kujau
2012-01-21 0:45 ` Christian Kujau
2012-01-24 23:18 ` Benjamin Herrenschmidt [this message]
2012-01-24 23:18 ` Benjamin Herrenschmidt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1327447120.19850.46.camel@pasglop \
--to=benh@kernel.crashing.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=lists@nerdbynature.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.