30d1c4d947
Tzvetomir Stoyanov reported an issue with using macro
perf_cpu_map__for_each_cpu using private perf_cpu object.
The issue is caused by recent change that wrapped cpu in struct perf_cpu
to distinguish it from cpu indexes. We need to make struct perf_cpu
public.
Add a simple test for using the perf_cpu_map__for_each_cpu macro.
Fixes: 6d18804b963b78dc ("perf cpumap: Give CPUs their own type")
Reported-by: Tzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Ian Rogers <irogers@google.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lore.kernel.org/lkml/20220215153713.31395-1-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
44 lines
789 B
C
44 lines
789 B
C
// SPDX-License-Identifier: GPL-2.0
|
|
#include <stdarg.h>
|
|
#include <stdio.h>
|
|
#include <perf/cpumap.h>
|
|
#include <internal/tests.h>
|
|
#include "tests.h"
|
|
|
|
static int libperf_print(enum libperf_print_level level,
|
|
const char *fmt, va_list ap)
|
|
{
|
|
return vfprintf(stderr, fmt, ap);
|
|
}
|
|
|
|
int test_cpumap(int argc, char **argv)
|
|
{
|
|
struct perf_cpu_map *cpus;
|
|
struct perf_cpu cpu;
|
|
int idx;
|
|
|
|
__T_START;
|
|
|
|
libperf_init(libperf_print);
|
|
|
|
cpus = perf_cpu_map__dummy_new();
|
|
if (!cpus)
|
|
return -1;
|
|
|
|
perf_cpu_map__get(cpus);
|
|
perf_cpu_map__put(cpus);
|
|
perf_cpu_map__put(cpus);
|
|
|
|
cpus = perf_cpu_map__default_new();
|
|
if (!cpus)
|
|
return -1;
|
|
|
|
perf_cpu_map__for_each_cpu(cpu, idx, cpus)
|
|
__T("wrong cpu number", cpu.cpu != -1);
|
|
|
|
perf_cpu_map__put(cpus);
|
|
|
|
__T_END;
|
|
return tests_failed == 0 ? 0 : -1;
|
|
}
|