aboutsummaryrefslogtreecommitdiffstats
path: root/tools/perf/tests/kmod-path.c
diff options
context:
space:
mode:
authorArnaldo Carvalho de Melo <acme@redhat.com>2015-11-19 12:01:48 -0300
committerArnaldo Carvalho de Melo <acme@redhat.com>2015-11-19 13:19:15 -0300
commit721a1f53df6aad3ea941f5fe95519d0d8e02bd65 (patch)
tree00ba77e268761346d7a38871fd3f6d507576ec12 /tools/perf/tests/kmod-path.c
parentperf bpf: Use same BPF program if arguments are identical (diff)
downloadlinux-dev-721a1f53df6aad3ea941f5fe95519d0d8e02bd65.tar.xz
linux-dev-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/kmod-path.c')
-rw-r--r--tools/perf/tests/kmod-path.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/tests/kmod-path.c b/tools/perf/tests/kmod-path.c
index 08c433b4bf4f..d2af78193153 100644
--- a/tools/perf/tests/kmod-path.c
+++ b/tools/perf/tests/kmod-path.c
@@ -49,7 +49,7 @@ static int test_is_kernel_module(const char *path, int cpumode, bool expect)
#define M(path, c, e) \
TEST_ASSERT_VAL("failed", !test_is_kernel_module(path, c, e))
-int test__kmod_path__parse(void)
+int test__kmod_path__parse(int subtest __maybe_unused)
{
/* path alloc_name alloc_ext kmod comp name ext */
T("/xxxx/xxxx/x-x.ko", true , true , true, false, "[x_x]", NULL);