perf test: Correct hwmon test PMU detection

captain5050 · namhyung · commit fc26637d70d3 · 2024-11-22T13:34:31.000-08:00
Use name to avoid potential other hwmon PMUs. Signed-off-by: Ian Rogers <irogers@google.com> Link: https://lore.kernel.org/r/20241118052638.754981-1-irogers@google.com Signed-off-by: Namhyung Kim <namhyung@kernel.org>
diff --git a/tools/perf/tests/hwmon_pmu.c b/tools/perf/tests/hwmon_pmu.c
@@ -173,7 +173,8 @@ static int do_test(size_t i, bool with_pmu, bool with_alias)
 	}
 
 	evlist__for_each_entry(evlist, evsel) {
-		if (!perf_pmu__is_hwmon(evsel->pmu))
+		if (!evsel->pmu || !evsel->pmu->name ||
+		    strcmp(evsel->pmu->name, "hwmon_a_test_hwmon_pmu"))
 			continue;
 
 		if (evsel->core.attr.config != (u64)test_events[i].config) {

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,8 @@ static int do_test(size_t i, bool with_pmu, bool with_alias)`
`173`	`173`	`}`
`174`	`174`
`175`	`175`	`evlist__for_each_entry(evlist, evsel) {`
`176`		`- if (!perf_pmu__is_hwmon(evsel->pmu))`
	`176`	`+ if (!evsel->pmu \|\| !evsel->pmu->name \|\|`
	`177`	`+ strcmp(evsel->pmu->name, "hwmon_a_test_hwmon_pmu"))`
`177`	`178`	`continue;`
`178`	`179`
`179`	`180`	`if (evsel->core.attr.config != (u64)test_events[i].config) {`