diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2015-11-19 16:01:48 +0100 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2015-11-19 17:19:15 +0100 |
commit | 721a1f53df6aad3ea941f5fe95519d0d8e02bd65 (patch) | |
tree | 00ba77e268761346d7a38871fd3f6d507576ec12 /tools/perf/tests/bp_signal_overflow.c | |
parent | perf bpf: Use same BPF program if arguments are identical (diff) | |
download | linux-721a1f53df6aad3ea941f5fe95519d0d8e02bd65.tar.xz linux-721a1f53df6aad3ea941f5fe95519d0d8e02bd65.zip |
perf tests: Pass the subtest index to each test routine
Some tests have sub-tests we want to run, so allow passing this.
Wang tried to avoid having to touch all tests, but then, having the
test.func in an anonymous union makes the build fail on older compilers,
like the one in RHEL6, where:
test a = {
.func = foo,
};
fails.
To fix it leave the func pointer in the main structure and pass the subtest
index to all tests, end result function is the same, but we have just one
function pointer, not two, with and without the subtest index as an argument.
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-5genj0ficwdmelpoqlds0u4y@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/tests/bp_signal_overflow.c')
-rw-r--r-- | tools/perf/tests/bp_signal_overflow.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/tests/bp_signal_overflow.c b/tools/perf/tests/bp_signal_overflow.c index e76537724491..89f92fa67cc4 100644 --- a/tools/perf/tests/bp_signal_overflow.c +++ b/tools/perf/tests/bp_signal_overflow.c @@ -58,7 +58,7 @@ static long long bp_count(int fd) #define EXECUTIONS 10000 #define THRESHOLD 100 -int test__bp_signal_overflow(void) +int test__bp_signal_overflow(int subtest __maybe_unused) { struct perf_event_attr pe; struct sigaction sa; |