* [linux-next:master 6243/6567] WARNING: vmlinux.o(.text.unlikely+0x5fb7): Section mismatch in reference from the function __def_free() to the function .init.text:__free_pages_boot_core()
@ 2017-10-18 10:41 kbuild test robot
2017-10-18 21:40 ` Andrew Morton
0 siblings, 1 reply; 3+ messages in thread
From: kbuild test robot @ 2017-10-18 10:41 UTC (permalink / raw)
To: Pavel Tatashin
Cc: kbuild-all, Mark Brown, Steven Sistare, Daniel Jordan, Bob Picco,
Andrew Morton, Linux Memory Management List
[-- Attachment #1: Type: text/plain, Size: 1058 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: a7dd40274d75326ca868479c62090b1198a357ad
commit: 430676b385fb341d5a33950bae284d0df2e70117 [6243/6567] mm: deferred_init_memmap improvements
config: x86_64-randconfig-it0-10181522 (attached as .config)
compiler: gcc-4.9 (Debian 4.9.4-2) 4.9.4
reproduce:
git checkout 430676b385fb341d5a33950bae284d0df2e70117
# save the attached .config to linux build tree
make ARCH=x86_64
All warnings (new ones prefixed by >>):
>> WARNING: vmlinux.o(.text.unlikely+0x5fb7): Section mismatch in reference from the function __def_free() to the function .init.text:__free_pages_boot_core()
The function __def_free() references
the function __init __free_pages_boot_core().
This is often because __def_free lacks a __init
annotation or the annotation of __free_pages_boot_core is wrong.
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 23652 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [linux-next:master 6243/6567] WARNING: vmlinux.o(.text.unlikely+0x5fb7): Section mismatch in reference from the function __def_free() to the function .init.text:__free_pages_boot_core()
2017-10-18 10:41 [linux-next:master 6243/6567] WARNING: vmlinux.o(.text.unlikely+0x5fb7): Section mismatch in reference from the function __def_free() to the function .init.text:__free_pages_boot_core() kbuild test robot
@ 2017-10-18 21:40 ` Andrew Morton
2017-10-19 14:22 ` Pavel Tatashin
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Morton @ 2017-10-18 21:40 UTC (permalink / raw)
To: kbuild test robot
Cc: Pavel Tatashin, kbuild-all, Mark Brown, Steven Sistare,
Daniel Jordan, Bob Picco, Linux Memory Management List
On Wed, 18 Oct 2017 18:41:44 +0800 kbuild test robot <fengguang.wu@intel.com> wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head: a7dd40274d75326ca868479c62090b1198a357ad
> commit: 430676b385fb341d5a33950bae284d0df2e70117 [6243/6567] mm: deferred_init_memmap improvements
> config: x86_64-randconfig-it0-10181522 (attached as .config)
> compiler: gcc-4.9 (Debian 4.9.4-2) 4.9.4
> reproduce:
> git checkout 430676b385fb341d5a33950bae284d0df2e70117
> # save the attached .config to linux build tree
> make ARCH=x86_64
>
> All warnings (new ones prefixed by >>):
>
> >> WARNING: vmlinux.o(.text.unlikely+0x5fb7): Section mismatch in reference from the function __def_free() to the function .init.text:__free_pages_boot_core()
> The function __def_free() references
> the function __init __free_pages_boot_core().
> This is often because __def_free lacks a __init
> annotation or the annotation of __free_pages_boot_core is wrong.
This?
--- a/mm/page_alloc.c~mm-deferred_init_memmap-improvements-fix
+++ a/mm/page_alloc.c
@@ -1448,7 +1448,7 @@ static inline void __init pgdat_init_rep
* Helper for deferred_init_range, free the given range, reset the counters, and
* return number of pages freed.
*/
-static inline unsigned long __def_free(unsigned long *nr_free,
+static unsigned long __init __def_free(unsigned long *nr_free,
unsigned long *free_base_pfn,
struct page **page)
{
@@ -1462,8 +1462,8 @@ static inline unsigned long __def_free(u
return nr;
}
-static unsigned long deferred_init_range(int nid, int zid, unsigned long pfn,
- unsigned long end_pfn)
+static unsigned long __init deferred_init_range(int nid, int zid,
+ unsigned long pfn, unsigned long end_pfn)
{
struct mminit_pfnnid_cache nid_init_state = { };
unsigned long nr_pgmask = pageblock_nr_pages - 1;
_
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [linux-next:master 6243/6567] WARNING: vmlinux.o(.text.unlikely+0x5fb7): Section mismatch in reference from the function __def_free() to the function .init.text:__free_pages_boot_core()
2017-10-18 21:40 ` Andrew Morton
@ 2017-10-19 14:22 ` Pavel Tatashin
0 siblings, 0 replies; 3+ messages in thread
From: Pavel Tatashin @ 2017-10-19 14:22 UTC (permalink / raw)
To: Andrew Morton
Cc: kbuild test robot, kbuild-all, Mark Brown, Steven Sistare,
Daniel Jordan, Bob Picco, Linux Memory Management List
Hi Andrew,
Yes, we need __init for both: deferred_init_range() and __def_free().
Thank you,
Pavel
On Wed, Oct 18, 2017 at 5:40 PM, Andrew Morton
<akpm@linux-foundation.org> wrote:
> On Wed, 18 Oct 2017 18:41:44 +0800 kbuild test robot <fengguang.wu@intel.com> wrote:
>
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
>> head: a7dd40274d75326ca868479c62090b1198a357ad
>> commit: 430676b385fb341d5a33950bae284d0df2e70117 [6243/6567] mm: deferred_init_memmap improvements
>> config: x86_64-randconfig-it0-10181522 (attached as .config)
>> compiler: gcc-4.9 (Debian 4.9.4-2) 4.9.4
>> reproduce:
>> git checkout 430676b385fb341d5a33950bae284d0df2e70117
>> # save the attached .config to linux build tree
>> make ARCH=x86_64
>>
>> All warnings (new ones prefixed by >>):
>>
>> >> WARNING: vmlinux.o(.text.unlikely+0x5fb7): Section mismatch in reference from the function __def_free() to the function .init.text:__free_pages_boot_core()
>> The function __def_free() references
>> the function __init __free_pages_boot_core().
>> This is often because __def_free lacks a __init
>> annotation or the annotation of __free_pages_boot_core is wrong.
>
> This?
>
> --- a/mm/page_alloc.c~mm-deferred_init_memmap-improvements-fix
> +++ a/mm/page_alloc.c
> @@ -1448,7 +1448,7 @@ static inline void __init pgdat_init_rep
> * Helper for deferred_init_range, free the given range, reset the counters, and
> * return number of pages freed.
> */
> -static inline unsigned long __def_free(unsigned long *nr_free,
> +static unsigned long __init __def_free(unsigned long *nr_free,
> unsigned long *free_base_pfn,
> struct page **page)
> {
> @@ -1462,8 +1462,8 @@ static inline unsigned long __def_free(u
> return nr;
> }
>
> -static unsigned long deferred_init_range(int nid, int zid, unsigned long pfn,
> - unsigned long end_pfn)
> +static unsigned long __init deferred_init_range(int nid, int zid,
> + unsigned long pfn, unsigned long end_pfn)
> {
> struct mminit_pfnnid_cache nid_init_state = { };
> unsigned long nr_pgmask = pageblock_nr_pages - 1;
> _
>
> --
> To unsubscribe, send a message with 'unsubscribe linux-mm' in
> the body to majordomo@kvack.org. For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-10-19 14:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-18 10:41 [linux-next:master 6243/6567] WARNING: vmlinux.o(.text.unlikely+0x5fb7): Section mismatch in reference from the function __def_free() to the function .init.text:__free_pages_boot_core() kbuild test robot
2017-10-18 21:40 ` Andrew Morton
2017-10-19 14:22 ` Pavel Tatashin
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.