[PATCH v5 02/14] perf evsel: Refactor evsel__set_config_if_unset() arguments

[Ian Rogers]

On Wed, Jan 14, 2026 at 7:57 AM James Clark <james.clark at linaro.org> wrote:
>
> Make the evsel argument first to match the other evsel__* functions
> and remove the redundant pmu argument, which can be accessed via evsel.
>
> Signed-off-by: James Clark <james.clark at linaro.org>

Reviewed-by: Ian Rogers <irogers at google.com>

Thanks,
Ian

> ---
>  tools/perf/arch/arm/util/cs-etm.c    | 9 +++------
>  tools/perf/arch/arm64/util/arm-spe.c | 2 +-
>  tools/perf/arch/x86/util/intel-pt.c  | 3 +--
>  tools/perf/util/evsel.h              | 4 ++--
>  tools/perf/util/pmu.c                | 6 +++---
>  5 files changed, 10 insertions(+), 14 deletions(-)
>
> diff --git a/tools/perf/arch/arm/util/cs-etm.c b/tools/perf/arch/arm/util/cs-etm.c
> index ea891d12f8f4..c28208361d91 100644
> --- a/tools/perf/arch/arm/util/cs-etm.c
> +++ b/tools/perf/arch/arm/util/cs-etm.c
> @@ -441,10 +441,8 @@ static int cs_etm_recording_options(struct auxtrace_record *itr,
>          * when a context switch happened.
>          */
>         if (!perf_cpu_map__is_any_cpu_or_is_empty(cpus)) {
> -               evsel__set_config_if_unset(cs_etm_pmu, cs_etm_evsel,
> -                                          "timestamp", 1);
> -               evsel__set_config_if_unset(cs_etm_pmu, cs_etm_evsel,
> -                                          "contextid", 1);
> +               evsel__set_config_if_unset(cs_etm_evsel, "timestamp", 1);
> +               evsel__set_config_if_unset(cs_etm_evsel, "contextid", 1);
>         }
>
>         /*
> @@ -453,8 +451,7 @@ static int cs_etm_recording_options(struct auxtrace_record *itr,
>          * timestamp tracing.
>          */
>         if (opts->sample_time_set)
> -               evsel__set_config_if_unset(cs_etm_pmu, cs_etm_evsel,
> -                                          "timestamp", 1);
> +               evsel__set_config_if_unset(cs_etm_evsel, "timestamp", 1);
>
>         /* Add dummy event to keep tracking */
>         err = parse_event(evlist, "dummy:u");
> diff --git a/tools/perf/arch/arm64/util/arm-spe.c b/tools/perf/arch/arm64/util/arm-spe.c
> index d5ec1408d0ae..51014f8bff97 100644
> --- a/tools/perf/arch/arm64/util/arm-spe.c
> +++ b/tools/perf/arch/arm64/util/arm-spe.c
> @@ -274,7 +274,7 @@ static void arm_spe_setup_evsel(struct evsel *evsel, struct perf_cpu_map *cpus)
>          */
>         if (!perf_cpu_map__is_any_cpu_or_is_empty(cpus)) {
>                 evsel__set_sample_bit(evsel, CPU);
> -               evsel__set_config_if_unset(evsel->pmu, evsel, "ts_enable", 1);
> +               evsel__set_config_if_unset(evsel, "ts_enable", 1);
>         }
>
>         /*
> diff --git a/tools/perf/arch/x86/util/intel-pt.c b/tools/perf/arch/x86/util/intel-pt.c
> index b394ad9cc635..c131a727774f 100644
> --- a/tools/perf/arch/x86/util/intel-pt.c
> +++ b/tools/perf/arch/x86/util/intel-pt.c
> @@ -664,8 +664,7 @@ static int intel_pt_recording_options(struct auxtrace_record *itr,
>                 return 0;
>
>         if (opts->auxtrace_sample_mode)
> -               evsel__set_config_if_unset(intel_pt_pmu, intel_pt_evsel,
> -                                          "psb_period", 0);
> +               evsel__set_config_if_unset(intel_pt_evsel, "psb_period", 0);
>
>         err = intel_pt_validate_config(intel_pt_pmu, intel_pt_evsel);
>         if (err)
> diff --git a/tools/perf/util/evsel.h b/tools/perf/util/evsel.h
> index a08130ff2e47..2cf87bc67df7 100644
> --- a/tools/perf/util/evsel.h
> +++ b/tools/perf/util/evsel.h
> @@ -575,8 +575,8 @@ void evsel__uniquify_counter(struct evsel *counter);
>         ((((src) >> (pos)) & ((1ull << (size)) - 1)) << (63 - ((pos) + (size) - 1)))
>
>  u64 evsel__bitfield_swap_branch_flags(u64 value);
> -void evsel__set_config_if_unset(struct perf_pmu *pmu, struct evsel *evsel,
> -                               const char *config_name, u64 val);
> +void evsel__set_config_if_unset(struct evsel *evsel, const char *config_name,
> +                               u64 val);
>
>  bool evsel__is_offcpu_event(struct evsel *evsel);
>
> diff --git a/tools/perf/util/pmu.c b/tools/perf/util/pmu.c
> index 956ea273c2c7..e87c12946d71 100644
> --- a/tools/perf/util/pmu.c
> +++ b/tools/perf/util/pmu.c
> @@ -1382,8 +1382,8 @@ bool evsel__is_aux_event(const struct evsel *evsel)
>   * something to true, pass 1 for val rather than a pre shifted value.
>   */
>  #define field_prep(_mask, _val) (((_val) << (ffsll(_mask) - 1)) & (_mask))
> -void evsel__set_config_if_unset(struct perf_pmu *pmu, struct evsel *evsel,
> -                               const char *config_name, u64 val)
> +void evsel__set_config_if_unset(struct evsel *evsel, const char *config_name,
> +                               u64 val)
>  {
>         u64 user_bits = 0, bits;
>         struct evsel_config_term *term = evsel__get_config_term(evsel, CFG_CHG);
> @@ -1391,7 +1391,7 @@ void evsel__set_config_if_unset(struct perf_pmu *pmu, struct evsel *evsel,
>         if (term)
>                 user_bits = term->val.cfg_chg;
>
> -       bits = perf_pmu__format_bits(pmu, config_name);
> +       bits = perf_pmu__format_bits(evsel->pmu, config_name);
>
>         /* Do nothing if the user changed the value */
>         if (bits & user_bits)
>
> --
> 2.34.1
>