Fail the build on objtool warnings
Status from this message: The following commit has been merged into the objtool/core branch of tip:
Get series with b4
b4 am [email protected]Versions newest first
Patches in v1 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/13 | x86/traps: Make exc_double_fault() consistently noreturnarch/ tools/ | +18 | 2 replies | |
| 2/13 | objtool: Fix error handling inconsistencies in check()tools/ | +6 | 2 replies | |
| 3/13 | objtool: Improve __noreturn annotation warningtools/ tools/ | +6 | 2 replies | |
| 4/13 | objtool: Update documentationtools/ | +53 | 2 replies | |
| 5/13 | objtool: Increase per-function WARN_FUNC() rate limittools/ tools/ tools/ | +13 | 3 replies | |
| 6/13 | objtool: Remove --unret dependency on --rethunktools/ | +0 | 4 replies | |
| 7/13 | objtool: Consolidate option validationtools/ | +24 | 2 replies | |
| 8/13 | objtool: Upgrade "Linked object detected" warning to errortools/ | +2 | 2 replies | |
| 9/13 | objtool: Add --output optiontools/ tools/ tools/ +2 more | +89 | 2 replies | |
| 10/13 | objtool: Add --Werror optiontools/ tools/ tools/ | +14 | 1 reply | |
| 11/13 | objtool: Change "warning:" to "error:" for --Werrortools/ | +4 | 2 replies | |
| 12/13 | objtool: Create backup on error and print argstools/ tools/ tools/ | +65 | 2 replies | |
| 13/13 | objtool: Add CONFIG_OBJTOOL_WERRORlib/ scripts/ | +13 | 14 replies |
Reviews trailers per patch
9 of 13 patches have a Reviewed-by or Acked-by.
| Patch | Brendan Jackman | Miroslav Benes |
|---|---|---|
| 1/13 | R-b | R-b |
| 2/13 | R-b | R-b |
| 3/13 | ||
| 4/13 | ||
| 5/13 | ||
| 6/13 | ||
| 7/13 | R-b | R-b |
| 8/13 | R-b | |
| 9/13 | R-b | |
| 10/13 | R-b | |
| 11/13 | R-b | R-b |
| 12/13 | R-b | |
| 13/13 | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 00/13] Fail the build on objtool warnings
Add an option to fail the build on objtool warnings, and enable it by default except for CONFIG_COMPILE_TEST.
This is long overdue, probably the only reason we didn't do it before (other than lack of time) was because there are still some obscure warnings out there, especially once KCOV and sanitizers get added to the mix.
But those are mostly bot issues, the main concern for objtool warnings is actual real live kernels. Hopefully the !COMPILE_TEST guard will avoid too much wreckage for now.
Josh Poimboeuf (13):
x86/traps: Make exc_double_fault() consistently noreturn
objtool: Fix error handling inconsistencies in check()
objtool: Improve __noreturn annotation warning
objtool: Update documentation
objtool: Increase per-function WARN_FUNC() rate limit
objtool: Remove --unret dependency on --rethunk
objtool: Consolidate option validation
objtool: Upgrade "Linked object detected" warning to error
objtool: Add --output option
objtool: Add --Werror option
objtool: Change "warning:" to "error:" for --Werror
objtool: Create backup on error and print args
objtool: Add CONFIG_OBJTOOL_WERROR
arch/x86/kernel/traps.c | 18 +- lib/Kconfig.debug | 12 ++ scripts/Makefile.lib | 1 + tools/objtool/Documentation/objtool.txt | 105 ++++++------ tools/objtool/builtin-check.c | 212 ++++++++++++++++++------ tools/objtool/check.c | 60 +++---- tools/objtool/elf.c | 3 - tools/objtool/include/objtool/builtin.h | 3 +- tools/objtool/include/objtool/elf.h | 2 +- tools/objtool/include/objtool/warn.h | 20 ++- tools/objtool/objtool.c | 78 +-------- tools/objtool/orc_dump.c | 7 +- 12 files changed, 290 insertions(+), 231 deletions(-)
Signature · 1 line
2.48.1