Re: [PATCH v2 0/2] perf pmu: Event parsing and listing fixes

[Ian Rogers]

On Wed, Jun 26, 2024 at 7:55 AM James Clark <james.clark@xxxxxxx> wrote:
>
> The second fix is related to the discussion here [1].
>
> The first fix is unrelated but I just noticed it while fixing the
> listing issue.
>
> [1]: https://lore.kernel.org/all/ce31a50b-53db-4c6f-9cb1-242280b0951c@xxxxxxx/
>
> Changes since v1:
>
>  * Add Ian's suggested-by tag on patch 2
>  * Fix mistake where info.pmu_name instead of pmu->name should have
>    been used in perf_pmu__for_each_event()
>
> James Clark (2):
>   perf pmu: Restore full PMU name wildcard support
>   perf pmu: Don't de-duplicate core PMUs

Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>

Looking forward to an update on the bugs in:
https://lore.kernel.org/lkml/CAP-5=fWxdZ1ibv7Pi0L38+NF-vNWZCtcDS9JVf6k3z9nDsrQQw@xxxxxxxxxxxxxx/

Thanks,
Ian


>  tools/perf/tests/pmu.c | 78 ++++++++++++++++++++++++++++++++++++++++++
>  tools/perf/util/pmu.c  | 29 ++++++++++++----
>  2 files changed, 100 insertions(+), 7 deletions(-)
>
> --
> 2.34.1
>