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

From: Guenter Roeck <linux@roeck-us.net>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Enable '-Werror' by default for all kernel builds
Date: Mon, 6 Sep 2021 07:26:15 -0700	[thread overview]
Message-ID: <20210906142615.GA1917503@roeck-us.net> (raw)

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