115 lines
4.2 KiB
Diff
115 lines
4.2 KiB
Diff
From 83aa0120487e8bc3f231e72c460add783f71f17c Mon Sep 17 00:00:00 2001
|
|
From: Andres Freund <andres@anarazel.de>
|
|
Date: Sun, 31 Jul 2022 18:38:30 -0700
|
|
Subject: [PATCH] tools perf: Fix compilation error with new binutils
|
|
|
|
binutils changed the signature of init_disassemble_info(), which now causes
|
|
compilation failures for tools/perf/util/annotate.c, e.g. on debian
|
|
unstable.
|
|
|
|
Relevant binutils commit:
|
|
|
|
https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=60a3da00bd5407f07
|
|
|
|
Wire up the feature test and switch to init_disassemble_info_compat(),
|
|
which were introduced in prior commits, fixing the compilation failure.
|
|
|
|
I verified that perf can still disassemble bpf programs by using bpftrace
|
|
under load, recording a perf trace, and then annotating the bpf "function"
|
|
with and without the changes. With old binutils there's no change in output
|
|
before/after this patch. When comparing the output from old binutils (2.35)
|
|
to new bintuils with the patch (upstream snapshot) there are a few output
|
|
differences, but they are unrelated to this patch. An example hunk is:
|
|
|
|
1.15 : 55:mov %rbp,%rdx
|
|
0.00 : 58:add $0xfffffffffffffff8,%rdx
|
|
0.00 : 5c:xor %ecx,%ecx
|
|
- 1.03 : 5e:callq 0xffffffffe12aca3c
|
|
+ 1.03 : 5e:call 0xffffffffe12aca3c
|
|
0.00 : 63:xor %eax,%eax
|
|
- 2.18 : 65:leaveq
|
|
- 2.82 : 66:retq
|
|
+ 2.18 : 65:leave
|
|
+ 2.82 : 66:ret
|
|
|
|
Signed-off-by: Andres Freund <andres@anarazel.de>
|
|
Acked-by: Quentin Monnet <quentin@isovalent.com>
|
|
Cc: Alexei Starovoitov <ast@kernel.org>
|
|
Cc: Ben Hutchings <benh@debian.org>
|
|
Cc: Jiri Olsa <jolsa@kernel.org>
|
|
Cc: Sedat Dilek <sedat.dilek@gmail.com>
|
|
Cc: bpf@vger.kernel.org
|
|
Link: http://lore.kernel.org/lkml/20220622181918.ykrs5rsnmx3og4sv@alap3.anarazel.de
|
|
Link: https://lore.kernel.org/r/20220801013834.156015-5-andres@anarazel.de
|
|
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
|
|
---
|
|
tools/perf/Makefile.config | 8 ++++++++
|
|
tools/perf/util/annotate.c | 7 ++++---
|
|
2 files changed, 12 insertions(+), 3 deletions(-)
|
|
|
|
diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config
|
|
index d3c254c0f5c611..23648ea54e8d3d 100644
|
|
--- a/tools/perf/Makefile.config
|
|
+++ b/tools/perf/Makefile.config
|
|
@@ -298,6 +298,7 @@ FEATURE_CHECK_LDFLAGS-libpython := $(PYTHON_EMBED_LDOPTS)
|
|
FEATURE_CHECK_LDFLAGS-libaio = -lrt
|
|
|
|
FEATURE_CHECK_LDFLAGS-disassembler-four-args = -lbfd -lopcodes -ldl
|
|
+FEATURE_CHECK_LDFLAGS-disassembler-init-styled = -lbfd -lopcodes -ldl
|
|
|
|
CORE_CFLAGS += -fno-omit-frame-pointer
|
|
CORE_CFLAGS += -ggdb3
|
|
@@ -818,13 +819,16 @@ else
|
|
ifeq ($(feature-libbfd-liberty), 1)
|
|
EXTLIBS += -lbfd -lopcodes -liberty
|
|
FEATURE_CHECK_LDFLAGS-disassembler-four-args += -liberty -ldl
|
|
+ FEATURE_CHECK_LDFLAGS-disassembler-init-styled += -liberty -ldl
|
|
else
|
|
ifeq ($(feature-libbfd-liberty-z), 1)
|
|
EXTLIBS += -lbfd -lopcodes -liberty -lz
|
|
FEATURE_CHECK_LDFLAGS-disassembler-four-args += -liberty -lz -ldl
|
|
+ FEATURE_CHECK_LDFLAGS-disassembler-init-styled += -liberty -lz -ldl
|
|
endif
|
|
endif
|
|
$(call feature_check,disassembler-four-args)
|
|
+ $(call feature_check,disassembler-init-styled)
|
|
endif
|
|
|
|
ifeq ($(feature-libbfd-buildid), 1)
|
|
@@ -1044,6 +1048,10 @@ ifeq ($(feature-disassembler-four-args), 1)
|
|
CFLAGS += -DDISASM_FOUR_ARGS_SIGNATURE
|
|
endif
|
|
|
|
+ifeq ($(feature-disassembler-init-styled), 1)
|
|
+ CFLAGS += -DDISASM_INIT_STYLED
|
|
+endif
|
|
+
|
|
ifeq (${IS_64_BIT}, 1)
|
|
ifndef NO_PERF_READ_VDSO32
|
|
$(call feature_check,compile-32)
|
|
diff --git a/tools/perf/util/annotate.c b/tools/perf/util/annotate.c
|
|
index 82cc396ef516c4..2c6a485c3de5d9 100644
|
|
--- a/tools/perf/util/annotate.c
|
|
+++ b/tools/perf/util/annotate.c
|
|
@@ -1676,6 +1676,7 @@ fallback:
|
|
#define PACKAGE "perf"
|
|
#include <bfd.h>
|
|
#include <dis-asm.h>
|
|
+#include <tools/dis-asm-compat.h>
|
|
|
|
static int symbol__disassemble_bpf(struct symbol *sym,
|
|
struct annotate_args *args)
|
|
@@ -1762,9 +1763,9 @@ static int symbol__disassemble_bpf(struct symbol *sym,
|
|
ret = errno;
|
|
goto out;
|
|
}
|
|
- init_disassemble_info(&info, s,
|
|
- (fprintf_ftype) fprintf);
|
|
-
|
|
+ init_disassemble_info_compat(&info, s,
|
|
+ (fprintf_ftype) fprintf,
|
|
+ fprintf_styled);
|
|
info.arch = bfd_get_arch(bfdf);
|
|
info.mach = bfd_get_mach(bfdf);
|
|
|