linux-kernel - Re: [PATCH v9 0/5] powerpc/perf: Add json file support for hv

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200907100529.GF1199773@krava>
Date:   Mon, 7 Sep 2020 12:05:29 +0200
From:   Jiri Olsa <jolsa@...hat.com>
To:     Kajol Jain <kjain@...ux.ibm.com>
Cc:     acme@...nel.org, peterz@...radead.org, mingo@...hat.com,
        mark.rutland@....com, alexander.shishkin@...ux.intel.com,
        pc@...ibm.com, namhyung@...nel.org, ak@...ux.intel.com,
        yao.jin@...ux.intel.com, linux-kernel@...r.kernel.org,
        linux-perf-users@...r.kernel.org, irogers@...gle.com,
        maddy@...ux.ibm.com, ravi.bangoria@...ux.ibm.com,
        john.garry@...wei.com
Subject: Re: [PATCH v9 0/5] powerpc/perf: Add json file support for hv_24x7
 core level events

On Mon, Sep 07, 2020 at 12:11:28PM +0530, Kajol Jain wrote:
> Patchset enhance current runtime parameter support. It introduces new
> fields like "PerChip" and "PerCore" similar to the field "PerPkg" which is
> used to specify perpkg events.
> 
> The "PerCore" and "PerChip" specifies whether its core or chip events.
> Based on which we can decide which runtime parameter user want to
> access. Now character  '?' can refers different parameter based on user
> requirement.
> 
> Initially, every time we want to add new terms like chip, core, thread
> etc, we need to create corrsponding fields in pmu_events and event
> struct.
> This patchset adds an enum called 'aggr_mode_class' which store all these
> aggregation like perchip/percore. It also adds new field 'AggregationMode'
> to capture these terms.
> Now, if user wants to add any new term, they just need to add it in
> the enum defined.
> 
> This patchset also adds  changes of adding new structure
> called 'json_event' inside jevents.c
> file to improve the callback prototype inside jevent file.
> Initially, whenever user want to add new field, they need to update
> in all function callback which makes it more and more complex with
> increased number of parmeters.
> With this change, we just need to add it in new structure 'json_event'.
> link to the RFC patch: https://lkml.org/lkml/2020/8/25/217
> 
> Changelog:
> v8 -> v9
> - Free aggr_mode memory [Jiri Olsa].

Acked-by: Jiri Olsa <jolsa@...hat.com>

thanks,
jirka

> 
> v7 -> v8
> - Change commit typo from jevents.h to jevents.c
> - Make json_events function static in first patch.
> 
> v6 -> v7
> - Remove min and get_cpu_str functions from jevents.c
> - Make json_events function static to solve warning part [John Garry].
> - Add event attribute in json_events function to remove strcpy from
>   real_event function, as suggested by Jiri Olsa.
> - Add warning while checking aggr_mode value [Jiri Olsa].
> 
> Kajol Jain (5):
>   perf/jevents: Remove jevents.h file
>   perf/jevents: Add new structure to pass json fields.
>   perf jevents: Add support for parsing perchip/percore events
>   perf/tools: Pass pmu_event structure as a parameter for
>     arch_get_runtimeparam
>   perf/tools/pmu_events/powerpc: Add hv_24x7 core level metric events
> 
>  tools/perf/arch/powerpc/util/header.c         |   7 +-
>  .../arch/powerpc/power9/nest_metrics.json     |  35 ++-
>  tools/perf/pmu-events/jevents.c               | 234 +++++++++---------
>  tools/perf/pmu-events/jevents.h               |  23 --
>  tools/perf/pmu-events/pmu-events.h            |   6 +
>  tools/perf/util/metricgroup.c                 |   5 +-
>  tools/perf/util/metricgroup.h                 |   3 +-
>  7 files changed, 154 insertions(+), 159 deletions(-)
>  delete mode 100644 tools/perf/pmu-events/jevents.h
> 
> -- 
> 2.26.2
>