Re: [PATCH v3 0/5] perf: generate events for BPF metadata
From: Blake Jones
Date: Wed Jun 11 2025 - 20:39:33 EST
Hi Namhyung,
On Wed, Jun 11, 2025 at 11:29 AM Namhyung Kim <namhyung@xxxxxxxxxx> wrote:
> I tried to process your patches but it failed to build like below:
> [...]
> Please run 'make build-test' and send v4.
Very sorry about that. I've fixed the two issues you noticed, as well as
one additional one where I was using the wrong include path to check for
the presence of the libbpf-strings feature.
I'm trying to test my fixes using "make build-test", but it's proving a bit
of a challenge. I installed libgtk-4-dev, binutils-dev, and libopencsd-dev
to fix build problems as they came up; I also installed libtraceevent-dev,
but somehow it still wasn't detected by the build process and so I had to
use NO_LIBTRACEEVENT=1.
Even after installing these libraries, I'm still hitting errors when doing
"make build-test" on a copy of the perf source *without* my changes:
In file included from util/disasm_bpf.c:18:
.../tools/include/tools/dis-asm-compat.h:10:6:
error: redeclaration of 'enum disassembler_style'
10 | enum disassembler_style {DISASSEMBLER_STYLE_NOT_EMPTY};
| ^~~~~~~~~~~~~~~~~~
In file included from util/disasm_bpf.c:15:
/usr/include/dis-asm.h:53:6: note: originally defined here
53 | enum disassembler_style
| ^~~~~~~~~~~~~~~~~~
I noticed that tools/perf/BUILD_TEST_FEATURE_DUMP has
"feature-disassembler-four-args=0" and "feature-disassembler-init-styled=0"
as of when this failed, which seems to be upstream of the observed failure
(the version of binutils-dev that I installed seems to have newer-style
versions of these interfaces).
Is there anything written up about how to set up a machine so that
"make build-test" works reliably?
Thanks.
Blake