]> nv-tegra.nvidia Code Review - linux-2.6.git/commitdiff
x86: fix section mismatch warning - spp_getpage()
authorMarcin Slusarz <marcin.slusarz@gmail.com>
Fri, 15 Aug 2008 16:32:24 +0000 (18:32 +0200)
committerIngo Molnar <mingo@elte.hu>
Fri, 15 Aug 2008 17:16:06 +0000 (19:16 +0200)
WARNING: vmlinux.o(.text+0x17a3e): Section mismatch in reference from the function set_pte_vaddr_pud() to the function .init.text:spp_getpage()
The function set_pte_vaddr_pud() references
the function __init spp_getpage().
This is often because set_pte_vaddr_pud lacks a __init
annotation or the annotation of spp_getpage is wrong.

spp_getpage is called from __init (__init_extra_mapping) and
non __init (set_pte_vaddr_pud) functions, so it can't be __init.
Unfortunately it calls alloc_bootmem_pages which is __init,
but does it only when bootmem allocator is available (after_bootmem == 0).

So annotate it accordingly.

Signed-off-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Cc: H. Peter Anvin <hpa@zytor.com>
arch/x86/mm/init_64.c

index b3e6c3075accd63958afe3f1ec1ec6b681a82a26..a87ea0e4b3dca3e7149aa03321af508cc1216b0d 100644 (file)
@@ -88,7 +88,11 @@ early_param("gbpages", parse_direct_gbpages_on);
 
 int after_bootmem;
 
 
 int after_bootmem;
 
-static __init void *spp_getpage(void)
+/*
+ * NOTE: This function is marked __ref because it calls __init function
+ * (alloc_bootmem_pages). It's safe to do it ONLY when after_bootmem == 0.
+ */
+static __ref void *spp_getpage(void)
 {
        void *ptr;
 
 {
        void *ptr;