Re: [PATCH -tip v7 1/2] irq: Make irqentry text section unconditional

[Masami Hiramatsu]

On Sat, 29 Jul 2017 19:11:23 +0800
kbuild test robot <lkp@xxxxxxxxx> wrote:

> Hi Masami,
> 
> [auto build test ERROR on tip/x86/core]
> [also build test ERROR on v4.13-rc2 next-20170728]
> [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/Masami-Hiramatsu/irq-Make-irqentry-text-section-unconditional/20170729-170710
> config: h8300-h8300h-sim_defconfig (attached as .config)
> compiler: h8300-linux-gcc (GCC) 6.2.0
> reproduce:
>         wget https://raw.githubusercontent.com/01org/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=h8300 
> 
> All errors (new ones prefixed by >>):
> 
>    In file included from arch/h8300/kernel/process.c:45:0:
> >> arch/h8300/include/asm/traps.h:36:21: error: conflicting types for '_etext'
>     extern char _start, _etext;
>                         ^~~~~~
>    In file included from ./arch/h8300/include/generated/asm/sections.h:1:0,
>                     from include/linux/interrupt.h:21,
>                     from arch/h8300/kernel/process.c:38:
>    include/asm-generic/sections.h:33:32: note: previous declaration of '_etext' was here
>     extern char _text[], _stext[], _etext[];
>                                    ^~~~~~

OK, it is like opening a pandra's box, but I would like to fix
arch/h8300/kernel/process.c because include/asm-generic/sections.h
is a generic header and will be used anywhere.

Thanks,

> 
> vim +/_etext +36 arch/h8300/include/asm/traps.h
> 
> d2a5f499 Yoshinori Sato 2015-05-11  35  
> d2a5f499 Yoshinori Sato 2015-05-11 @36  extern char _start, _etext;
> d2a5f499 Yoshinori Sato 2015-05-11  37  #define check_kernel_text(addr) \
> d2a5f499 Yoshinori Sato 2015-05-11  38  	((addr >= (unsigned long)(&_start)) && \
> bf24eec3 Yoshinori Sato 2016-01-15  39  	 (addr <  (unsigned long)(&_etext)) && !(addr & 1))
> d2a5f499 Yoshinori Sato 2015-05-11  40  
> 
> :::::: The code at line 36 was first introduced by commit
> :::::: d2a5f4999f6c211adf30d9788349e13988d6f2a7 h8300: Assembly headers
> 
> :::::: TO: Yoshinori Sato <ysato@xxxxxxxxxxxxxxxxxxxx>
> :::::: CC: Yoshinori Sato <ysato@xxxxxxxxxxxxxxxxxxxx>
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation


-- 
Masami Hiramatsu <mhiramat@xxxxxxxxxx>