Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

perf cpumap: Add checking for reference counter

For the CPU map merging test, add an extra check for the reference
counter before releasing the last CPU map.

Reviewed-by: Adrian Hunter <adrian.hunter@intel.com>
Signed-off-by: Leo Yan <leo.yan@arm.com>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Ian Rogers <irogers@google.com>
Cc: James Clark <james.clark@linaro.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: https://lore.kernel.org/r/20241107125308.41226-4-leo.yan@arm.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

authored by

Leo Yan and committed by
Arnaldo Carvalho de Melo
9eef3ec9 fb953dfa

+9
+9
tools/perf/tests/cpumap.c
··· 167 167 cpu_map__snprint(a, buf, sizeof(buf)); 168 168 TEST_ASSERT_VAL("failed to merge map: bad result", !strcmp(buf, expected)); 169 169 perf_cpu_map__put(b); 170 + 171 + /* 172 + * If 'b' is a superset of 'a', 'a' points to the same map with the 173 + * map 'b'. In this case, the owner 'b' has released the resource above 174 + * but 'a' still keeps the ownership, the reference counter should be 1. 175 + */ 176 + TEST_ASSERT_VAL("unexpected refcnt: bad result", 177 + refcount_read(perf_cpu_map__refcnt(a)) == 1); 178 + 170 179 perf_cpu_map__put(a); 171 180 return 0; 172 181 }