| Message ID | 20221025073023.16137-1-xin3.li@intel.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | kbuild: upgrade the orphan section warning to an error if CONFIG_WERROR is set | expand |
On Tue, 25 Oct 2022 00:30:22 -0700, Xin Li wrote: > Andrew Cooper suggested upgrading the orphan section warning to a hard link > error. However Nathan Chancellor said outright turning the warning into an > error with no escape hatch might be too aggressive, as we have had these > warnings triggered by new compiler generated sections, and suggested turning > orphan sections into an error only if CONFIG_WERROR is set. Kees Cook echoed > and emphasized that the mandate from Linus is that we should avoid breaking > builds. It wrecks bisection, it causes problems across compiler versions, etc. > > [...] Applied to for-next/hardening, thanks! [1/1] kbuild: upgrade the orphan section warning to an error if CONFIG_WERROR is set https://git.kernel.org/kees/c/450301ceb4e6
Andrew Cooper suggested upgrading the orphan section warning to a hard link error. However Nathan Chancellor said outright turning the warning into an error with no escape hatch might be too aggressive, as we have had these warnings triggered by new compiler generated sections, and suggested turning orphan sections into an error only if CONFIG_WERROR is set. Kees Cook echoed and emphasized that the mandate from Linus is that we should avoid breaking builds. It wrecks bisection, it causes problems across compiler versions, etc. Change since v1: * Introduce a new Kconfig item CONFIG_LD_ORPHAN_WARN_LEVEL to replace (if $(CONFIG_WERROR),error,warn) (Nathan Chancellor). Xin Li (1): kbuild: upgrade the orphan section warning to an error if CONFIG_WERROR is set Makefile | 2 +- arch/arm/boot/compressed/Makefile | 2 +- arch/arm64/kernel/vdso/Makefile | 2 +- arch/arm64/kernel/vdso32/Makefile | 2 +- arch/x86/boot/compressed/Makefile | 2 +- init/Kconfig | 15 ++++++++++++--- 6 files changed, 17 insertions(+), 8 deletions(-)