From a34454722b35fb2f62ca01af1cf28485ffa0837e Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman Date: Fri, 29 Jan 2021 11:12:39 +0100 Subject: [PATCH] 5.10-stable patches added patches: objtool-don-t-fail-on-missing-symbol-table.patch --- ...l-don-t-fail-on-missing-symbol-table.patch | 47 +++++++++++++++++++ queue-5.10/series | 1 + 2 files changed, 48 insertions(+) create mode 100644 queue-5.10/objtool-don-t-fail-on-missing-symbol-table.patch diff --git a/queue-5.10/objtool-don-t-fail-on-missing-symbol-table.patch b/queue-5.10/objtool-don-t-fail-on-missing-symbol-table.patch new file mode 100644 index 00000000000..3df6e573344 --- /dev/null +++ b/queue-5.10/objtool-don-t-fail-on-missing-symbol-table.patch @@ -0,0 +1,47 @@ +From 1d489151e9f9d1647110277ff77282fe4d96d09b Mon Sep 17 00:00:00 2001 +From: Josh Poimboeuf +Date: Thu, 14 Jan 2021 16:14:01 -0600 +Subject: objtool: Don't fail on missing symbol table + +From: Josh Poimboeuf + +commit 1d489151e9f9d1647110277ff77282fe4d96d09b upstream. + +Thanks to a recent binutils change which doesn't generate unused +symbols, it's now possible for thunk_64.o be completely empty without +CONFIG_PREEMPTION: no text, no data, no symbols. + +We could edit the Makefile to only build that file when +CONFIG_PREEMPTION is enabled, but that will likely create confusion +if/when the thunks end up getting used by some other code again. + +Just ignore it and move on. + +Reported-by: Nathan Chancellor +Reviewed-by: Nathan Chancellor +Reviewed-by: Miroslav Benes +Tested-by: Nathan Chancellor +Link: https://github.com/ClangBuiltLinux/linux/issues/1254 +Signed-off-by: Josh Poimboeuf +Signed-off-by: Greg Kroah-Hartman + +--- + tools/objtool/elf.c | 7 +++++-- + 1 file changed, 5 insertions(+), 2 deletions(-) + +--- a/tools/objtool/elf.c ++++ b/tools/objtool/elf.c +@@ -354,8 +354,11 @@ static int read_symbols(struct elf *elf) + + symtab = find_section_by_name(elf, ".symtab"); + if (!symtab) { +- WARN("missing symbol table"); +- return -1; ++ /* ++ * A missing symbol table is actually possible if it's an empty ++ * .o file. This can happen for thunk_64.o. ++ */ ++ return 0; + } + + symtab_shndx = find_section_by_name(elf, ".symtab_shndx"); diff --git a/queue-5.10/series b/queue-5.10/series index 5d5595f5372..5f1c6a18962 100644 --- a/queue-5.10/series +++ b/queue-5.10/series @@ -24,3 +24,4 @@ io_uring-fix-uring_flush-in-exit_files-warning.patch io_uring-fix-skipping-disabling-sqo-on-exec.patch io_uring-dont-kill-fasync-under-completion_lock.patch io_uring-fix-sleeping-under-spin-in-__io_clean_op.patch +objtool-don-t-fail-on-missing-symbol-table.patch -- 2.47.3