Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-13 Thread Michael Ellerman
On Mon, 2016-06-13 at 13:11 -0700, Kees Cook wrote: > On Mon, Jun 13, 2016 at 11:32 AM, Austin S. Hemmelgarn > wrote: > > On 2016-06-12 20:18, Emese Revfy wrote: > > > > > > On Sun, 12 Jun 2016 15:25:39 -0700 > > > Kees Cook wrote: > > > > > > > I don't like this because it means if someone spe

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-13 Thread Kees Cook
On Mon, Jun 13, 2016 at 1:40 AM, Sedat Dilek wrote: > On Sat, Jun 11, 2016 at 6:12 PM, Kees Cook wrote: >> Since adding the gcc plugin development headers is required for the >> gcc plugin support, we should ease into this new kernel build dependency >> more slowly. For now, disable the gcc plugi

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-13 Thread Kees Cook
On Mon, Jun 13, 2016 at 11:32 AM, Austin S. Hemmelgarn wrote: > On 2016-06-12 20:18, Emese Revfy wrote: >> >> On Sun, 12 Jun 2016 15:25:39 -0700 >> Kees Cook wrote: >> >>> I don't like this because it means if someone specifically selects >>> some plugins in their .config, and the headers are mis

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-13 Thread Austin S. Hemmelgarn
On 2016-06-12 20:18, Emese Revfy wrote: On Sun, 12 Jun 2016 15:25:39 -0700 Kees Cook wrote: I don't like this because it means if someone specifically selects some plugins in their .config, and the headers are missing, the kernel will successfully compile. For many plugins, this results in a k

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-13 Thread Sedat Dilek
On Sat, Jun 11, 2016 at 6:12 PM, Kees Cook wrote: > Since adding the gcc plugin development headers is required for the > gcc plugin support, we should ease into this new kernel build dependency > more slowly. For now, disable the gcc plugins under COMPILE_TEST so that > all*config builds will ski

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-13 Thread Sudip Mukherjee
On Sat, Jun 11, 2016 at 09:12:12AM -0700, Kees Cook wrote: > Since adding the gcc plugin development headers is required for the > gcc plugin support, we should ease into this new kernel build dependency > more slowly. For now, disable the gcc plugins under COMPILE_TEST so that > all*config builds

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-12 Thread Emese Revfy
On Sun, 12 Jun 2016 15:25:39 -0700 Kees Cook wrote: > I don't like this because it means if someone specifically selects > some plugins in their .config, and the headers are missing, the kernel > will successfully compile. For many plugins, this results in a kernel > that lacks the requested secu

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-12 Thread Kees Cook
On Sun, Jun 12, 2016 at 3:12 PM, Emese Revfy wrote: > On Sat, 11 Jun 2016 12:29:26 -0400 > Paul Gortmaker wrote: > >> [[PATCH] gcc-plugins: disable under COMPILE_TEST] On 11/06/2016 (Sat 09:12) >> Kees Cook wrote: >> >> > Since adding the gcc plugin dev

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-12 Thread Emese Revfy
On Sat, 11 Jun 2016 12:29:26 -0400 Paul Gortmaker wrote: > [[PATCH] gcc-plugins: disable under COMPILE_TEST] On 11/06/2016 (Sat 09:12) > Kees Cook wrote: > > > Since adding the gcc plugin development headers is required for the > > gcc plugin support, we should ease into

Re: [PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-11 Thread Paul Gortmaker
[[PATCH] gcc-plugins: disable under COMPILE_TEST] On 11/06/2016 (Sat 09:12) Kees Cook wrote: > Since adding the gcc plugin development headers is required for the > gcc plugin support, we should ease into this new kernel build dependency > more slowly. For now, disable the gcc plug

[PATCH] gcc-plugins: disable under COMPILE_TEST

2016-06-11 Thread Kees Cook
Since adding the gcc plugin development headers is required for the gcc plugin support, we should ease into this new kernel build dependency more slowly. For now, disable the gcc plugins under COMPILE_TEST so that all*config builds will skip it. Signed-off-by: Kees Cook --- arch/Kconfig | 1