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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20170730120315.26288a2507b2adb0e0e043d1@kernel.org>
Date:   Sun, 30 Jul 2017 12:03:15 +0900
From:   Masami Hiramatsu <mhiramat@...nel.org>
To:     kbuild test robot <lkp@...el.com>,
        Yoshinori Sato <ysato@...rs.sourceforge.jp>
Cc:     kbuild-all@...org, Ingo Molnar <mingo@...nel.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Francis Deslauriers <francis.deslauriers@...icios.com>,
        mathieu.desnoyers@...icios.com, Ingo Molnar <mingo@...hat.com>,
        "H . Peter Anvin" <hpa@...or.com>, x86@...nel.org,
        Ananth N Mavinakayanahalli <ananth@...ibm.com>,
        Anil S Keshavamurthy <anil.s.keshavamurthy@...el.com>,
        "David S . Miller" <davem@...emloft.net>,
        linux-kernel@...r.kernel.org, uclinux-h8-devel@...ts.sourceforge.jp
Subject: Re: [PATCH -tip v7 1/2] irq: Make irqentry text section
 unconditional

On Sat, 29 Jul 2017 19:11:23 +0800
kbuild test robot <lkp@...el.com> 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@...rs.sourceforge.jp>
> :::::: CC: Yoshinori Sato <ysato@...rs.sourceforge.jp>
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation


-- 
Masami Hiramatsu <mhiramat@...nel.org>