On Thu, Dec 26, 2024 at 06:14:12PM -0300, Arnaldo Carvalho de Melo wrote:
> On Thu, Dec 26, 2024 at 06:11:57PM -0300, Arnaldo Carvalho de Melo wrote:
> > On Thu, Dec 12, 2024 at 07:50:57PM +0000, Alan Maguire wrote:
> > > On 11/12/2024 02:12, Eduard Zingerman wrote:
> > > > When using BTF as a source, pfunct should now be able to print
> > > > btf_decl_tags for programs like below:
> > > >
> > > > #define __tag(x) __attribute__((btf_decl_tag(#x)))
> > > > __tag(a) __tag(b) void foo(void) {}
> > > >
> > > > This situation arises after recent kernel changes, where tags 'kfunc'
> > > > and 'bpf_fastcall' are added to some functions. To avoid dependency on
> > > > a recent kernel version test this by compiling a small C program using
> > > > clang with --target=bpf, which would instruct clang to generate .BTF
> > > > section.
> > > >
> > > > Signed-off-by: Eduard Zingerman <eddyz87@xxxxxxxxx>
> > >
> > > nit: the test is great but it would be good to print out a description
> > > even in non-verbose mode; when I run it via ./tests I see
> > >
> > > 5: Ok
> > >
> > > could we just echo the comment below, i.e.
> > >
> > > 5 : Check that pfunct can print btf_decl_tags read from BTF: Ok
> > >
> > > ?
>
> To clarify, I'm doing as Alan suggests and adding that message when the
> test succeeds.
Running just this test:
root@number:/home/acme/git/pahole# tests/pfunct-btf-decl-tags.sh
Check that pfunct can print btf_decl_tags read from BTF: Ok
root@number:/home/acme/git/pahole#
> > > Reviewed-by: Alan Maguire <alan.maguire@xxxxxxxxxx>
> > > Tested-by: Alan Maguire <alan.maguire@xxxxxxxxxx>
> >
> > Thanks, applied.
> >
> > - Arnaldo
> >
