Re: [PATCH] perf lock contention: Change stack_id type to s32

From: kernel test robot
Date: Mon Aug 12 2024 - 10:52:30 EST


Hi Namhyung,

kernel test robot noticed the following build errors:

[auto build test ERROR on perf-tools-next/perf-tools-next]
[also build test ERROR on tip/perf/core perf-tools/perf-tools linus/master v6.11-rc2 next-20240809]
[cannot apply to acme/perf/core]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Namhyung-Kim/perf-lock-contention-Change-stack_id-type-to-s32/20240811-031933
base: https://git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools-next.git perf-tools-next
patch link: https://lore.kernel.org/r/20240810191704.1948365-1-namhyung%40kernel.org
patch subject: [PATCH] perf lock contention: Change stack_id type to s32
:::::: branch date: 23 hours ago
:::::: commit date: 23 hours ago
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240812/202408120233.2JuKgpj9-lkp@xxxxxxxxx/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Closes: https://lore.kernel.org/r/202408120233.2JuKgpj9-lkp@xxxxxxxxx/

All errors (new ones prefixed by >>):

Makefile.config:671: No libunwind found. Please install libunwind-dev[el] >= 1.1 and/or set LIBUNWIND_DIR
PERF_VERSION = 6.11.rc2.gcbe444bb08ed
In file included from util/bpf_skel/lock_contention.bpf.c:9:
>> util/bpf_skel/lock_data.h:10:2: error: unknown type name 's32'; did you mean 'u32'?
10 | s32 stack_id;
| ^~~
| u32
util/bpf_skel/vmlinux.h:17:15: note: 'u32' declared here
17 | typedef __u32 u32;
| ^
In file included from util/bpf_skel/lock_contention.bpf.c:9:
util/bpf_skel/lock_data.h:14:2: error: unknown type name 's32'; did you mean 'u32'?
14 | s32 stack_id;
| ^~~
| u32
util/bpf_skel/vmlinux.h:17:15: note: 'u32' declared here
17 | typedef __u32 u32;
| ^
2 errors generated.
make[5]: *** [Makefile.perf:1247: tools/perf/util/bpf_skel/.tmp/lock_contention.bpf.o] Error 1
make[5]: *** Waiting for unfinished jobs....
make[4]: *** [Makefile.perf:292: sub-make] Error 2
make[3]: *** [Makefile:76: all] Error 2


vim +10 tools/perf/util/bpf_skel/lock_data.h

fd507d3e359c7e Namhyung Kim 2022-12-09 5
b44d6653685939 Namhyung Kim 2024-02-27 6 struct tstamp_data {
b44d6653685939 Namhyung Kim 2024-02-27 7 u64 timestamp;
b44d6653685939 Namhyung Kim 2024-02-27 8 u64 lock;
b44d6653685939 Namhyung Kim 2024-02-27 9 u32 flags;
cbe444bb08ed25 Namhyung Kim 2024-08-10 @10 s32 stack_id;
b44d6653685939 Namhyung Kim 2024-02-27 11 };
b44d6653685939 Namhyung Kim 2024-02-27 12

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki