Commit 212f3d97 authored by Jin Yao's avatar Jin Yao Committed by Arnaldo Carvalho de Melo

perf tests: Fix 'Roundtrip evsel->name' on core-only system

If the atom CPUs are offlined, the 'cpu_atom' is not valid.
Perf will not create two events for one hw event, so the
evsel->idx doesn't need to be divided by 2 before comparing.
Signed-off-by: default avatarJin Yao <yao.jin@linux.intel.com>
Acked-by: default avatarJiri Olsa <jolsa@redhat.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jin Yao <yao.jin@intel.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lore.kernel.org/lkml/20210708013701.20347-4-yao.jin@linux.intel.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 490e9a8f
...@@ -5,6 +5,7 @@ ...@@ -5,6 +5,7 @@
#include "tests.h" #include "tests.h"
#include "debug.h" #include "debug.h"
#include "pmu.h" #include "pmu.h"
#include "pmu-hybrid.h"
#include <errno.h> #include <errno.h>
#include <linux/kernel.h> #include <linux/kernel.h>
...@@ -102,7 +103,7 @@ int test__perf_evsel__roundtrip_name_test(struct test *test __maybe_unused, int ...@@ -102,7 +103,7 @@ int test__perf_evsel__roundtrip_name_test(struct test *test __maybe_unused, int
{ {
int err = 0, ret = 0; int err = 0, ret = 0;
if (perf_pmu__has_hybrid()) if (perf_pmu__has_hybrid() && perf_pmu__hybrid_mounted("cpu_atom"))
return perf_evsel__name_array_test(evsel__hw_names, 2); return perf_evsel__name_array_test(evsel__hw_names, 2);
err = perf_evsel__name_array_test(evsel__hw_names, 1); err = perf_evsel__name_array_test(evsel__hw_names, 1);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment