]> www.infradead.org Git - users/hch/misc.git/commitdiff
perf srcline: Fallback between addr2line implementations
authorIan Rogers <irogers@google.com>
Sun, 5 Oct 2025 21:22:08 +0000 (14:22 -0700)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 6 Oct 2025 19:47:49 +0000 (16:47 -0300)
Factor the addr2line function implementation into separate source files
(addr2line.[ch]) and rename the addr2line function cmd__addr2line. In
srcline replace the ifdef-ed addr2line implementations with one that
first tries the llvm__addr2line implementation, then the deprecated
libbfd__addr2line function and on failure uses cmd__addr2line.

If HAVE_LIBLLVM_SUPPORT is enabled the llvm__addr2line will execute
against the libLLVM.so it is linked against.

If HAVE_LIBLLVM_DYNAMIC is enabled then libperf-llvm.so (that links
against libLLVM.so) will be dlopened. If the dlopen succeeds then the
behavior should match HAVE_LIBLLVM_SUPPORT. On failure cmd__addr2line is
used. The dlopen is only tried once.

If HAVE_LIBLLVM_DYNAMIC isn't enabled then llvm__addr2line immediately
fails and cmd__addr2line is used.

Clean up the dso__free_a2l logic, which is only needed in the non-LLVM
version and moved to addr2line.c.

Signed-off-by: Ian Rogers <irogers@google.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Alexandre Ghiti <alexghiti@rivosinc.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Athira Rajeev <atrajeev@linux.ibm.com>
Cc: Bill Wendling <morbo@google.com>
Cc: Charlie Jenkins <charlie@rivosinc.com>
Cc: Collin Funk <collin.funk1@gmail.com>
Cc: Dmitriy Vyukov <dvyukov@google.com>
Cc: Dr. David Alan Gilbert <linux@treblig.org>
Cc: Eric Biggers <ebiggers@kernel.org>
Cc: Haibo Xu <haibo1.xu@intel.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: James Clark <james.clark@linaro.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Justin Stitt <justinstitt@google.com>
Cc: Li Huafei <lihuafei1@huawei.com>
Cc: Masami Hiramatsu <mhiramat@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Nathan Chancellor <nathan@kernel.org>
Cc: Nick Desaulniers <nick.desaulniers+lkml@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Song Liu <song@kernel.org>
Cc: Stephen Brennan <stephen.s.brennan@oracle.com>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/Build
tools/perf/util/addr2line.c [new file with mode: 0644]
tools/perf/util/addr2line.h [new file with mode: 0644]
tools/perf/util/config.c
tools/perf/util/llvm.c
tools/perf/util/llvm.h
tools/perf/util/srcline.c
tools/perf/util/srcline.h

index e6b396d919bbbf9f4dc7e528b438d1e6360875c5..4be313cd115aeacfc16437456e46ae9d300bf602 100644 (file)
@@ -2,6 +2,7 @@ include $(srctree)/tools/scripts/Makefile.include
 include $(srctree)/tools/scripts/utilities.mak
 
 perf-util-y += arm64-frame-pointer-unwind-support.o
+perf-util-y += addr2line.o
 perf-util-y += addr_location.o
 perf-util-y += annotate.o
 perf-util-y += block-info.o
diff --git a/tools/perf/util/addr2line.c b/tools/perf/util/addr2line.c
new file mode 100644 (file)
index 0000000..f2d94a3
--- /dev/null
@@ -0,0 +1,439 @@
+// SPDX-License-Identifier: GPL-2.0
+#include "addr2line.h"
+#include "debug.h"
+#include "dso.h"
+#include "string2.h"
+#include "srcline.h"
+#include "symbol.h"
+#include "symbol_conf.h"
+
+#include <api/io.h>
+#include <linux/zalloc.h>
+#include <subcmd/run-command.h>
+
+#include <inttypes.h>
+#include <signal.h>
+#include <stdlib.h>
+#include <string.h>
+
+#define MAX_INLINE_NEST 1024
+
+/* If addr2line doesn't return data for 1 second then timeout. */
+int addr2line_timeout_ms = 1 * 1000;
+
+static int filename_split(char *filename, unsigned int *line_nr)
+{
+       char *sep;
+
+       sep = strchr(filename, '\n');
+       if (sep)
+               *sep = '\0';
+
+       if (!strcmp(filename, "??:0"))
+               return 0;
+
+       sep = strchr(filename, ':');
+       if (sep) {
+               *sep++ = '\0';
+               *line_nr = strtoul(sep, NULL, 0);
+               return 1;
+       }
+       pr_debug("addr2line missing ':' in filename split\n");
+       return 0;
+}
+
+static void addr2line_subprocess_cleanup(struct child_process *a2l)
+{
+       if (a2l->pid != -1) {
+               kill(a2l->pid, SIGKILL);
+               finish_command(a2l); /* ignore result, we don't care */
+               a2l->pid = -1;
+               close(a2l->in);
+               close(a2l->out);
+       }
+
+       free(a2l);
+}
+
+static struct child_process *addr2line_subprocess_init(const char *addr2line_path,
+                                                       const char *binary_path)
+{
+       const char *argv[] = {
+               addr2line_path ?: "addr2line",
+               "-e", binary_path,
+               "-a", "-i", "-f", NULL
+       };
+       struct child_process *a2l = zalloc(sizeof(*a2l));
+       int start_command_status = 0;
+
+       if (a2l == NULL) {
+               pr_err("Failed to allocate memory for addr2line");
+               return NULL;
+       }
+
+       a2l->pid = -1;
+       a2l->in = -1;
+       a2l->out = -1;
+       a2l->no_stderr = 1;
+
+       a2l->argv = argv;
+       start_command_status = start_command(a2l);
+       a2l->argv = NULL; /* it's not used after start_command; avoid dangling pointers */
+
+       if (start_command_status != 0) {
+               pr_warning("could not start addr2line (%s) for %s: start_command return code %d\n",
+                       addr2line_path, binary_path, start_command_status);
+               addr2line_subprocess_cleanup(a2l);
+               return NULL;
+       }
+
+       return a2l;
+}
+
+enum a2l_style {
+       BROKEN,
+       GNU_BINUTILS,
+       LLVM,
+};
+
+static enum a2l_style addr2line_configure(struct child_process *a2l, const char *dso_name)
+{
+       static bool cached;
+       static enum a2l_style style;
+
+       if (!cached) {
+               char buf[128];
+               struct io io;
+               int ch;
+               int lines;
+
+               if (write(a2l->in, ",\n", 2) != 2)
+                       return BROKEN;
+
+               io__init(&io, a2l->out, buf, sizeof(buf));
+               ch = io__get_char(&io);
+               if (ch == ',') {
+                       style = LLVM;
+                       cached = true;
+                       lines = 1;
+                       pr_debug3("Detected LLVM addr2line style\n");
+               } else if (ch == '0') {
+                       style = GNU_BINUTILS;
+                       cached = true;
+                       lines = 3;
+                       pr_debug3("Detected binutils addr2line style\n");
+               } else {
+                       if (!symbol_conf.disable_add2line_warn) {
+                               char *output = NULL;
+                               size_t output_len;
+
+                               io__getline(&io, &output, &output_len);
+                               pr_warning("%s %s: addr2line configuration failed\n",
+                                          __func__, dso_name);
+                               pr_warning("\t%c%s", ch, output);
+                       }
+                       pr_debug("Unknown/broken addr2line style\n");
+                       return BROKEN;
+               }
+               while (lines) {
+                       ch = io__get_char(&io);
+                       if (ch <= 0)
+                               break;
+                       if (ch == '\n')
+                               lines--;
+               }
+               /* Ignore SIGPIPE in the event addr2line exits. */
+               signal(SIGPIPE, SIG_IGN);
+       }
+       return style;
+}
+
+static int read_addr2line_record(struct io *io,
+                                enum a2l_style style,
+                                const char *dso_name,
+                                u64 addr,
+                                bool first,
+                                char **function,
+                                char **filename,
+                                unsigned int *line_nr)
+{
+       /*
+        * Returns:
+        * -1 ==> error
+        * 0 ==> sentinel (or other ill-formed) record read
+        * 1 ==> a genuine record read
+        */
+       char *line = NULL;
+       size_t line_len = 0;
+       unsigned int dummy_line_nr = 0;
+       int ret = -1;
+
+       if (function != NULL)
+               zfree(function);
+
+       if (filename != NULL)
+               zfree(filename);
+
+       if (line_nr != NULL)
+               *line_nr = 0;
+
+       /*
+        * Read the first line. Without an error this will be:
+        * - for the first line an address like 0x1234,
+        * - the binutils sentinel 0x0000000000000000,
+        * - the llvm-addr2line the sentinel ',' character,
+        * - the function name line for an inlined function.
+        */
+       if (io__getline(io, &line, &line_len) < 0 || !line_len)
+               goto error;
+
+       pr_debug3("%s %s: addr2line read address for sentinel: %s", __func__, dso_name, line);
+       if (style == LLVM && line_len == 2 && line[0] == ',') {
+               /* Found the llvm-addr2line sentinel character. */
+               zfree(&line);
+               return 0;
+       } else if (style == GNU_BINUTILS && (!first || addr != 0)) {
+               int zero_count = 0, non_zero_count = 0;
+               /*
+                * Check for binutils sentinel ignoring it for the case the
+                * requested address is 0.
+                */
+
+               /* A given address should always start 0x. */
+               if (line_len >= 2 || line[0] != '0' || line[1] != 'x') {
+                       for (size_t i = 2; i < line_len; i++) {
+                               if (line[i] == '0')
+                                       zero_count++;
+                               else if (line[i] != '\n')
+                                       non_zero_count++;
+                       }
+                       if (!non_zero_count) {
+                               int ch;
+
+                               if (first && !zero_count) {
+                                       /* Line was erroneous just '0x'. */
+                                       goto error;
+                               }
+                               /*
+                                * Line was 0x0..0, the sentinel for binutils. Remove
+                                * the function and filename lines.
+                                */
+                               zfree(&line);
+                               do {
+                                       ch = io__get_char(io);
+                               } while (ch > 0 && ch != '\n');
+                               do {
+                                       ch = io__get_char(io);
+                               } while (ch > 0 && ch != '\n');
+                               return 0;
+                       }
+               }
+       }
+       /* Read the second function name line (if inline data then this is the first line). */
+       if (first && (io__getline(io, &line, &line_len) < 0 || !line_len))
+               goto error;
+
+       pr_debug3("%s %s: addr2line read line: %s", __func__, dso_name, line);
+       if (function != NULL)
+               *function = strdup(strim(line));
+
+       zfree(&line);
+       line_len = 0;
+
+       /* Read the third filename and line number line. */
+       if (io__getline(io, &line, &line_len) < 0 || !line_len)
+               goto error;
+
+       pr_debug3("%s %s: addr2line filename:number : %s", __func__, dso_name, line);
+       if (filename_split(line, line_nr == NULL ? &dummy_line_nr : line_nr) == 0 &&
+           style == GNU_BINUTILS) {
+               ret = 0;
+               goto error;
+       }
+
+       if (filename != NULL)
+               *filename = strdup(line);
+
+       zfree(&line);
+       line_len = 0;
+
+       return 1;
+
+error:
+       free(line);
+       if (function != NULL)
+               zfree(function);
+       if (filename != NULL)
+               zfree(filename);
+       return ret;
+}
+
+static int inline_list__append_record(struct dso *dso,
+                                     struct inline_node *node,
+                                     struct symbol *sym,
+                                     const char *function,
+                                     const char *filename,
+                                     unsigned int line_nr)
+{
+       struct symbol *inline_sym = new_inline_sym(dso, sym, function);
+
+       return inline_list__append(inline_sym, srcline_from_fileline(filename, line_nr), node);
+}
+
+int cmd__addr2line(const char *dso_name, u64 addr,
+                  char **file, unsigned int *line_nr,
+                  struct dso *dso,
+                  bool unwind_inlines,
+                  struct inline_node *node,
+                  struct symbol *sym __maybe_unused)
+{
+       struct child_process *a2l = dso__a2l(dso);
+       char *record_function = NULL;
+       char *record_filename = NULL;
+       unsigned int record_line_nr = 0;
+       int record_status = -1;
+       int ret = 0;
+       size_t inline_count = 0;
+       int len;
+       char buf[128];
+       ssize_t written;
+       struct io io = { .eof = false };
+       enum a2l_style a2l_style;
+
+       if (!a2l) {
+               if (!filename__has_section(dso_name, ".debug_line"))
+                       goto out;
+
+               dso__set_a2l(dso,
+                            addr2line_subprocess_init(symbol_conf.addr2line_path, dso_name));
+               a2l = dso__a2l(dso);
+       }
+
+       if (a2l == NULL) {
+               if (!symbol_conf.disable_add2line_warn)
+                       pr_warning("%s %s: addr2line_subprocess_init failed\n", __func__, dso_name);
+               goto out;
+       }
+       a2l_style = addr2line_configure(a2l, dso_name);
+       if (a2l_style == BROKEN)
+               goto out;
+
+       /*
+        * Send our request and then *deliberately* send something that can't be
+        * interpreted as a valid address to ask addr2line about (namely,
+        * ","). This causes addr2line to first write out the answer to our
+        * request, in an unbounded/unknown number of records, and then to write
+        * out the lines "0x0...0", "??" and "??:0", for GNU binutils, or ","
+        * for llvm-addr2line, so that we can detect when it has finished giving
+        * us anything useful.
+        */
+       len = snprintf(buf, sizeof(buf), "%016"PRIx64"\n,\n", addr);
+       written = len > 0 ? write(a2l->in, buf, len) : -1;
+       if (written != len) {
+               if (!symbol_conf.disable_add2line_warn)
+                       pr_warning("%s %s: could not send request\n", __func__, dso_name);
+               goto out;
+       }
+       io__init(&io, a2l->out, buf, sizeof(buf));
+       io.timeout_ms = addr2line_timeout_ms;
+       switch (read_addr2line_record(&io, a2l_style, dso_name, addr, /*first=*/true,
+                                     &record_function, &record_filename, &record_line_nr)) {
+       case -1:
+               if (!symbol_conf.disable_add2line_warn)
+                       pr_warning("%s %s: could not read first record\n", __func__, dso_name);
+               goto out;
+       case 0:
+               /*
+                * The first record was invalid, so return failure, but first
+                * read another record, since we sent a sentinel ',' for the
+                * sake of detected the last inlined function. Treat this as the
+                * first of a record as the ',' generates a new start with GNU
+                * binutils, also force a non-zero address as we're no longer
+                * reading that record.
+                */
+               switch (read_addr2line_record(&io, a2l_style, dso_name,
+                                             /*addr=*/1, /*first=*/true,
+                                             NULL, NULL, NULL)) {
+               case -1:
+                       if (!symbol_conf.disable_add2line_warn)
+                               pr_warning("%s %s: could not read sentinel record\n",
+                                          __func__, dso_name);
+                       break;
+               case 0:
+                       /* The sentinel as expected. */
+                       break;
+               default:
+                       if (!symbol_conf.disable_add2line_warn)
+                               pr_warning("%s %s: unexpected record instead of sentinel",
+                                          __func__, dso_name);
+                       break;
+               }
+               goto out;
+       default:
+               /* First record as expected. */
+               break;
+       }
+
+       if (file) {
+               *file = strdup(record_filename);
+               ret = 1;
+       }
+       if (line_nr)
+               *line_nr = record_line_nr;
+
+       if (unwind_inlines) {
+               if (node && inline_list__append_record(dso, node, sym,
+                                                      record_function,
+                                                      record_filename,
+                                                      record_line_nr)) {
+                       ret = 0;
+                       goto out;
+               }
+       }
+
+       /*
+        * We have to read the records even if we don't care about the inline
+        * info. This isn't the first record and force the address to non-zero
+        * as we're reading records beyond the first.
+        */
+       while ((record_status = read_addr2line_record(&io,
+                                                     a2l_style,
+                                                     dso_name,
+                                                     /*addr=*/1,
+                                                     /*first=*/false,
+                                                     &record_function,
+                                                     &record_filename,
+                                                     &record_line_nr)) == 1) {
+               if (unwind_inlines && node && inline_count++ < MAX_INLINE_NEST) {
+                       if (inline_list__append_record(dso, node, sym,
+                                                      record_function,
+                                                      record_filename,
+                                                      record_line_nr)) {
+                               ret = 0;
+                               goto out;
+                       }
+                       ret = 1; /* found at least one inline frame */
+               }
+       }
+
+out:
+       free(record_function);
+       free(record_filename);
+       if (io.eof) {
+               dso__set_a2l(dso, NULL);
+               addr2line_subprocess_cleanup(a2l);
+       }
+       return ret;
+}
+
+void dso__free_a2l(struct dso *dso)
+{
+       struct child_process *a2l = dso__a2l(dso);
+
+       if (!a2l)
+               return;
+
+       addr2line_subprocess_cleanup(a2l);
+
+       dso__set_a2l(dso, NULL);
+}
diff --git a/tools/perf/util/addr2line.h b/tools/perf/util/addr2line.h
new file mode 100644 (file)
index 0000000..d35a47b
--- /dev/null
@@ -0,0 +1,20 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+#ifndef __PERF_ADDR2LINE_H
+#define __PERF_ADDR2LINE_H
+
+#include <linux/types.h>
+
+struct dso;
+struct inline_node;
+struct symbol;
+
+extern int addr2line_timeout_ms;
+
+int cmd__addr2line(const char *dso_name, u64 addr,
+                  char **file, unsigned int *line_nr,
+                  struct dso *dso,
+                  bool unwind_inlines,
+                  struct inline_node *node,
+                  struct symbol *sym);
+
+#endif /* __PERF_ADDR2LINE_H */
index ae72b66b6ded04b570deba834b1cf2cfa3f7c0ab..6f914620c6ff29a1e0b22ccc4b8960d855b5256a 100644 (file)
@@ -19,7 +19,7 @@
 #include "util/hist.h"  /* perf_hist_config */
 #include "util/stat.h"  /* perf_stat__set_big_num */
 #include "util/evsel.h"  /* evsel__hw_names, evsel__use_bpf_counters */
-#include "util/srcline.h"  /* addr2line_timeout_ms */
+#include "util/addr2line.h"  /* addr2line_timeout_ms */
 #include "build-id.h"
 #include "debug.h"
 #include "config.h"
index 43e6fecfce59fbb60730c310a596b64b3cace0d1..2ebf1f5f65bf77c7db188e464ec3fcdd4b6c1145 100644 (file)
@@ -70,11 +70,6 @@ int llvm__addr2line(const char *dso_name __maybe_unused, u64 addr __maybe_unused
 #endif
 }
 
-void dso__free_a2l_llvm(struct dso *dso __maybe_unused)
-{
-       /* Nothing to free. */
-}
-
 #ifdef HAVE_LIBLLVM_SUPPORT
 static void init_llvm(void)
 {
index 8aa19bb6b0689da8e05ce037c3e2aa54fd592420..57f6bafb24bbb37e42978863d6a26f913cfea62d 100644 (file)
@@ -15,9 +15,6 @@ int llvm__addr2line(const char *dso_name, u64 addr,
                bool unwind_inlines, struct inline_node *node,
                struct symbol *sym);
 
-
-void dso__free_a2l_llvm(struct dso *dso);
-
 int symbol__disassemble_llvm(const char *filename, struct symbol *sym,
                             struct annotate_args *args);
 
index 23b942d4729e9e31d25d1fb35efc963627a69ead..27c0966611abb70baf4eed78d67316b3b3671af1 100644 (file)
@@ -1,30 +1,15 @@
 // SPDX-License-Identifier: GPL-2.0
-#include <inttypes.h>
-#include <signal.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-#include <sys/types.h>
-
-#include <linux/compiler.h>
-#include <linux/kernel.h>
-#include <linux/string.h>
-#include <linux/zalloc.h>
-
-#include <api/io.h>
-
-#include "util/dso.h"
-#include "util/debug.h"
-#include "util/callchain.h"
-#include "util/symbol_conf.h"
-#include "llvm.h"
 #include "srcline.h"
-#include "string2.h"
+#include "addr2line.h"
+#include "dso.h"
+#include "callchain.h"
+#include "libbfd.h"
+#include "llvm.h"
 #include "symbol.h"
-#include "subcmd/run-command.h"
 
-/* If addr2line doesn't return data for 1 second then timeout. */
-int addr2line_timeout_ms = 1 * 1000;
+#include <inttypes.h>
+#include <string.h>
+
 bool srcline_full_filename;
 
 char *srcline__unknown = (char *)"??:0";
@@ -129,458 +114,23 @@ struct symbol *new_inline_sym(struct dso *dso,
        return inline_sym;
 }
 
-#ifdef HAVE_LIBLLVM_SUPPORT
-#include "llvm.h"
-
-static int addr2line(const char *dso_name, u64 addr,
-                    char **file, unsigned int *line, struct dso *dso,
-                     bool unwind_inlines, struct inline_node *node,
-                     struct symbol *sym)
-{
-       return llvm__addr2line(dso_name, addr, file, line, dso, unwind_inlines, node, sym);
-}
-
-void dso__free_a2l(struct dso *dso)
-{
-       dso__free_a2l_llvm(dso);
-}
-#elif defined(HAVE_LIBBFD_SUPPORT)
-#include "libbfd.h"
-
-static int addr2line(const char *dso_name, u64 addr,
-                    char **file, unsigned int *line, struct dso *dso,
-                    bool unwind_inlines, struct inline_node *node,
+static int addr2line(const char *dso_name, u64 addr, char **file, unsigned int *line_nr,
+                    struct dso *dso, bool unwind_inlines, struct inline_node *node,
                     struct symbol *sym)
 {
-       return libbfd__addr2line(dso_name, addr, file, line, dso, unwind_inlines, node, sym);
-}
-
-void dso__free_a2l(struct dso *dso)
-{
-       dso__free_a2l_libbfd(dso);
-}
-
-#else /* HAVE_LIBBFD_SUPPORT */
-
-static int filename_split(char *filename, unsigned int *line_nr)
-{
-       char *sep;
-
-       sep = strchr(filename, '\n');
-       if (sep)
-               *sep = '\0';
-
-       if (!strcmp(filename, "??:0"))
-               return 0;
-
-       sep = strchr(filename, ':');
-       if (sep) {
-               *sep++ = '\0';
-               *line_nr = strtoul(sep, NULL, 0);
-               return 1;
-       }
-       pr_debug("addr2line missing ':' in filename split\n");
-       return 0;
-}
-
-static void addr2line_subprocess_cleanup(struct child_process *a2l)
-{
-       if (a2l->pid != -1) {
-               kill(a2l->pid, SIGKILL);
-               finish_command(a2l); /* ignore result, we don't care */
-               a2l->pid = -1;
-               close(a2l->in);
-               close(a2l->out);
-       }
-
-       free(a2l);
-}
-
-static struct child_process *addr2line_subprocess_init(const char *addr2line_path,
-                                                       const char *binary_path)
-{
-       const char *argv[] = {
-               addr2line_path ?: "addr2line",
-               "-e", binary_path,
-               "-a", "-i", "-f", NULL
-       };
-       struct child_process *a2l = zalloc(sizeof(*a2l));
-       int start_command_status = 0;
-
-       if (a2l == NULL) {
-               pr_err("Failed to allocate memory for addr2line");
-               return NULL;
-       }
-
-       a2l->pid = -1;
-       a2l->in = -1;
-       a2l->out = -1;
-       a2l->no_stderr = 1;
-
-       a2l->argv = argv;
-       start_command_status = start_command(a2l);
-       a2l->argv = NULL; /* it's not used after start_command; avoid dangling pointers */
-
-       if (start_command_status != 0) {
-               pr_warning("could not start addr2line (%s) for %s: start_command return code %d\n",
-                       addr2line_path, binary_path, start_command_status);
-               addr2line_subprocess_cleanup(a2l);
-               return NULL;
-       }
-
-       return a2l;
-}
-
-enum a2l_style {
-       BROKEN,
-       GNU_BINUTILS,
-       LLVM,
-};
-
-static enum a2l_style addr2line_configure(struct child_process *a2l, const char *dso_name)
-{
-       static bool cached;
-       static enum a2l_style style;
-
-       if (!cached) {
-               char buf[128];
-               struct io io;
-               int ch;
-               int lines;
-
-               if (write(a2l->in, ",\n", 2) != 2)
-                       return BROKEN;
-
-               io__init(&io, a2l->out, buf, sizeof(buf));
-               ch = io__get_char(&io);
-               if (ch == ',') {
-                       style = LLVM;
-                       cached = true;
-                       lines = 1;
-                       pr_debug3("Detected LLVM addr2line style\n");
-               } else if (ch == '0') {
-                       style = GNU_BINUTILS;
-                       cached = true;
-                       lines = 3;
-                       pr_debug3("Detected binutils addr2line style\n");
-               } else {
-                       if (!symbol_conf.disable_add2line_warn) {
-                               char *output = NULL;
-                               size_t output_len;
-
-                               io__getline(&io, &output, &output_len);
-                               pr_warning("%s %s: addr2line configuration failed\n",
-                                          __func__, dso_name);
-                               pr_warning("\t%c%s", ch, output);
-                       }
-                       pr_debug("Unknown/broken addr2line style\n");
-                       return BROKEN;
-               }
-               while (lines) {
-                       ch = io__get_char(&io);
-                       if (ch <= 0)
-                               break;
-                       if (ch == '\n')
-                               lines--;
-               }
-               /* Ignore SIGPIPE in the event addr2line exits. */
-               signal(SIGPIPE, SIG_IGN);
-       }
-       return style;
-}
-
-static int read_addr2line_record(struct io *io,
-                                enum a2l_style style,
-                                const char *dso_name,
-                                u64 addr,
-                                bool first,
-                                char **function,
-                                char **filename,
-                                unsigned int *line_nr)
-{
-       /*
-        * Returns:
-        * -1 ==> error
-        * 0 ==> sentinel (or other ill-formed) record read
-        * 1 ==> a genuine record read
-        */
-       char *line = NULL;
-       size_t line_len = 0;
-       unsigned int dummy_line_nr = 0;
-       int ret = -1;
-
-       if (function != NULL)
-               zfree(function);
-
-       if (filename != NULL)
-               zfree(filename);
-
-       if (line_nr != NULL)
-               *line_nr = 0;
-
-       /*
-        * Read the first line. Without an error this will be:
-        * - for the first line an address like 0x1234,
-        * - the binutils sentinel 0x0000000000000000,
-        * - the llvm-addr2line the sentinel ',' character,
-        * - the function name line for an inlined function.
-        */
-       if (io__getline(io, &line, &line_len) < 0 || !line_len)
-               goto error;
-
-       pr_debug3("%s %s: addr2line read address for sentinel: %s", __func__, dso_name, line);
-       if (style == LLVM && line_len == 2 && line[0] == ',') {
-               /* Found the llvm-addr2line sentinel character. */
-               zfree(&line);
-               return 0;
-       } else if (style == GNU_BINUTILS && (!first || addr != 0)) {
-               int zero_count = 0, non_zero_count = 0;
-               /*
-                * Check for binutils sentinel ignoring it for the case the
-                * requested address is 0.
-                */
-
-               /* A given address should always start 0x. */
-               if (line_len >= 2 || line[0] != '0' || line[1] != 'x') {
-                       for (size_t i = 2; i < line_len; i++) {
-                               if (line[i] == '0')
-                                       zero_count++;
-                               else if (line[i] != '\n')
-                                       non_zero_count++;
-                       }
-                       if (!non_zero_count) {
-                               int ch;
-
-                               if (first && !zero_count) {
-                                       /* Line was erroneous just '0x'. */
-                                       goto error;
-                               }
-                               /*
-                                * Line was 0x0..0, the sentinel for binutils. Remove
-                                * the function and filename lines.
-                                */
-                               zfree(&line);
-                               do {
-                                       ch = io__get_char(io);
-                               } while (ch > 0 && ch != '\n');
-                               do {
-                                       ch = io__get_char(io);
-                               } while (ch > 0 && ch != '\n');
-                               return 0;
-                       }
-               }
-       }
-       /* Read the second function name line (if inline data then this is the first line). */
-       if (first && (io__getline(io, &line, &line_len) < 0 || !line_len))
-               goto error;
-
-       pr_debug3("%s %s: addr2line read line: %s", __func__, dso_name, line);
-       if (function != NULL)
-               *function = strdup(strim(line));
-
-       zfree(&line);
-       line_len = 0;
-
-       /* Read the third filename and line number line. */
-       if (io__getline(io, &line, &line_len) < 0 || !line_len)
-               goto error;
-
-       pr_debug3("%s %s: addr2line filename:number : %s", __func__, dso_name, line);
-       if (filename_split(line, line_nr == NULL ? &dummy_line_nr : line_nr) == 0 &&
-           style == GNU_BINUTILS) {
-               ret = 0;
-               goto error;
-       }
-
-       if (filename != NULL)
-               *filename = strdup(line);
-
-       zfree(&line);
-       line_len = 0;
-
-       return 1;
-
-error:
-       free(line);
-       if (function != NULL)
-               zfree(function);
-       if (filename != NULL)
-               zfree(filename);
-       return ret;
-}
-
-static int inline_list__append_record(struct dso *dso,
-                                     struct inline_node *node,
-                                     struct symbol *sym,
-                                     const char *function,
-                                     const char *filename,
-                                     unsigned int line_nr)
-{
-       struct symbol *inline_sym = new_inline_sym(dso, sym, function);
-
-       return inline_list__append(inline_sym, srcline_from_fileline(filename, line_nr), node);
-}
-
-static int addr2line(const char *dso_name, u64 addr,
-                    char **file, unsigned int *line_nr,
-                    struct dso *dso,
-                    bool unwind_inlines,
-                    struct inline_node *node,
-                    struct symbol *sym __maybe_unused)
-{
-       struct child_process *a2l = dso__a2l(dso);
-       char *record_function = NULL;
-       char *record_filename = NULL;
-       unsigned int record_line_nr = 0;
-       int record_status = -1;
-       int ret = 0;
-       size_t inline_count = 0;
-       int len;
-       char buf[128];
-       ssize_t written;
-       struct io io = { .eof = false };
-       enum a2l_style a2l_style;
-
-       if (!a2l) {
-               if (!filename__has_section(dso_name, ".debug_line"))
-                       goto out;
-
-               dso__set_a2l(dso,
-                            addr2line_subprocess_init(symbol_conf.addr2line_path, dso_name));
-               a2l = dso__a2l(dso);
-       }
-
-       if (a2l == NULL) {
-               if (!symbol_conf.disable_add2line_warn)
-                       pr_warning("%s %s: addr2line_subprocess_init failed\n", __func__, dso_name);
-               goto out;
-       }
-       a2l_style = addr2line_configure(a2l, dso_name);
-       if (a2l_style == BROKEN)
-               goto out;
-
-       /*
-        * Send our request and then *deliberately* send something that can't be
-        * interpreted as a valid address to ask addr2line about (namely,
-        * ","). This causes addr2line to first write out the answer to our
-        * request, in an unbounded/unknown number of records, and then to write
-        * out the lines "0x0...0", "??" and "??:0", for GNU binutils, or ","
-        * for llvm-addr2line, so that we can detect when it has finished giving
-        * us anything useful.
-        */
-       len = snprintf(buf, sizeof(buf), "%016"PRIx64"\n,\n", addr);
-       written = len > 0 ? write(a2l->in, buf, len) : -1;
-       if (written != len) {
-               if (!symbol_conf.disable_add2line_warn)
-                       pr_warning("%s %s: could not send request\n", __func__, dso_name);
-               goto out;
-       }
-       io__init(&io, a2l->out, buf, sizeof(buf));
-       io.timeout_ms = addr2line_timeout_ms;
-       switch (read_addr2line_record(&io, a2l_style, dso_name, addr, /*first=*/true,
-                                     &record_function, &record_filename, &record_line_nr)) {
-       case -1:
-               if (!symbol_conf.disable_add2line_warn)
-                       pr_warning("%s %s: could not read first record\n", __func__, dso_name);
-               goto out;
-       case 0:
-               /*
-                * The first record was invalid, so return failure, but first
-                * read another record, since we sent a sentinel ',' for the
-                * sake of detected the last inlined function. Treat this as the
-                * first of a record as the ',' generates a new start with GNU
-                * binutils, also force a non-zero address as we're no longer
-                * reading that record.
-                */
-               switch (read_addr2line_record(&io, a2l_style, dso_name,
-                                             /*addr=*/1, /*first=*/true,
-                                             NULL, NULL, NULL)) {
-               case -1:
-                       if (!symbol_conf.disable_add2line_warn)
-                               pr_warning("%s %s: could not read sentinel record\n",
-                                          __func__, dso_name);
-                       break;
-               case 0:
-                       /* The sentinel as expected. */
-                       break;
-               default:
-                       if (!symbol_conf.disable_add2line_warn)
-                               pr_warning("%s %s: unexpected record instead of sentinel",
-                                          __func__, dso_name);
-                       break;
-               }
-               goto out;
-       default:
-               /* First record as expected. */
-               break;
-       }
-
-       if (file) {
-               *file = strdup(record_filename);
-               ret = 1;
-       }
-       if (line_nr)
-               *line_nr = record_line_nr;
-
-       if (unwind_inlines) {
-               if (node && inline_list__append_record(dso, node, sym,
-                                                      record_function,
-                                                      record_filename,
-                                                      record_line_nr)) {
-                       ret = 0;
-                       goto out;
-               }
-       }
-
-       /*
-        * We have to read the records even if we don't care about the inline
-        * info. This isn't the first record and force the address to non-zero
-        * as we're reading records beyond the first.
-        */
-       while ((record_status = read_addr2line_record(&io,
-                                                     a2l_style,
-                                                     dso_name,
-                                                     /*addr=*/1,
-                                                     /*first=*/false,
-                                                     &record_function,
-                                                     &record_filename,
-                                                     &record_line_nr)) == 1) {
-               if (unwind_inlines && node && inline_count++ < MAX_INLINE_NEST) {
-                       if (inline_list__append_record(dso, node, sym,
-                                                      record_function,
-                                                      record_filename,
-                                                      record_line_nr)) {
-                               ret = 0;
-                               goto out;
-                       }
-                       ret = 1; /* found at least one inline frame */
-               }
-       }
-
-out:
-       free(record_function);
-       free(record_filename);
-       if (io.eof) {
-               dso__set_a2l(dso, NULL);
-               addr2line_subprocess_cleanup(a2l);
-       }
-       return ret;
-}
-
-void dso__free_a2l(struct dso *dso)
-{
-       struct child_process *a2l = dso__a2l(dso);
+       int ret;
 
-       if (!a2l)
-               return;
+       ret = llvm__addr2line(dso_name, addr, file, line_nr, dso, unwind_inlines, node, sym);
+       if (ret > 0)
+               return ret;
 
-       addr2line_subprocess_cleanup(a2l);
+       ret = libbfd__addr2line(dso_name, addr, file, line_nr, dso, unwind_inlines, node, sym);
+       if (ret > 0)
+               return ret;
 
-       dso__set_a2l(dso, NULL);
+       return cmd__addr2line(dso_name, addr, file, line_nr, dso, unwind_inlines, node, sym);
 }
 
-#endif /* HAVE_LIBBFD_SUPPORT */
-
 static struct inline_node *addr2inlines(const char *dso_name, u64 addr,
                                        struct dso *dso, struct symbol *sym)
 {
@@ -595,7 +145,9 @@ static struct inline_node *addr2inlines(const char *dso_name, u64 addr,
        INIT_LIST_HEAD(&node->val);
        node->addr = addr;
 
-       addr2line(dso_name, addr, NULL, NULL, dso, true, node, sym);
+       addr2line(dso_name, addr, /*file=*/NULL, /*line_nr=*/NULL, dso,
+                 /*unwind_inlines=*/true, node, sym);
+
        return node;
 }
 
@@ -622,7 +174,7 @@ char *__get_srcline(struct dso *dso, u64 addr, struct symbol *sym,
                goto out_err;
 
        if (!addr2line(dso_name, addr, &file, &line, dso,
-                      unwind_inlines, NULL, sym))
+                      unwind_inlines, /*node=*/NULL, sym))
                goto out_err;
 
        srcline = srcline_from_fileline(file, line);
@@ -668,7 +220,8 @@ char *get_srcline_split(struct dso *dso, u64 addr, unsigned *line)
        if (dso_name == NULL)
                goto out_err;
 
-       if (!addr2line(dso_name, addr, &file, line, dso, true, NULL, NULL))
+       if (!addr2line(dso_name, addr, &file, line, dso, /*unwind_inlines=*/true,
+                       /*node=*/NULL, /*sym=*/NULL))
                goto out_err;
 
        dso__set_a2l_fails(dso, 0);
index 6e66ddbcc879e5fc71628849cb37e8960f7b5579..c36f573cd3390bf783900778d39c637e0997a45d 100644 (file)
@@ -9,7 +9,6 @@
 struct dso;
 struct symbol;
 
-extern int addr2line_timeout_ms;
 extern bool srcline_full_filename;
 char *get_srcline(struct dso *dso, u64 addr, struct symbol *sym,
                  bool show_sym, bool show_addr, u64 ip);