Re: EXPORT_SYMBOL(min_low_pfn);

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi all,

On Tue, 5 Feb 2013 11:31:22 -0800 Tony Luck <tony.luck@xxxxxxxxx> wrote:
>
> Today i noticed in linux-next this commit:
> 
> commit b0d9c17503444c2df8cb8e93ccfa4877edd71933
> Author: Sanjay Lal <sanjayl@xxxxxxxxxxx>
> Date:   Wed Nov 21 18:34:14 2012 -0800
> 
>     MIPS: Export symbols used by KVM/MIPS module
> 
> which causes this kernel build warning on ia64:
> 
> WARNING: vmlinux: 'min_low_pfn' exported twice. Previous export was in vmlinux
> 
> 
> We just need to delete the arch/ia64 specific export (and grep says
> for arch/metag too):
> 
> $ git grep min_low_pfn | grep EXPORT
> arch/ia64/kernel/ia64_ksyms.c:EXPORT_SYMBOL(min_low_pfn);	/* defined
> by bootmem.c, but not exported by generic code */
> arch/metag/kernel/metag_ksyms.c:EXPORT_SYMBOL(min_low_pfn);
> mm/bootmem.c:EXPORT_SYMBOL(min_low_pfn);
> 
> 
> Can you do this in the same branch as the mips change so the bits
> will flow to Linus together in the next window?
> 
> ia64 change (which gmail will no doubt mangle)

It did :-)

> From: "Luck, Tony" <tony.luck@xxxxxxxxx>
> 
> min_low_pfn is now exported by generic code
> 
> Signed-off-by: Tony Luck <tony.luck@xxxxxxxxx>
> 
> ---
> 
> diff --git a/arch/ia64/kernel/ia64_ksyms.c b/arch/ia64/kernel/ia64_ksyms.c
> index 5b7791d..ca95c8b 100644
> --- a/arch/ia64/kernel/ia64_ksyms.c
> +++ b/arch/ia64/kernel/ia64_ksyms.c
> @@ -25,7 +25,6 @@ EXPORT_SYMBOL(copy_page);
> 
>  #ifdef CONFIG_VIRTUAL_MEM_MAP
>  #include <linux/bootmem.h>
> -EXPORT_SYMBOL(min_low_pfn);    /* defined by bootmem.c, but not
> exported by generic code */
>  EXPORT_SYMBOL(max_low_pfn);    /* defined by bootmem.c, but not
> exported by generic code */
>  #endif

I applied this as a merge fix to the mips tree merge for today, but it
should be added to the mips tree.

I also applied this as a merge fix (which only I can apply as the metag
tree is not in Linus' tree yet):

From: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
Date: Wed, 6 Feb 2013 11:31:24 +1100
Subject: [PATCH] metag: min_low_pfn is now exported by generic code

Signed-off-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
---
 arch/metag/kernel/metag_ksyms.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/metag/kernel/metag_ksyms.c b/arch/metag/kernel/metag_ksyms.c
index 3cc4d1d..5381514 100644
--- a/arch/metag/kernel/metag_ksyms.c
+++ b/arch/metag/kernel/metag_ksyms.c
@@ -43,7 +43,6 @@ EXPORT_SYMBOL(pfn_base);
 #ifdef CONFIG_FLATMEM
 /* needed for the pfn_valid macro */
 EXPORT_SYMBOL(max_pfn);
-EXPORT_SYMBOL(min_low_pfn);
 #endif
 
 /* TBI symbols */
-- 
1.8.1

-- 
Cheers,
Stephen Rothwell                    sfr@xxxxxxxxxxxxxxxx

Attachment: pgpn6j6pdZXAK.pgp
Description: PGP signature


[Index of Archives]     [Linux Kernel]     [Linux USB Development]     [Yosemite News]     [Linux SCSI]

  Powered by Linux