diff options
| author | Namhyung Kim <namhyung@kernel.org> | 2022-09-26 14:56:38 -0700 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2022-10-06 08:03:51 -0300 |
| commit | 433b31fa00797a2a6205a023e9345f2c5e7896b6 (patch) | |
| tree | 3d2f8965b8631efef0dc697ca2b437b553a8d6c3 /tools | |
| parent | c7c43e38b236eb80ae6ee60d3dffd8f894cd751c (diff) | |
perf lock contention: Fix a build error on 32-bit
It was reported that it failed to build the BPF lock contention skeleton
on 32 bit arch due to the size of long. The lost count is used only for
reporting errors due to lack of stackmap space through bad_hist which
type is 'int'. Let's use int type then.
Fixes: 6d499a6b3d90277d ("perf lock: Print the number of lost entries for BPF")
Reported-by: Jiri Slaby <jirislaby@kernel.org>
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Song Liu <songliubraving@fb.com>
Link: http://lore.kernel.org/lkml/20220926215638.3931222-1-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/perf/util/bpf_skel/lock_contention.bpf.c | 2 | ||||
| -rw-r--r-- | tools/perf/util/lock-contention.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/tools/perf/util/bpf_skel/lock_contention.bpf.c b/tools/perf/util/bpf_skel/lock_contention.bpf.c index e107d71f0f1a..1bb8628e7c9f 100644 --- a/tools/perf/util/bpf_skel/lock_contention.bpf.c +++ b/tools/perf/util/bpf_skel/lock_contention.bpf.c @@ -75,7 +75,7 @@ int has_task; int stack_skip; /* error stat */ -unsigned long lost; +int lost; static inline int can_record(void) { diff --git a/tools/perf/util/lock-contention.h b/tools/perf/util/lock-contention.h index 67db311fc9df..b8cb8830b7bc 100644 --- a/tools/perf/util/lock-contention.h +++ b/tools/perf/util/lock-contention.h @@ -114,7 +114,7 @@ struct lock_contention { struct machine *machine; struct hlist_head *result; unsigned long map_nr_entries; - unsigned long lost; + int lost; int max_stack; int stack_skip; }; |