Hi, I dont'' know the arm architecture but while searching the code I found the following copy failure I think: Dietmar. diff --git a/xen/arch/arm/smpboot.c b/xen/arch/arm/smpboot.c index b836be4..43fb821 100644 --- a/xen/arch/arm/smpboot.c +++ b/xen/arch/arm/smpboot.c @@ -33,7 +33,7 @@ cpumask_t cpu_online_map; EXPORT_SYMBOL(cpu_online_map); cpumask_t cpu_present_map; -EXPORT_SYMBOL(cpu_online_map); +EXPORT_SYMBOL(cpu_present_map); cpumask_t cpu_possible_map; EXPORT_SYMBOL(cpu_possible_map); -- Company details: http://ts.fujitsu.com/imprint.html
On Fri, 15 Nov 2013, Dietmar Hahn wrote:> Hi, > > I dont'' know the arm architecture but while searching the code I found the > following copy failure I think: > > Dietmar. >Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>> diff --git a/xen/arch/arm/smpboot.c b/xen/arch/arm/smpboot.c > index b836be4..43fb821 100644 > --- a/xen/arch/arm/smpboot.c > +++ b/xen/arch/arm/smpboot.c > @@ -33,7 +33,7 @@ > cpumask_t cpu_online_map; > EXPORT_SYMBOL(cpu_online_map); > cpumask_t cpu_present_map; > -EXPORT_SYMBOL(cpu_online_map); > +EXPORT_SYMBOL(cpu_present_map); > cpumask_t cpu_possible_map; > EXPORT_SYMBOL(cpu_possible_map); > > > -- > Company details: http://ts.fujitsu.com/imprint.html > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel >
On Fri, 2013-11-15 at 14:50 +0000, Stefano Stabellini wrote:> On Fri, 15 Nov 2013, Dietmar Hahn wrote: > > Hi, > > > > I dont'' know the arm architecture but while searching the code I found the > > following copy failure I think: > > > > Dietmar. > > > > > Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>EXPORT_SYMBOL is meaningless in Xen and provided only for compatibility with code taken from Linux. But our smpboot.c is not taken from Linux, I expect we blindly inherited them from the x86 version (which has also long since diverged). I think we can just nuke them. Ian.
On Tue, 2013-11-19 at 10:36 +0000, Ian Campbell wrote:> On Fri, 2013-11-15 at 14:50 +0000, Stefano Stabellini wrote: > > On Fri, 15 Nov 2013, Dietmar Hahn wrote: > > > Hi, > > > > > > I dont'' know the arm architecture but while searching the code I found the > > > following copy failure I think: > > > > > > Dietmar. > > > > > > > > > Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> > > EXPORT_SYMBOL is meaningless in Xen and provided only for compatibility > with code taken from Linux. But our smpboot.c is not taken from Linux, I > expect we blindly inherited them from the x86 version (which has also > long since diverged). > > I think we can just nuke them.Like so: 8>------------- From 5f5cc43a23b89f095b00f654c698e85875d01998 Mon Sep 17 00:00:00 2001 From: Ian Campbell <ian.campbell@citrix.com> Date: Thu, 21 Nov 2013 10:49:14 +0000 Subject: [PATCH] xen: arm: Remove useless and incorrect EXPORT_SYMBOLS Dietmar noticed that one of these was incorrect, but in actual fact they are not even used. They are defined as nops in Xen for the benefit of code imported to Linux which ARM''s smpboot.c is not. Just get rid of them. Reported-by: Dietmar Hahn <dietmar.hahn@ts.fujitsu.com> Signed-off-by: Ian Campbell <ian.campbell@citrix.com> --- xen/arch/arm/smpboot.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/xen/arch/arm/smpboot.c b/xen/arch/arm/smpboot.c index 6c90fa6..ff712bb 100644 --- a/xen/arch/arm/smpboot.c +++ b/xen/arch/arm/smpboot.c @@ -31,11 +31,8 @@ #include <asm/gic.h> cpumask_t cpu_online_map; -EXPORT_SYMBOL(cpu_online_map); cpumask_t cpu_present_map; -EXPORT_SYMBOL(cpu_online_map); cpumask_t cpu_possible_map; -EXPORT_SYMBOL(cpu_possible_map); struct cpuinfo_arm cpu_data[NR_CPUS]; -- 1.7.10.4
On Thu, 2013-11-21 at 10:53 +0000, Ian Campbell wrote:> On Tue, 2013-11-19 at 10:36 +0000, Ian Campbell wrote: > > On Fri, 2013-11-15 at 14:50 +0000, Stefano Stabellini wrote: > > > On Fri, 15 Nov 2013, Dietmar Hahn wrote: > > > > Hi, > > > > > > > > I dont'' know the arm architecture but while searching the code I found the > > > > following copy failure I think: > > > > > > > > Dietmar. > > > > > > > > > > > > > Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> > > > > EXPORT_SYMBOL is meaningless in Xen and provided only for compatibility > > with code taken from Linux. But our smpboot.c is not taken from Linux, I > > expect we blindly inherited them from the x86 version (which has also > > long since diverged). > > > > I think we can just nuke them. > > Like so: > > 8>------------- > > From 5f5cc43a23b89f095b00f654c698e85875d01998 Mon Sep 17 00:00:00 2001 > From: Ian Campbell <ian.campbell@citrix.com> > Date: Thu, 21 Nov 2013 10:49:14 +0000 > Subject: [PATCH] xen: arm: Remove useless and incorrect EXPORT_SYMBOLS > > Dietmar noticed that one of these was incorrect, but in actual fact they are > not even used. They are defined as nops in Xen for the benefit of code > imported to Linux which ARM''s smpboot.c is not. > > Just get rid of them. > > Reported-by: Dietmar Hahn <dietmar.hahn@ts.fujitsu.com> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>This is so trivial I''ve just applied it.