Re: [PATCH v3 0/8] perf trace: Augment enum arguments with BTF

From: Arnaldo Carvalho de Melo
Date: Thu Jul 04 2024 - 15:23:22 EST


On Sat, Jun 29, 2024 at 08:27:42AM +0800, Howard Chu wrote:
> Hello Namhyung,
>
> Thank you for testing this, I will fix this.

So for this case I further updated the patch introducing the workload to
the patch at the end of this message, but then there is still this
problem I'm looking at:

43 7.54 ubuntu:18.04-x-arm : FAIL gcc version 7.5.0 (Ubuntu/Linaro 7.5.0-3ubuntu1~18.04)
tests/workloads/landlock.c:27:15: error: packed attribute causes inefficient alignment for 'allowed_access' [-Werror=attributes]
__u64 allowed_access;
^~~~~~~~~~~~~~
tests/workloads/landlock.c:28:15: error: packed attribute causes inefficient alignment for 'parent_fd' [-Werror=attributes]
__s32 parent_fd;
^~~~~~~~~
cc1: all warnings being treated as errors
/git/perf-6.10.0-rc3/tools/build/Makefile.build:158: recipe for target 'workloads' failed
make[4]: *** [workloads] Error 2
/git/perf-6.10.0-rc3/tools/build/Makefile.build:158: recipe for target 'tests' failed
make[3]: *** [tests] Error 2

-----------------

Which is real and a problem when building with those landlock patches on
32-bit arches, probably that was considered a non-issue and so I'll just
disable that warning when building this specific file with perf.

- Arnaldo

-----------------