Re: Kbuild: CC= handling broken for tools/

2016-11-11 Thread Josh Poimboeuf
On Fri, Nov 11, 2016 at 06:08:24PM +0100, Jiri Slaby wrote: > On 11/11/2016, 04:30 PM, Josh Poimboeuf wrote: > > Does it work if you use KCFLAGS instead of EXTRA_CFLAGS? > > Yes, the kernel builds with the flags now. > > But objtool is unexpectedly still built by a different compiler. > > I woul

Re: Kbuild: CC= handling broken for tools/

2016-11-11 Thread Jiri Slaby
On 11/11/2016, 04:30 PM, Josh Poimboeuf wrote: > Does it work if you use KCFLAGS instead of EXTRA_CFLAGS? Yes, the kernel builds with the flags now. But objtool is unexpectedly still built by a different compiler. I would also prefer the flags (like the warning above) to be passed even when buil

Re: Kbuild: CC= handling broken for tools/

2016-11-11 Thread Josh Poimboeuf
On Fri, Nov 11, 2016 at 04:14:27PM +0100, Jiri Olsa wrote: > On Fri, Nov 11, 2016 at 03:04:35PM +0100, Jiri Slaby wrote: > > On 11/11/2016, 03:00 PM, Jiri Olsa wrote: > > > On Fri, Nov 11, 2016 at 12:23:25PM +0100, Jiri Slaby wrote: > > >> Hi, > > >> > > >> I am trying a new gcc with new warnings e

Re: Kbuild: CC= handling broken for tools/

2016-11-11 Thread Jiri Olsa
On Fri, Nov 11, 2016 at 03:04:35PM +0100, Jiri Slaby wrote: > On 11/11/2016, 03:00 PM, Jiri Olsa wrote: > > On Fri, Nov 11, 2016 at 12:23:25PM +0100, Jiri Slaby wrote: > >> Hi, > >> > >> I am trying a new gcc with new warnings enabled: > >> > >> make O=../a/gcc7/ CC='gcc-7' V=1 kernel/exit.o > >> E

Re: Kbuild: CC= handling broken for tools/

2016-11-11 Thread Jiri Slaby
On 11/11/2016, 03:00 PM, Jiri Olsa wrote: > On Fri, Nov 11, 2016 at 12:23:25PM +0100, Jiri Slaby wrote: >> Hi, >> >> I am trying a new gcc with new warnings enabled: >> >> make O=../a/gcc7/ CC='gcc-7' V=1 kernel/exit.o >> EXTRA_CFLAGS='-Wimplicit-fallthrough=3' >> >> But the build fails when buildi

Re: Kbuild: CC= handling broken for tools/

2016-11-11 Thread Jiri Olsa
On Fri, Nov 11, 2016 at 12:23:25PM +0100, Jiri Slaby wrote: > Hi, > > I am trying a new gcc with new warnings enabled: > > make O=../a/gcc7/ CC='gcc-7' V=1 kernel/exit.o > EXTRA_CFLAGS='-Wimplicit-fallthrough=3' > > But the build fails when building under tools/: > ... > make -f /home/latest/lin

Re: Kbuild: CC= handling broken for tools/

2016-11-11 Thread Michal Marek
On 2016-11-11 12:23, Jiri Slaby wrote: > Hi, > > I am trying a new gcc with new warnings enabled: > > make O=../a/gcc7/ CC='gcc-7' V=1 kernel/exit.o > EXTRA_CFLAGS='-Wimplicit-fallthrough=3' Note that EXTRA_CFLAGS is a makefile variable read by both kbuild and tools/build apparently. > > But t

Kbuild: CC= handling broken for tools/

2016-11-11 Thread Jiri Slaby
Hi, I am trying a new gcc with new warnings enabled: make O=../a/gcc7/ CC='gcc-7' V=1 kernel/exit.o EXTRA_CFLAGS='-Wimplicit-fallthrough=3' But the build fails when building under tools/: ... make -f /home/latest/linux/tools/build/Makefile.build dir=. obj=fixdep gcc -Wp,-MD,/home/latest/a/gcc7