perf test: BP tests, remove is_supported use
authorIan Rogers <irogers@google.com>
Thu, 4 Nov 2021 06:42:05 +0000 (23:42 -0700)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Sat, 13 Nov 2021 21:11:50 +0000 (18:11 -0300)
Migrate the is_supported functionality to returning TEST_SKIP.
Motivation is kunit has no is_supported function.

Signed-off-by: Ian Rogers <irogers@google.com>
Tested-by: Sohaib Mohamed <sohaib.amhmd@gmail.com>
Acked-by: Jiri Olsa <jolsa@redhat.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Brendan Higgins <brendanhiggins@google.com>
Cc: Daniel Latypov <dlatypov@google.com>
Cc: David Gow <davidgow@google.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jin Yao <yao.jin@linux.intel.com>
Cc: John Garry <john.garry@huawei.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paul Clarke <pc@us.ibm.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Link: https://lore.kernel.org/r/20211104064208.3156807-20-irogers@google.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/tests/bp_account.c
tools/perf/tests/bp_signal.c
tools/perf/tests/bp_signal_overflow.c
tools/perf/tests/tests.h

index e4f7b63..d1ebb55 100644 (file)
 #include "../perf-sys.h"
 #include "cloexec.h"
 
+/*
+ * PowerPC and S390 do not support creation of instruction breakpoints using the
+ * perf_event interface.
+ *
+ * Just disable the test for these architectures until these issues are
+ * resolved.
+ */
+#if defined(__powerpc__) || defined(__s390x__)
+#define BP_ACCOUNT_IS_SUPPORTED 0
+#else
+#define BP_ACCOUNT_IS_SUPPORTED 1
+#endif
+
 static volatile long the_var;
 
 static noinline int test_function(void)
