diff options
author | Jiri Olsa <jolsa@redhat.com> | 2017-06-01 22:54:50 +0200 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2017-06-05 19:18:01 +0200 |
commit | 598762cf91532e6e86dd21199b5e7f803df73f49 (patch) | |
tree | 7579f70cc4d7f76b08df7db3a32f4c6eef1cb17e /tools/perf/tests/bp_signal.c | |
parent | perf symbols: Use correct filename for compressed modules in build-id cache (diff) | |
download | linux-598762cf91532e6e86dd21199b5e7f803df73f49.tar.xz linux-598762cf91532e6e86dd21199b5e7f803df73f49.zip |
perf test: Disable breakpoint signal tests for powerpc
The following tests are failing on powerpc:
# perf test break
18: Breakpoint overflow signal handler : FAILED!
19: Breakpoint overflow sampling : FAILED!
The powerpc kenel so far does not have support to even create
instruction breakpoints using the perf event interface, so those tests
fail early in the config phase.
I added a '->is_supported()' callback to test struct to be able to
disable specific tests. It seems better than putting ifdefs directly to
the test array.
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/20170601205450.GA398@krava
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/tests/bp_signal.c')
-rw-r--r-- | tools/perf/tests/bp_signal.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/tools/perf/tests/bp_signal.c b/tools/perf/tests/bp_signal.c index e7664fe3bd33..8ba2c4618fe9 100644 --- a/tools/perf/tests/bp_signal.c +++ b/tools/perf/tests/bp_signal.c @@ -288,3 +288,17 @@ int test__bp_signal(int subtest __maybe_unused) return count1 == 1 && overflows == 3 && count2 == 3 && overflows_2 == 3 && count3 == 2 ? TEST_OK : TEST_FAIL; } + +bool test__bp_signal_is_supported(void) +{ +/* + * The powerpc so far does not have support to even create + * instruction breakpoint using the perf event interface. + * Once it's there we can release this. + */ +#ifdef __powerpc__ + return false; +#else + return true; +#endif +} |