[PATCH v5 09/15] lib: sbi: Add PMU specific platform hooks

[Anup Patel]

On Sat, Jul 10, 2021 at 9:48 PM Atish Patra <atish.patra at wdc.com> wrote:
>
> A platform hook to initialize PMU allows platform vendors to provide
> their own mechanism to define pmu event-counter mappings in addition
> to the DT based approach.
>
> Another platform hook that allows platform vendors customize the
> final mhpmevent value configuration.
>
> Reviewed-by: Anup Patel <anup.patel at wdc.com>
> Signed-off-by: Atish Patra <atish.patra at wdc.com>

Applied this patch to the riscv/opensbi repo.

Thanks,
Anup

> ---
>  include/sbi/sbi_platform.h | 39 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 39 insertions(+)
>
> diff --git a/include/sbi/sbi_platform.h b/include/sbi/sbi_platform.h
> index f8074d28db65..4d192f2a0ec5 100644
> --- a/include/sbi/sbi_platform.h
> +++ b/include/sbi/sbi_platform.h
> @@ -89,6 +89,12 @@ struct sbi_platform_operations {
>         /** Initialize (or populate) domains for the platform */
>         int (*domains_init)(void);
>
> +       /** Initialize hw performance counters */
> +       int (*pmu_init)(void);
> +
> +       /** Get platform specific mhpmevent value */
> +       uint64_t (*pmu_xlate_to_mhpmevent)(uint32_t event_idx, uint64_t data);
> +
>         /** Initialize the platform console */
>         int (*console_init)(void);
>
> @@ -391,6 +397,39 @@ static inline int sbi_platform_domains_init(const struct sbi_platform *plat)
>         return 0;
>  }
>
> +/**
> + * Setup hw PMU events for the platform
> + *
> + * @param plat pointer to struct sbi_platform
> + *
> + * @return 0 on success and negative error code on failure
> + */
> +static inline int sbi_platform_pmu_init(const struct sbi_platform *plat)
> +{
> +       if (plat && sbi_platform_ops(plat)->pmu_init)
> +               return sbi_platform_ops(plat)->pmu_init();
> +       return 0;
> +}
> +
> +/**
> + * Get the value to be written in mhpmeventx for event_idx
> + *
> + * @param plat pointer to struct sbi_platform
> + * @param event_idx ID of the PMU event
> + * @param data Additional configuration data passed from supervisor software
> + *
> + * @return expected value by the platform or 0 if platform doesn't know about
> + * the event
> + */
> +static inline uint64_t sbi_platform_pmu_xlate_to_mhpmevent(const struct sbi_platform *plat,
> +                                                     uint32_t event_idx, uint64_t data)
> +{
> +       if (plat && sbi_platform_ops(plat)->pmu_xlate_to_mhpmevent)
> +               return sbi_platform_ops(plat)->pmu_xlate_to_mhpmevent(event_idx,
> +                                                                     data);
> +       return 0;
> +}
> +
>  /**
>   * Initialize the platform console
>   *
> --
> 2.31.1
>
>
> --
> opensbi mailing list
> opensbi at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi