Re: [PATCH] Enable '-Werror' by default for all kernel builds

From: Guenter Roeck
Date: Mon Sep 06 2021 - 10:26:20 EST


On Sun, Sep 05, 2021 at 11:24:05AM -0700, Linus Torvalds wrote:
> ... but make it a config option so that broken environments can disable
> it when required.
>
> We really should always have a clean build, and will disable specific
> over-eager warnings as required, if we can't fix them. But while I
> fairly religiously enforce that in my own tree, it doesn't get enforced
> by various build robots that don't necessarily report warnings.
>
> So this just makes '-Werror' a default compiler flag, but allows people
> to disable it for their configuration if they have some particular
> issues.
>
> Occasionally, new compiler versions end up enabling new warnings, and it
> can take a while before we have them fixed (or the warnings disabled if
> that is what it takes), so the config option allows for that situation.
>
> Hopefully this will mean that I get fewer pull requests that have new
> warnings that were not noticed by various automation we have in place.
>
> Knock wood.
>

I guess the good news is that some builds still pass.

Build results:
total: 153 pass: 89 fail: 64
Failed builds:
alpha:defconfig
alpha:allmodconfig
arcv2:defconfig
arcv2:axs103_defconfig
arcv2:vdk_hs38_smp_defconfig
arm:s3c2410_defconfig
arm:ixp4xx_defconfig
arm:omap1_defconfig
arm:footbridge_defconfig
arm:keystone_defconfig
arm:vexpress_defconfig
arm:imx_v4_v5_defconfig
arm:s3c6400_defconfig
arm:s5pv210_defconfig
arm:integrator_defconfig
arm:pxa910_defconfig
arm:clps711x_defconfig
csky:defconfig
h8300:edosk2674_defconfig
h8300:h8300h-sim_defconfig
h8300:h8s-sim_defconfig
hexagon:defconfig
i386:allyesconfig
i386:allmodconfig
ia64:defconfig
m68k:defconfig
m68k:allmodconfig
m68k:sun3_defconfig
m68k_nommu:m5272c3_defconfig
m68k_nommu:m5307c3_defconfig
m68k_nommu:m5249evb_defconfig
m68k_nommu:m5407c3_defconfig
m68k_nommu:m5475evb_defconfig
microblaze:mmu_defconfig
mips:allmodconfig
mips:bcm63xx_defconfig
mips:e55_defconfig
mips:malta_defconfig
nds32:defconfig
nds32:allmodconfig
nios2:3c120_defconfig
parisc:allmodconfig
parisc:generic-32bit_defconfig
parisc64:generic-64bit_defconfig
powerpc:allmodconfig
powerpc:cell_defconfig
powerpc:maple_defconfig
powerpc:ppc6xx_defconfig
powerpc:mpc83xx_defconfig
powerpc:tqm8xx_defconfig
powerpc:83xx/mpc834x_mds_defconfig
riscv32:allmodconfig
riscv:allmodconfig
s390:allmodconfig
sh:defconfig
sh:dreamcast_defconfig
sh:microdev_defconfig
sh:shx3_defconfig
sparc32:defconfig
sparc64:allmodconfig
sparc64:defconfig
um:defconfig
xtensa:defconfig
xtensa:allmodconfig
Qemu test results:
total: 479 pass: 340 fail: 139
Failed tests:
<many>

Guenter