xref: /linux-6.15/scripts/Makefile.extrawarn (revision f5982cce)
1b2441318SGreg Kroah-Hartman# SPDX-License-Identifier: GPL-2.0
2a86fe353SMasahiro Yamada# ==========================================================================
3a86fe353SMasahiro Yamada# make W=... settings
4a86fe353SMasahiro Yamada#
5c77d06e7SYann Droneaud# There are four warning groups enabled by W=1, W=2, W=3, and W=e
6c77d06e7SYann Droneaud# They are independent, and can be combined like W=12 or W=123e.
7a86fe353SMasahiro Yamada# ==========================================================================
8a86fe353SMasahiro Yamada
9e88ca243SArnd Bergmann# Default set of warnings, always enabled
10e88ca243SArnd BergmannKBUILD_CFLAGS += -Wall
11e88ca243SArnd BergmannKBUILD_CFLAGS += -Wundef
12e88ca243SArnd BergmannKBUILD_CFLAGS += -Werror=implicit-function-declaration
13e88ca243SArnd BergmannKBUILD_CFLAGS += -Werror=implicit-int
14e88ca243SArnd BergmannKBUILD_CFLAGS += -Werror=return-type
15e88ca243SArnd BergmannKBUILD_CFLAGS += -Werror=strict-prototypes
16e88ca243SArnd BergmannKBUILD_CFLAGS += -Wno-format-security
17e88ca243SArnd BergmannKBUILD_CFLAGS += -Wno-trigraphs
18e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning,frame-address,)
19e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, address-of-packed-member)
200fcb7085SArnd BergmannKBUILD_CFLAGS += -Wmissing-declarations
210fcb7085SArnd BergmannKBUILD_CFLAGS += -Wmissing-prototypes
22e88ca243SArnd Bergmann
23e88ca243SArnd Bergmannifneq ($(CONFIG_FRAME_WARN),0)
24e88ca243SArnd BergmannKBUILD_CFLAGS += -Wframe-larger-than=$(CONFIG_FRAME_WARN)
25e88ca243SArnd Bergmannendif
26e88ca243SArnd Bergmann
27e88ca243SArnd BergmannKBUILD_CPPFLAGS-$(CONFIG_WERROR) += -Werror
28e88ca243SArnd BergmannKBUILD_CPPFLAGS += $(KBUILD_CPPFLAGS-y)
29e88ca243SArnd BergmannKBUILD_CFLAGS-$(CONFIG_CC_NO_ARRAY_BOUNDS) += -Wno-array-bounds
30e88ca243SArnd Bergmann
31e88ca243SArnd Bergmannifdef CONFIG_CC_IS_CLANG
32e88ca243SArnd Bergmann# The kernel builds with '-std=gnu11' so use of GNU extensions is acceptable.
33e88ca243SArnd BergmannKBUILD_CFLAGS += -Wno-gnu
34e88ca243SArnd Bergmannelse
35e88ca243SArnd Bergmann
36e88ca243SArnd Bergmann# gcc inanely warns about local variables called 'main'
37e88ca243SArnd BergmannKBUILD_CFLAGS += -Wno-main
38e88ca243SArnd Bergmannendif
39e88ca243SArnd Bergmann
40e88ca243SArnd Bergmann# These warnings generated too much noise in a regular build.
41e88ca243SArnd Bergmann# Use make W=1 to enable them (see scripts/Makefile.extrawarn)
42e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, unused-but-set-variable)
43e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, unused-const-variable)
44e88ca243SArnd Bergmann
45e88ca243SArnd Bergmann# These result in bogus false positives
46e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, dangling-pointer)
47e88ca243SArnd Bergmann
48e88ca243SArnd Bergmann# Variable Length Arrays (VLAs) should not be used anywhere in the kernel
49e88ca243SArnd BergmannKBUILD_CFLAGS += -Wvla
50e88ca243SArnd Bergmann
51e88ca243SArnd Bergmann# disable pointer signed / unsigned warnings in gcc 4.0
52e88ca243SArnd BergmannKBUILD_CFLAGS += -Wno-pointer-sign
53e88ca243SArnd Bergmann
54e88ca243SArnd Bergmann# In order to make sure new function cast mismatches are not introduced
55e88ca243SArnd Bergmann# in the kernel (to avoid tripping CFI checking), the kernel should be
56e88ca243SArnd Bergmann# globally built with -Wcast-function-type.
57e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-option, -Wcast-function-type)
58e88ca243SArnd Bergmann
59e88ca243SArnd Bergmann# The allocators already balk at large sizes, so silence the compiler
60e88ca243SArnd Bergmann# warnings for bounds checks involving those possible values. While
61e88ca243SArnd Bergmann# -Wno-alloc-size-larger-than would normally be used here, earlier versions
62e88ca243SArnd Bergmann# of gcc (<9.1) weirdly don't handle the option correctly when _other_
63e88ca243SArnd Bergmann# warnings are produced (?!). Using -Walloc-size-larger-than=SIZE_MAX
64e88ca243SArnd Bergmann# doesn't work (as it is documented to), silently resolving to "0" prior to
65e88ca243SArnd Bergmann# version 9.1 (and producing an error more recently). Numeric values larger
66e88ca243SArnd Bergmann# than PTRDIFF_MAX also don't work prior to version 9.1, which are silently
67e88ca243SArnd Bergmann# ignored, continuing to default to PTRDIFF_MAX. So, left with no other
68e88ca243SArnd Bergmann# choice, we must perform a versioned check to disable this warning.
69e88ca243SArnd Bergmann# https://lore.kernel.org/lkml/[email protected]
70e88ca243SArnd BergmannKBUILD_CFLAGS-$(call gcc-min-version, 90100) += -Wno-alloc-size-larger-than
71e88ca243SArnd BergmannKBUILD_CFLAGS += $(KBUILD_CFLAGS-y) $(CONFIG_CC_IMPLICIT_FALLTHROUGH)
72e88ca243SArnd Bergmann
73e88ca243SArnd Bergmann# Prohibit date/time macros, which would make the build non-deterministic
74e88ca243SArnd BergmannKBUILD_CFLAGS += -Werror=date-time
75e88ca243SArnd Bergmann
76e88ca243SArnd Bergmann# enforce correct pointer usage
77e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-option,-Werror=incompatible-pointer-types)
78e88ca243SArnd Bergmann
79e88ca243SArnd Bergmann# Require designated initializers for all marked structures
80e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-option,-Werror=designated-init)
81e88ca243SArnd Bergmann
82e88ca243SArnd Bergmann# Warn if there is an enum types mismatch
83e88ca243SArnd BergmannKBUILD_CFLAGS += $(call cc-option,-Wenum-conversion)
84e88ca243SArnd Bergmann
85*f5982cceSArnd BergmannKBUILD_CFLAGS += -Wextra
86*f5982cceSArnd BergmannKBUILD_CFLAGS += -Wunused
87*f5982cceSArnd Bergmann
8864a91907SMasahiro Yamada#
8964a91907SMasahiro Yamada# W=1 - warnings which may be relevant and do not occur too often
9064a91907SMasahiro Yamada#
91e27128dbSMasahiro Yamadaifneq ($(findstring 1, $(KBUILD_EXTRA_WARN)),)
92a86fe353SMasahiro Yamada
9326030cb9SArnd BergmannKBUILD_CFLAGS += $(call cc-option, -Wrestrict)
9464a91907SMasahiro YamadaKBUILD_CFLAGS += -Wmissing-format-attribute
9564a91907SMasahiro YamadaKBUILD_CFLAGS += -Wold-style-definition
9664a91907SMasahiro YamadaKBUILD_CFLAGS += -Wmissing-include-dirs
9764a91907SMasahiro YamadaKBUILD_CFLAGS += $(call cc-option, -Wunused-but-set-variable)
9864a91907SMasahiro YamadaKBUILD_CFLAGS += $(call cc-option, -Wunused-const-variable)
9964a91907SMasahiro YamadaKBUILD_CFLAGS += $(call cc-option, -Wpacked-not-aligned)
1006d4ab2e9SArnd BergmannKBUILD_CFLAGS += $(call cc-option, -Wformat-overflow)
1016d4ab2e9SArnd BergmannKBUILD_CFLAGS += $(call cc-option, -Wformat-truncation)
10264a91907SMasahiro YamadaKBUILD_CFLAGS += $(call cc-option, -Wstringop-truncation)
103a86fe353SMasahiro Yamada
10480b6093bSMasahiro YamadaKBUILD_CPPFLAGS += -Wundef
1056863f564SMasahiro YamadaKBUILD_CPPFLAGS += -DKBUILD_EXTRA_WARN1
1066863f564SMasahiro Yamada
10726ea6bb1SBehan Websterelse
10826ea6bb1SBehan Webster
10964a91907SMasahiro Yamada# Some diagnostics enabled by default are noisy.
11064a91907SMasahiro Yamada# Suppress them by using -Wno... except for W=1.
1112cd3271bSArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, unused-but-set-variable)
1122cd3271bSArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, unused-const-variable)
11326030cb9SArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, restrict)
1142cd3271bSArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, packed-not-aligned)
1156d4ab2e9SArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, format-overflow)
1166d4ab2e9SArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, format-truncation)
1172cd3271bSArnd BergmannKBUILD_CFLAGS += $(call cc-disable-warning, stringop-truncation)
11864a91907SMasahiro Yamada
119c40845e3SArnd BergmannKBUILD_CFLAGS += -Wno-override-init # alias for -Wno-initializer-overrides in clang
120c40845e3SArnd Bergmann
121076f421dSMasahiro Yamadaifdef CONFIG_CC_IS_CLANG
122b0839b28SNick Desaulniers# Clang before clang-16 would warn on default argument promotions.
12388b61e3bSNick Desaulniersifneq ($(call clang-min-version, 160000),y)
124b0839b28SNick Desaulniers# Disable -Wformat
12521f9c8a1SLinus TorvaldsKBUILD_CFLAGS += -Wno-format
126b0839b28SNick Desaulniers# Then re-enable flags that were part of the -Wformat group that aren't
127b0839b28SNick Desaulniers# problematic.
128b0839b28SNick DesaulniersKBUILD_CFLAGS += -Wformat-extra-args -Wformat-invalid-specifier
129b0839b28SNick DesaulniersKBUILD_CFLAGS += -Wformat-zero-length -Wnonnull
130b0839b28SNick Desaulniers# Requires clang-12+.
13188b61e3bSNick Desaulniersifeq ($(call clang-min-version, 120000),y)
132b0839b28SNick DesaulniersKBUILD_CFLAGS += -Wformat-insufficient-args
133b0839b28SNick Desaulniersendif
134b0839b28SNick Desaulniersendif
13582f2bc2fSNathan ChancellorKBUILD_CFLAGS += $(call cc-disable-warning, pointer-to-enum-cast)
136afe956c5SNathan ChancellorKBUILD_CFLAGS += -Wno-tautological-constant-out-of-range-compare
1371cf5f151SNathan ChancellorKBUILD_CFLAGS += $(call cc-disable-warning, unaligned-access)
13821206351SSami TolvanenKBUILD_CFLAGS += $(call cc-disable-warning, cast-function-type-strict)
13975b5ab13SNathan ChancellorKBUILD_CFLAGS += -Wno-enum-compare-conditional
14075b5ab13SNathan ChancellorKBUILD_CFLAGS += -Wno-enum-enum-conversion
14126ea6bb1SBehan Websterendif
14264a91907SMasahiro Yamada
14364a91907SMasahiro Yamadaendif
14464a91907SMasahiro Yamada
14564a91907SMasahiro Yamada#
14664a91907SMasahiro Yamada# W=2 - warnings which occur quite often but may still be relevant
14764a91907SMasahiro Yamada#
148e27128dbSMasahiro Yamadaifneq ($(findstring 2, $(KBUILD_EXTRA_WARN)),)
14964a91907SMasahiro Yamada
15064a91907SMasahiro YamadaKBUILD_CFLAGS += -Wdisabled-optimization
15164a91907SMasahiro YamadaKBUILD_CFLAGS += -Wshadow
15264a91907SMasahiro YamadaKBUILD_CFLAGS += $(call cc-option, -Wlogical-op)
15364a91907SMasahiro YamadaKBUILD_CFLAGS += -Wmissing-field-initializers
154355a3587SRikard FalkebornKBUILD_CFLAGS += -Wtype-limits
15564a91907SMasahiro YamadaKBUILD_CFLAGS += $(call cc-option, -Wmaybe-uninitialized)
15664a91907SMasahiro YamadaKBUILD_CFLAGS += $(call cc-option, -Wunused-macros)
15764a91907SMasahiro Yamada
1586863f564SMasahiro YamadaKBUILD_CPPFLAGS += -DKBUILD_EXTRA_WARN2
1596863f564SMasahiro Yamada
1602cd3271bSArnd Bergmannelse
1612cd3271bSArnd Bergmann
1622cd3271bSArnd Bergmann# The following turn off the warnings enabled by -Wextra
1632cd3271bSArnd BergmannKBUILD_CFLAGS += -Wno-missing-field-initializers
1642cd3271bSArnd BergmannKBUILD_CFLAGS += -Wno-type-limits
1652cd3271bSArnd BergmannKBUILD_CFLAGS += -Wno-shift-negative-value
1662cd3271bSArnd Bergmann
167c40845e3SArnd Bergmannifdef CONFIG_CC_IS_GCC
1682cd3271bSArnd BergmannKBUILD_CFLAGS += -Wno-maybe-uninitialized
1692cd3271bSArnd Bergmannendif
1702cd3271bSArnd Bergmann
17164a91907SMasahiro Yamadaendif
17264a91907SMasahiro Yamada
17364a91907SMasahiro Yamada#
17464a91907SMasahiro Yamada# W=3 - more obscure warnings, can most likely be ignored
17564a91907SMasahiro Yamada#
176e27128dbSMasahiro Yamadaifneq ($(findstring 3, $(KBUILD_EXTRA_WARN)),)
17764a91907SMasahiro Yamada
17864a91907SMasahiro YamadaKBUILD_CFLAGS += -Wbad-function-cast
179095fbca0SArnd BergmannKBUILD_CFLAGS += -Wcast-align
18064a91907SMasahiro YamadaKBUILD_CFLAGS += -Wcast-qual
18164a91907SMasahiro YamadaKBUILD_CFLAGS += -Wconversion
18264a91907SMasahiro YamadaKBUILD_CFLAGS += -Wpacked
18364a91907SMasahiro YamadaKBUILD_CFLAGS += -Wpadded
18464a91907SMasahiro YamadaKBUILD_CFLAGS += -Wpointer-arith
18564a91907SMasahiro YamadaKBUILD_CFLAGS += -Wredundant-decls
186a97ea93eSJoe PerchesKBUILD_CFLAGS += -Wsign-compare
18764a91907SMasahiro YamadaKBUILD_CFLAGS += -Wswitch-default
18864a91907SMasahiro Yamada
1896863f564SMasahiro YamadaKBUILD_CPPFLAGS += -DKBUILD_EXTRA_WARN3
1906863f564SMasahiro Yamada
1912cd3271bSArnd Bergmannelse
1922cd3271bSArnd Bergmann
1932cd3271bSArnd Bergmann# The following turn off the warnings enabled by -Wextra
1942cd3271bSArnd BergmannKBUILD_CFLAGS += -Wno-sign-compare
195*f5982cceSArnd BergmannKBUILD_CFLAGS += -Wno-unused-parameter
1962cd3271bSArnd Bergmann
197a86fe353SMasahiro Yamadaendif
198c77d06e7SYann Droneaud
199c77d06e7SYann Droneaud#
200c77d06e7SYann Droneaud# W=e - error out on warnings
201c77d06e7SYann Droneaud#
202c77d06e7SYann Droneaudifneq ($(findstring e, $(KBUILD_EXTRA_WARN)),)
203c77d06e7SYann Droneaud
204c77d06e7SYann DroneaudKBUILD_CFLAGS += -Werror
205c77d06e7SYann Droneaud
206c77d06e7SYann Droneaudendif
207