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

perf tests: Fix 'Convert perf time to TSC' on core-only system

If the atom CPUs are offlined, the 'cpu_atom' is not valid.
We don't need the test case for 'cpu_atom'.
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-5-yao.jin@linux.intel.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 212f3d97
...@@ -21,6 +21,7 @@ ...@@ -21,6 +21,7 @@
#include "mmap.h" #include "mmap.h"
#include "tests.h" #include "tests.h"
#include "pmu.h" #include "pmu.h"
#include "pmu-hybrid.h"
#define CHECK__(x) { \ #define CHECK__(x) { \
while ((x) < 0) { \ while ((x) < 0) { \
...@@ -93,7 +94,7 @@ int test__perf_time_to_tsc(struct test *test __maybe_unused, int subtest __maybe ...@@ -93,7 +94,7 @@ int test__perf_time_to_tsc(struct test *test __maybe_unused, int subtest __maybe
* For hybrid "cycles:u", it creates two events. * For hybrid "cycles:u", it creates two events.
* Init the second evsel here. * Init the second evsel here.
*/ */
if (perf_pmu__has_hybrid()) { if (perf_pmu__has_hybrid() && perf_pmu__hybrid_mounted("cpu_atom")) {
evsel = evsel__next(evsel); evsel = evsel__next(evsel);
evsel->core.attr.comm = 1; evsel->core.attr.comm = 1;
evsel->core.attr.disabled = 1; evsel->core.attr.disabled = 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