Re: [PATCH] Raise the minimum required gcc version to 4.6

[Geert Uytterhoeven]

Hi Joe,

On Mon, Aug 20, 2018 at 10:15 PM Joe Perches <joe@xxxxxxxxxxx> wrote:
> Various architectures fail to build properly with older versions
> of the gcc compiler.
>
> An example from Guenter Roeck via this thread:
> https://lore.kernel.org/lkml/20180814170904.GA12768@xxxxxxxxxxxx/
>
> ----------------------
> In file included from ./include/linux/mm.h:17:0,
>                  from ./include/linux/pid_namespace.h:7,
>                  from ./include/linux/ptrace.h:10,
>                  from arch/openrisc/kernel/asm-offsets.c:32:
> ./include/linux/mm_types.h:497:16: error: flexible array member in otherwise empty struct
>
> This is just an example with gcc 4.5.1 for or32. I have seen the problem
> with gcc 4.4 (for unicore32) as well.
> ----------------------
>
> So update the minimum required version of gcc to 4.6.
>
> Miscellanea:
>
> o Update Documentation/process/changes.rst
> o Remove and consolidate version test blocks in compiler-gcc.h
>   for versions lower than 4.6
>
> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
> Reviewed-by: Kees Cook <keescook@xxxxxxxxxxxx>
> Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>

Thanks for your patch!

I'm willing to upgrade. But there's one thing that's holding me back.

Does gcc 8.x gives again the same warnings as my venerable old gcc 4.1.2,
that no one else seems to see? Or will the real bugs I detect this way stay
unfixed? Polyculture is a good thing, also in compilers.

Reverted locally (incl. the follow-up), applied Andrew's fix, detected new
warnings in v4.18+, and sent patches where it makes sense...

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds