[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080509095837.GB19617@elte.hu>
Date: Fri, 9 May 2008 11:58:37 +0200
From: Ingo Molnar <mingo@...e.hu>
To: WANG Cong <xiyou.wangcong@...il.com>
Cc: toralf.foerster@....de, James.Bottomley@...senpartnership.com,
linux-kernel@...r.kernel.org, "H. Peter Anvin" <hpa@...or.com>,
Thomas Gleixner <tglx@...utronix.de>,
Mike Travis <travis@....com>
Subject: Re: build issue #380 for v2.6.26-rc1-279-g28a4acb : mach-voyager:
multiple definition of `phys_cpu_present_map'
* WANG Cong <xiyou.wangcong@...il.com> wrote:
> Sorry. Ignore previous one. This one should be correct, I think.
>
> ---
>
> diff --git a/arch/x86/mach-voyager/voyager_smp.c b/arch/x86/mach-voyager/voyager_smp.c
> index 8acbf0c..2e8388b 100644
> --- a/arch/x86/mach-voyager/voyager_smp.c
> +++ b/arch/x86/mach-voyager/voyager_smp.c
> @@ -74,7 +74,7 @@ EXPORT_SYMBOL(cpu_online_map);
>
> /* Bitmask of CPUs present in the system - exported by i386_syms.c, used
> * by scheduler but indexed physically */
> -cpumask_t phys_cpu_present_map = CPU_MASK_NONE;
> +static cpumask_t phys_cpu_present_map = CPU_MASK_NONE;
i've got the one below queued up but it's stalled as it could easily be
wrong and there's no ack from James yet and there's been discussion back
and forth. James, what do you suggest?
Ingo
-------------------->
Subject: x86, voyager: build fix
From: Ingo Molnar <mingo@...e.hu>
Date: Sun May 04 23:56:30 CEST 2008
Signed-off-by: Ingo Molnar <mingo@...e.hu>
---
arch/x86/mach-voyager/voyager_smp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-x86.q/arch/x86/mach-voyager/voyager_smp.c
===================================================================
--- linux-x86.q.orig/arch/x86/mach-voyager/voyager_smp.c
+++ linux-x86.q/arch/x86/mach-voyager/voyager_smp.c
@@ -74,7 +74,7 @@ EXPORT_SYMBOL(cpu_online_map);
/* Bitmask of CPUs present in the system - exported by i386_syms.c, used
* by scheduler but indexed physically */
-cpumask_t phys_cpu_present_map = CPU_MASK_NONE;
+extern cpumask_t phys_cpu_present_map;
/* The internal functions */
static void send_CPI(__u32 cpuset, __u8 cpi);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists