objtool: Ignore end-of-section jumps for KCOV/GCOV

When KCOV or GCOV is enabled, dead code can be left behind, in which
case objtool silences unreachable and undefined behavior (fallthrough)
warnings.

Fallthrough warnings, and their variant "end of section" warnings, were
silenced with the following commit:

  6b023c784204 ("objtool: Silence more KCOV warnings")

Another variant of a fallthrough warning is a jump to the end of a
function.  If that function happens to be at the end of a section, the
jump destination doesn't actually exist.

Normally that would be a fatal objtool error, but for KCOV/GCOV it's
just another undefined behavior fallthrough.  Silence it like the
others.

Fixes the following warning:

  drivers/iommu/dma-iommu.o: warning: objtool: iommu_dma_sw_msi+0x92: can't find jump dest instruction at .text+0x54d5

Fixes: 6b023c784204 ("objtool: Silence more KCOV warnings")
Reported-by: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Josh Poimboeuf <jpoimboe@kernel.org>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Link: https://lore.kernel.org/r/08fbe7d7e1e20612206f1df253077b94f178d93e.1743481539.git.jpoimboe@kernel.org
Closes: https://lore.kernel.org/314f8809-cd59-479b-97d7-49356bf1c8d1@infradead.org/

diff --git a/tools/objtool/check.c b/tools/objtool/check.c
index fff9d7a2947ae53e966bfb776c23877964a73939..e6c4eefe295ba1f3f41595012689a8721f4f5931 100644 (file)
--- a/tools/objtool/check.c
+++ b/tools/objtool/check.c
@@ -1488,6 +1488,8 @@ static int add_jump_destinations(struct objtool_file *file)
        int ret;
 
        for_each_insn(file, insn) {
+               struct symbol *func = insn_func(insn);
+
                if (insn->jump_dest) {
                        /*
                         * handle_group_alt() may have previously set
@@ -1513,7 +1515,7 @@ static int add_jump_destinations(struct objtool_file *file)
                } else if (reloc->sym->return_thunk) {
                        add_return_call(file, insn, true);
                        continue;
-               } else if (insn_func(insn)) {
+               } else if (func) {
                        /*
                         * External sibling call or internal sibling call with
                         * STT_FUNC reloc.
@@ -1548,6 +1550,15 @@ static int add_jump_destinations(struct objtool_file *file)
                                continue;
                        }
 
+                       /*
+                        * GCOV/KCOV dead code can jump to the end of the
+                        * function/section.
+                        */
+                       if (file->ignore_unreachables && func &&
+                           dest_sec == insn->sec &&
+                           dest_off == func->offset + func->len)
+                               continue;
+
                        WARN_INSN(insn, "can't find jump dest instruction at %s+0x%lx",
                                  dest_sec->name, dest_off);
                        return -1;
@@ -1574,8 +1585,7 @@ static int add_jump_destinations(struct objtool_file *file)
                /*
                 * Cross-function jump.
                 */
-               if (insn_func(insn) && insn_func(jump_dest) &&
-                   insn_func(insn) != insn_func(jump_dest)) {
+               if (func && insn_func(jump_dest) && func != insn_func(jump_dest)) {
 
                        /*
                         * For GCC 8+, create parent/child links for any cold
@@ -1592,10 +1602,10 @@ static int add_jump_destinations(struct objtool_file *file)
                         * case where the parent function's only reference to a
                         * subfunction is through a jump table.
                         */
-                       if (!strstr(insn_func(insn)->name, ".cold") &&
+                       if (!strstr(func->name, ".cold") &&
                            strstr(insn_func(jump_dest)->name, ".cold")) {
-                               insn_func(insn)->cfunc = insn_func(jump_dest);
-                               insn_func(jump_dest)->pfunc = insn_func(insn);
+                               func->cfunc = insn_func(jump_dest);
+                               insn_func(jump_dest)->pfunc = func;
                        }
                }