Re: [PATCH v2 0/5] perf trace: Augment enum arguments with BTF

From: Namhyung Kim
Date: Wed Jun 19 2024 - 14:19:58 EST


Hello,

On Wed, Jun 19, 2024 at 9:55 AM Arnaldo Carvalho de Melo
<acme@xxxxxxxxxx> wrote:
>
> On Wed, Jun 19, 2024 at 04:20:37PM +0800, Howard Chu wrote:
> > changes in v2:
> > - Move inline landlock_add_rule c code to tests/workloads
> > - Rename 'enum_aug_prereq' to 'check_vmlinux'
>
> Usually the versions descriptions comes at the end, after your signature
> line, just before the list of csets in the series.
>
> > Augment enum arguments in perf trace, including syscall arguments and
> > non-syscall tracepoint arguments. The augmentation is implemented using
> > BTF.
> >
> > This patch series also includes a bug fix by Arnaldo Carvalho de Melo
> > <acme@xxxxxxxxxx>, which makes more syscalls to be traceable by perf trace.
> >
> > Test is included.
>
> Thanks, the patch submission is now very good, at some point you'll be
> able to point to a git tree from where to do a pull, then have it with a
> signed tag, etc, all this is not necessary at this point in our
> collaboration, but as you evolve as a kernel developer, it eventually
> will be asked from you.
>
> And it comes with a test that introduces a 'perf test -w' workload,
> super great!
>
> - Arnaldo
>
> > Howard Chu (5):
> > perf trace: Fix iteration of syscall ids in syscalltbl->entries
> > perf trace: Augment enum syscall arguments with BTF
> > perf trace: Augment enum tracepoint arguments with BTF
> > perf trace: Filter enum arguments with enum names
> > perf trace: Add test for enum augmentation

Please make sure that your change doesn't break the build
in case libbpf is not available. For example, a build without
libelf seems to be broken.

$ make NO_LIBELF=1

Thanks,
Namhyung