diff options
author | Ian Rogers <irogers@google.com> | 2023-06-26 07:30:48 +0200 |
---|---|---|
committer | Namhyung Kim <namhyung@kernel.org> | 2023-06-27 21:13:03 +0200 |
commit | 710dffc969023315c3c32717dc08b543012e60d8 (patch) | |
tree | c39207e8991b5925eb7be3349c82c1bcd3203114 /lib/scatterlist.c | |
parent | perf tools: Add missing else to cmd_daemon subcommand condition (diff) | |
download | linux-710dffc969023315c3c32717dc08b543012e60d8.tar.xz linux-710dffc969023315c3c32717dc08b543012e60d8.zip |
perf pmu: Correct auto_merge_stats test
The original logic was to check is_pmu_hybrid() like in the below.
It just checks the name of PMU specifically for Intel hybrid systems
which means uncore PMU events should return false.
https://lore.kernel.org/all/20230527072210.2900565-35-irogers@google.com/
The is_pmu_hybrid() was replaced by arch-agnostic way but with the
incorrect condition which was fixed for core PMUs but not uncore.
This change fixes both.
Fixes: e23421426e13 ("perf pmu: Correct perf_pmu__auto_merge_stats() affecting hybrid")
Signed-off-by: Ian Rogers <irogers@google.com>
Tested-by: Namhyung Kim <namhyung@kernel.org>
Cc: James Clark <james.clark@arm.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Ingo Molnar <mingo@redhat.com>
Link: https://lore.kernel.org/all/CAP-5=fXOi=xQ4=j5xAq+jWLR9n7uvfsWK+PzXkY1MZ3Fz-xccw@mail.gmail.com/
Link: https://lore.kernel.org/r/20230626053048.257959-1-irogers@google.com
[ rephrase the commit log a bit ]
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Diffstat (limited to 'lib/scatterlist.c')
0 files changed, 0 insertions, 0 deletions