@@ -180,6 +193,11 @@ static int test__bp_accounting(struct test_suite *test __maybe_unused, int subte
        int bp_cnt = detect_cnt(true);
        int share  = detect_share(wp_cnt, bp_cnt);
 
+       if (!BP_ACCOUNT_IS_SUPPORTED) {
+               pr_debug("Test not supported on this architecture");
+               return TEST_SKIP;
+       }
+
        pr_debug("watchpoints count %d, breakpoints count %d, has_ioctl %d, share %d\n",
                 wp_cnt, bp_cnt, has_ioctl, share);
 
@@ -189,29 +207,4 @@ static int test__bp_accounting(struct test_suite *test __maybe_unused, int subte
        return bp_accounting(wp_cnt, share);
 }
 
-static bool test__bp_account_is_supported(void)
-{
-       /*
-        * PowerPC and S390 do not support creation of instruction
-        * breakpoints using the perf_event interface.
-        *
-        * Just disable the test for these architectures until these
-        * issues are resolved.
-        */
-#if defined(__powerpc__) || defined(__s390x__)
-       return false;
-#else
-       return true;
-#endif
-}
-
-static struct test_case bp_accounting_tests[] = {
-       TEST_CASE("Breakpoint accounting", bp_accounting),
-       { .name = NULL, }
-};
-
-struct test_suite suite__bp_accounting = {
-       .desc = "Breakpoint accounting",
-       .test_cases = bp_accounting_tests,
-       .is_supported = test__bp_account_is_supported,
-};
+DEFINE_SUITE("Breakpoint accounting", bp_accounting);
index 1676e3a..1f2908f 100644 (file)
@@ -166,6 +166,11 @@ static int test__bp_signal(struct test_suite *test __maybe_unused, int subtest _
        struct sigaction sa;
        long long count1, count2, count3;
 
+       if (!BP_SIGNAL_IS_SUPPORTED) {
+               pr_debug("Test not supported on this architecture");
+               return TEST_SKIP;
+       }
+
        /* setup SIGIO signal handler */
        memset(&sa, 0, sizeof(struct sigaction));
        sa.sa_sigaction = (void *) sig_handler;
@@ -285,40 +290,4 @@ static int test__bp_signal(struct test_suite *test __maybe_unused, int subtest _
                TEST_OK : TEST_FAIL;
 }
 
-bool test__bp_signal_is_supported(void)
-{
-       /*
-        * PowerPC and S390 do not support creation of instruction
-        * breakpoints using the perf_event interface.
-        *
-        * ARM requires explicit rounding down of the instruction
-        * pointer in Thumb mode, and then requires the single-step
-        * to be handled explicitly in the overflow handler to avoid
-        * stepping into the SIGIO handler and getting stuck on the
-        * breakpointed instruction.
-        *
-        * Since arm64 has the same issue with arm for the single-step
-        * handling, this case also gets stuck on the breakpointed
-        * instruction.
-        *
-        * Just disable the test for these architectures until these
-        * issues are resolved.
-        */
-#if defined(__powerpc__) || defined(__s390x__) || defined(__arm__) || \
-    defined(__aarch64__)
-       return false;
-#else
-       return true;
-#endif
-}
-
-static struct test_case bp_signal_tests[] = {
-       TEST_CASE("Breakpoint overflow signal handler", bp_signal),
-       { .name = NULL, }
-};
-
-struct test_suite suite__bp_signal = {
-       .desc = "Breakpoint overflow signal handler",
-       .test_cases = bp_signal_tests,
-       .is_supported = test__bp_signal_is_supported,
-};
+DEFINE_SUITE("Breakpoint overflow signal handler", bp_signal);
index bc1f138..4e897c2 100644 (file)
@@ -66,6 +66,11 @@ static int test__bp_signal_overflow(struct test_suite *test __maybe_unused, int
        long long count;
        int fd, i, fails = 0;
 
+       if (!BP_SIGNAL_IS_SUPPORTED) {
+               pr_debug("Test not supported on this architecture");
+               return TEST_SKIP;
+       }
+
        /* setup SIGIO signal handler */
        memset(&sa, 0, sizeof(struct sigaction));
        sa.sa_sigaction = (void *) sig_handler;
@@ -134,13 +139,4 @@ static int test__bp_signal_overflow(struct test_suite *test __maybe_unused, int
        return fails ? TEST_FAIL : TEST_OK;
 }
 
-static struct test_case bp_signal_overflow_tests[] = {
-       TEST_CASE("Breakpoint overflow sampling", bp_signal_overflow),
-       { .name = NULL, }
-};
-
-struct test_suite suite__bp_signal_overflow = {
-       .desc = "Breakpoint overflow sampling",
-       .test_cases = bp_signal_overflow_tests,
-       .is_supported = test__bp_signal_is_supported,
-};
+DEFINE_SUITE("Breakpoint overflow sampling", bp_signal_overflow);
index cae33c3..958f94d 100644 (file)
@@ -151,7 +151,26 @@ DECLARE_SUITE(expand_cgroup_events);
 DECLARE_SUITE(perf_time_to_tsc);
 DECLARE_SUITE(dlfilter);
 
-bool test__bp_signal_is_supported(void);
+/*
+ * PowerPC and S390 do not support creation of instruction breakpoints using the
+ * perf_event interface.
+ *
+ * ARM requires explicit rounding down of the instruction pointer in Thumb mode,
+ * and then requires the single-step to be handled explicitly in the overflow
+ * handler to avoid stepping into the SIGIO handler and getting stuck on the
+ * breakpointed instruction.
+ *
+ * Since arm64 has the same issue with arm for the single-step handling, this
+ * case also gets stuck on the breakpointed instruction.
+ *
+ * Just disable the test for these architectures until these issues are
+ * resolved.
+ */
+#if defined(__powerpc__) || defined(__s390x__) || defined(__arm__) || defined(__aarch64__)
+#define BP_SIGNAL_IS_SUPPORTED 0
+#else
+#define BP_SIGNAL_IS_SUPPORTED 1
+#endif
 
 #ifdef HAVE_DWARF_UNWIND_SUPPORT
 struct thread;