Re: [PATCH 1/1] mm/page_owner: ignore everything below the IRQ entry point

From: kbuild test robot
Date: Tue Dec 05 2017 - 00:40:22 EST


Hi Maninder,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on mmotm/master]
[also build test ERROR on v4.15-rc2 next-20171204]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Maninder-Singh/mm-page_owner-ignore-everything-below-the-IRQ-entry-point/20171205-122901
base: git://git.cmpxchg.org/linux-mmotm.git master
config: blackfin-allmodconfig (attached as .config)
compiler: bfin-uclinux-gcc (GCC) 6.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=blackfin

All errors (new ones prefixed by >>):

In file included from include/linux/interrupt.h:22:0,
from include/linux/kernel_stat.h:9,
from arch/blackfin/kernel/asm-offsets.c:11:
>> arch/blackfin/include/asm/sections.h:35:19: error: redefinition of 'arch_is_kernel_text'
static inline int arch_is_kernel_text(unsigned long addr)
^~~~~~~~~~~~~~~~~~~
In file included from include/linux/stacktrace.h:6:0,
from include/linux/lockdep.h:29,
from include/linux/rcupdate.h:42,
from include/linux/rculist.h:11,
from include/linux/pid.h:5,
from include/linux/sched.h:14,
from arch/blackfin/kernel/asm-offsets.c:10:
include/asm-generic/sections.h:63:19: note: previous definition of 'arch_is_kernel_text' was here
static inline int arch_is_kernel_text(unsigned long addr)
^~~~~~~~~~~~~~~~~~~
In file included from include/linux/interrupt.h:22:0,
from include/linux/kernel_stat.h:9,
from arch/blackfin/kernel/asm-offsets.c:11:
>> arch/blackfin/include/asm/sections.h:48:19: error: redefinition of 'arch_is_kernel_data'
static inline int arch_is_kernel_data(unsigned long addr)
^~~~~~~~~~~~~~~~~~~
In file included from include/linux/stacktrace.h:6:0,
from include/linux/lockdep.h:29,
from include/linux/rcupdate.h:42,
from include/linux/rculist.h:11,
from include/linux/pid.h:5,
from include/linux/sched.h:14,
from arch/blackfin/kernel/asm-offsets.c:10:
include/asm-generic/sections.h:70:19: note: previous definition of 'arch_is_kernel_data' was here
static inline int arch_is_kernel_data(unsigned long addr)
^~~~~~~~~~~~~~~~~~~
make[2]: *** [arch/blackfin/kernel/asm-offsets.s] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [prepare0] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [sub-make] Error 2

vim +/arch_is_kernel_text +35 arch/blackfin/include/asm/sections.h

e56770fb Mike Frysinger 2009-09-22 33
e56770fb Mike Frysinger 2009-09-22 34 /* Blackfin systems have discontinuous memory map and no virtualized memory */
e56770fb Mike Frysinger 2009-09-22 @35 static inline int arch_is_kernel_text(unsigned long addr)
e56770fb Mike Frysinger 2009-09-22 36 {
e56770fb Mike Frysinger 2009-09-22 37 return
e56770fb Mike Frysinger 2009-09-22 38 (L1_CODE_LENGTH &&
e56770fb Mike Frysinger 2009-09-22 39 addr >= (unsigned long)_stext_l1 &&
e56770fb Mike Frysinger 2009-09-22 40 addr < (unsigned long)_etext_l1)
e56770fb Mike Frysinger 2009-09-22 41 ||
e56770fb Mike Frysinger 2009-09-22 42 (L2_LENGTH &&
e56770fb Mike Frysinger 2009-09-22 43 addr >= (unsigned long)_stext_l2 &&
e56770fb Mike Frysinger 2009-09-22 44 addr < (unsigned long)_etext_l2);
e56770fb Mike Frysinger 2009-09-22 45 }
e56770fb Mike Frysinger 2009-09-22 46 #define arch_is_kernel_text(addr) arch_is_kernel_text(addr)
e56770fb Mike Frysinger 2009-09-22 47
e56770fb Mike Frysinger 2009-09-22 @48 static inline int arch_is_kernel_data(unsigned long addr)
e56770fb Mike Frysinger 2009-09-22 49 {
e56770fb Mike Frysinger 2009-09-22 50 return
e56770fb Mike Frysinger 2009-09-22 51 (L1_DATA_A_LENGTH &&
e56770fb Mike Frysinger 2009-09-22 52 addr >= (unsigned long)_sdata_l1 &&
e56770fb Mike Frysinger 2009-09-22 53 addr < (unsigned long)_ebss_l1)
e56770fb Mike Frysinger 2009-09-22 54 ||
e56770fb Mike Frysinger 2009-09-22 55 (L1_DATA_B_LENGTH &&
e56770fb Mike Frysinger 2009-09-22 56 addr >= (unsigned long)_sdata_b_l1 &&
e56770fb Mike Frysinger 2009-09-22 57 addr < (unsigned long)_ebss_b_l1)
e56770fb Mike Frysinger 2009-09-22 58 ||
e56770fb Mike Frysinger 2009-09-22 59 (L2_LENGTH &&
e56770fb Mike Frysinger 2009-09-22 60 addr >= (unsigned long)_sdata_l2 &&
e56770fb Mike Frysinger 2009-09-22 61 addr < (unsigned long)_ebss_l2);
e56770fb Mike Frysinger 2009-09-22 62 }
e56770fb Mike Frysinger 2009-09-22 63 #define arch_is_kernel_data(addr) arch_is_kernel_data(addr)
e56770fb Mike Frysinger 2009-09-22 64

:::::: The code at line 35 was first introduced by commit
:::::: e56770fbc48c1517f620f9f68e3f728e74d52bf5 Blackfin: override text/data checking functions

:::::: TO: Mike Frysinger <vapier@xxxxxxxxxx>
:::::: CC: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: application/gzip