perf tests: Add some pmu core functionality tests
authorIan Rogers <irogers@google.com>
Wed, 15 May 2024 06:01:14 +0000 (23:01 -0700)
committerNamhyung Kim <namhyung@kernel.org>
Tue, 28 May 2024 18:29:49 +0000 (11:29 -0700)
Test behavior of PMU names and comparisons wrt suffixes using Intel
uncore_cha, marvell mrvl_ddr_pmu and S390's cpum_cf as examples.

Signed-off-by: Ian Rogers <irogers@google.com>
Acked-by: Namhyung Kim <namhyung@kernel.org>
Cc: Ravi Bangoria <ravi.bangoria@amd.com>
Cc: James Clark <james.clark@arm.com>
Cc: Robin Murphy <robin.murphy@arm.com>
Cc: Stephane Eranian <eranian@google.com>
Cc: Will Deacon <will@kernel.org>
Cc: Thomas Richter <tmricht@linux.ibm.com>
Cc: Bharat Bhushan <bbhushan2@marvell.com>
Cc: Bhaskara Budiredla <bbudiredla@marvell.com>
Cc: Tuan Phan <tuanphan@os.amperecomputing.com>
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Link: https://lore.kernel.org/r/20240515060114.3268149-3-irogers@google.com
tools/perf/tests/pmu.c

index 06cc0e4..cc88b59 100644 (file)
@@ -3,6 +3,7 @@
 #include "evsel.h"
 #include "parse-events.h"
 #include "pmu.h"
+#include "pmus.h"
 #include "tests.h"
 #include "debug.h"
 #include "fncache.h"
@@ -340,10 +341,108 @@ static int test__pmu_event_names(struct test_suite *test __maybe_unused,
        return ret;
 }
 
+static const char * const uncore_chas[] = {
+       "uncore_cha_0",
+       "uncore_cha_1",
+       "uncore_cha_2",
+       "uncore_cha_3",
+       "uncore_cha_4",
+       "uncore_cha_5",
+       "uncore_cha_6",
+       "uncore_cha_7",
+       "uncore_cha_8",
+       "uncore_cha_9",
+       "uncore_cha_10",
+       "uncore_cha_11",
+       "uncore_cha_12",
+       "uncore_cha_13",
+       "uncore_cha_14",
+       "uncore_cha_15",
+       "uncore_cha_16",
+       "uncore_cha_17",
+       "uncore_cha_18",
+       "uncore_cha_19",
+       "uncore_cha_20",
+       "uncore_cha_21",
+       "uncore_cha_22",
+       "uncore_cha_23",
+       "uncore_cha_24",
+       "uncore_cha_25",
+       "uncore_cha_26",
+       "uncore_cha_27",
+       "uncore_cha_28",
+       "uncore_cha_29",
+       "uncore_cha_30",
+       "uncore_cha_31",
+};
+
+static const char * const mrvl_ddrs[] = {
+       "mrvl_ddr_pmu_87e1b0000000",
+       "mrvl_ddr_pmu_87e1b1000000",
+       "mrvl_ddr_pmu_87e1b2000000",
+       "mrvl_ddr_pmu_87e1b3000000",
+       "mrvl_ddr_pmu_87e1b4000000",
+       "mrvl_ddr_pmu_87e1b5000000",
+       "mrvl_ddr_pmu_87e1b6000000",
+       "mrvl_ddr_pmu_87e1b7000000",
+       "mrvl_ddr_pmu_87e1b8000000",
+       "mrvl_ddr_pmu_87e1b9000000",
+       "mrvl_ddr_pmu_87e1ba000000",
+       "mrvl_ddr_pmu_87e1bb000000",
+       "mrvl_ddr_pmu_87e1bc000000",
+       "mrvl_ddr_pmu_87e1bd000000",
+       "mrvl_ddr_pmu_87e1be000000",
+       "mrvl_ddr_pmu_87e1bf000000",
+};
+
+static int test__name_len(struct test_suite *test __maybe_unused, int subtest __maybe_unused)
+{
+       TEST_ASSERT_VAL("cpu", pmu_name_len_no_suffix("cpu") == strlen("cpu"));
+       TEST_ASSERT_VAL("i915", pmu_name_len_no_suffix("i915") == strlen("i915"));
+       TEST_ASSERT_VAL("cpum_cf", pmu_name_len_no_suffix("cpum_cf") == strlen("cpum_cf"));
+       for (size_t i = 0; i < ARRAY_SIZE(uncore_chas); i++) {
+               TEST_ASSERT_VAL("Strips uncore_cha suffix",
+                               pmu_name_len_no_suffix(uncore_chas[i]) ==
+                               strlen("uncore_cha"));
+       }
+       for (size_t i = 0; i < ARRAY_SIZE(mrvl_ddrs); i++) {
+               TEST_ASSERT_VAL("Strips mrvl_ddr_pmu suffix",
+                               pmu_name_len_no_suffix(mrvl_ddrs[i]) ==
+                               strlen("mrvl_ddr_pmu"));
+       }
+       return TEST_OK;
+}
+
+static int test__name_cmp(struct test_suite *test __maybe_unused, int subtest __maybe_unused)
+{
+       TEST_ASSERT_EQUAL("cpu", pmu_name_cmp("cpu", "cpu"), 0);
+       TEST_ASSERT_EQUAL("i915", pmu_name_cmp("i915", "i915"), 0);
+       TEST_ASSERT_EQUAL("cpum_cf", pmu_name_cmp("cpum_cf", "cpum_cf"), 0);
+       TEST_ASSERT_VAL("i915", pmu_name_cmp("cpu", "i915") < 0);
+       TEST_ASSERT_VAL("i915", pmu_name_cmp("i915", "cpu") > 0);
+       TEST_ASSERT_VAL("cpum_cf", pmu_name_cmp("cpum_cf", "cpum_ce") > 0);
+       TEST_ASSERT_VAL("cpum_cf", pmu_name_cmp("cpum_cf", "cpum_d0") < 0);
+       for (size_t i = 1; i < ARRAY_SIZE(uncore_chas); i++) {
+               TEST_ASSERT_VAL("uncore_cha suffixes ordered lt",
+                               pmu_name_cmp(uncore_chas[i-1], uncore_chas[i]) < 0);
+               TEST_ASSERT_VAL("uncore_cha suffixes ordered gt",
+                               pmu_name_cmp(uncore_chas[i], uncore_chas[i-1]) > 0);
+       }
+       for (size_t i = 1; i < ARRAY_SIZE(mrvl_ddrs); i++) {
+               TEST_ASSERT_VAL("mrvl_ddr_pmu suffixes ordered lt",
+                               pmu_name_cmp(mrvl_ddrs[i-1], mrvl_ddrs[i]) < 0);
+               TEST_ASSERT_VAL("mrvl_ddr_pmu suffixes ordered gt",
+                               pmu_name_cmp(mrvl_ddrs[i], mrvl_ddrs[i-1]) > 0);
+       }
+       return TEST_OK;
+}
+
 static struct test_case tests__pmu[] = {
        TEST_CASE("Parsing with PMU format directory", pmu_format),
        TEST_CASE("Parsing with PMU event", pmu_events),
        TEST_CASE("PMU event names", pmu_event_names),
+       TEST_CASE("PMU name combining", name_len),
+       TEST_CASE("PMU name comparison", name_cmp),
        {       .name = NULL, }
 };