lkml.org 
[lkml]   [2022]   [Oct]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 04/19] perf stat: Add aggr id for global mode
    Date
    To make the code simpler, I'd like to use the same aggregation code for
    the global mode. We can simply add an id function to return cpu 0 and
    use print_aggr().

    No functional change intended.

    Signed-off-by: Namhyung Kim <namhyung@kernel.org>
    ---
    tools/perf/builtin-stat.c | 36 ++++++++++++++++++++++++++++++++++--
    tools/perf/util/cpumap.c | 10 ++++++++++
    tools/perf/util/cpumap.h | 6 +++++-
    3 files changed, 49 insertions(+), 3 deletions(-)

    diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c
    index 265b05157972..75d16e9705a4 100644
    --- a/tools/perf/builtin-stat.c
    +++ b/tools/perf/builtin-stat.c
    @@ -1330,6 +1330,12 @@ static struct aggr_cpu_id perf_stat__get_node(struct perf_stat_config *config __
    return aggr_cpu_id__node(cpu, /*data=*/NULL);
    }

    +static struct aggr_cpu_id perf_stat__get_global(struct perf_stat_config *config __maybe_unused,
    + struct perf_cpu cpu)
    +{
    + return aggr_cpu_id__global(cpu, /*data=*/NULL);
    +}
    +
    static struct aggr_cpu_id perf_stat__get_aggr(struct perf_stat_config *config,
    aggr_get_id_t get_id, struct perf_cpu cpu)
    {
    @@ -1366,6 +1372,12 @@ static struct aggr_cpu_id perf_stat__get_node_cached(struct perf_stat_config *co
    return perf_stat__get_aggr(config, perf_stat__get_node, cpu);
    }

    +static struct aggr_cpu_id perf_stat__get_global_cached(struct perf_stat_config *config,
    + struct perf_cpu cpu)
    +{
    + return perf_stat__get_aggr(config, perf_stat__get_global, cpu);
    +}
    +
    static bool term_percore_set(void)
    {
    struct evsel *counter;
    @@ -1395,6 +1407,7 @@ static aggr_cpu_id_get_t aggr_mode__get_aggr(enum aggr_mode aggr_mode)

    return NULL;
    case AGGR_GLOBAL:
    + return aggr_cpu_id__global;
    case AGGR_THREAD:
    case AGGR_UNSET:
    case AGGR_MAX:
    @@ -1420,6 +1433,7 @@ static aggr_get_id_t aggr_mode__get_id(enum aggr_mode aggr_mode)
    }
    return NULL;
    case AGGR_GLOBAL:
    + return perf_stat__get_global_cached;
    case AGGR_THREAD:
    case AGGR_UNSET:
    case AGGR_MAX:
    @@ -1535,6 +1549,16 @@ static struct aggr_cpu_id perf_env__get_node_aggr_by_cpu(struct perf_cpu cpu, vo
    return id;
    }

    +static struct aggr_cpu_id perf_env__get_global_aggr_by_cpu(struct perf_cpu cpu __maybe_unused,
    + void *data __maybe_unused)
    +{
    + struct aggr_cpu_id id = aggr_cpu_id__empty();
    +
    + /* it always aggregates to the cpu 0 */
    + id.cpu = (struct perf_cpu){ .cpu = 0 };
    + return id;
    +}
    +
    static struct aggr_cpu_id perf_stat__get_socket_file(struct perf_stat_config *config __maybe_unused,
    struct perf_cpu cpu)
    {
    @@ -1558,6 +1582,12 @@ static struct aggr_cpu_id perf_stat__get_node_file(struct perf_stat_config *conf
    return perf_env__get_node_aggr_by_cpu(cpu, &perf_stat.session->header.env);
    }

    +static struct aggr_cpu_id perf_stat__get_global_file(struct perf_stat_config *config __maybe_unused,
    + struct perf_cpu cpu)
    +{
    + return perf_env__get_global_aggr_by_cpu(cpu, &perf_stat.session->header.env);
    +}
    +
    static aggr_cpu_id_get_t aggr_mode__get_aggr_file(enum aggr_mode aggr_mode)
    {
    switch (aggr_mode) {
    @@ -1569,8 +1599,9 @@ static aggr_cpu_id_get_t aggr_mode__get_aggr_file(enum aggr_mode aggr_mode)
    return perf_env__get_core_aggr_by_cpu;
    case AGGR_NODE:
    return perf_env__get_node_aggr_by_cpu;
    - case AGGR_NONE:
    case AGGR_GLOBAL:
    + return perf_env__get_global_aggr_by_cpu;
    + case AGGR_NONE:
    case AGGR_THREAD:
    case AGGR_UNSET:
    case AGGR_MAX:
    @@ -1590,8 +1621,9 @@ static aggr_get_id_t aggr_mode__get_id_file(enum aggr_mode aggr_mode)
    return perf_stat__get_core_file;
    case AGGR_NODE:
    return perf_stat__get_node_file;
    - case AGGR_NONE:
    case AGGR_GLOBAL:
    + return perf_stat__get_global_file;
    + case AGGR_NONE:
    case AGGR_THREAD:
    case AGGR_UNSET:
    case AGGR_MAX:
    diff --git a/tools/perf/util/cpumap.c b/tools/perf/util/cpumap.c
    index 8486ca3bec75..60209fe87456 100644
    --- a/tools/perf/util/cpumap.c
    +++ b/tools/perf/util/cpumap.c
    @@ -354,6 +354,16 @@ struct aggr_cpu_id aggr_cpu_id__node(struct perf_cpu cpu, void *data __maybe_unu
    return id;
    }

    +struct aggr_cpu_id aggr_cpu_id__global(struct perf_cpu cpu, void *data __maybe_unused)
    +{
    + struct aggr_cpu_id id = aggr_cpu_id__empty();
    +
    + /* it always aggregates to the cpu 0 */
    + cpu.cpu = 0;
    + id.cpu = cpu;
    + return id;
    +}
    +
    /* setup simple routines to easily access node numbers given a cpu number */
    static int get_max_num(char *path, int *max)
    {
    diff --git a/tools/perf/util/cpumap.h b/tools/perf/util/cpumap.h
    index 4a6d029576ee..b2ff648bc417 100644
    --- a/tools/perf/util/cpumap.h
    +++ b/tools/perf/util/cpumap.h
    @@ -133,5 +133,9 @@ struct aggr_cpu_id aggr_cpu_id__cpu(struct perf_cpu cpu, void *data);
    * cpu. The function signature is compatible with aggr_cpu_id_get_t.
    */
    struct aggr_cpu_id aggr_cpu_id__node(struct perf_cpu cpu, void *data);
    -
    +/**
    + * aggr_cpu_id__global - Create an aggr_cpu_id for global aggregation.
    + * The function signature is compatible with aggr_cpu_id_get_t.
    + */
    +struct aggr_cpu_id aggr_cpu_id__global(struct perf_cpu cpu, void *data);
    #endif /* __PERF_CPUMAP_H */
    --
    2.38.0.413.g74048e4d9e-goog
    \
     
     \ /
      Last update: 2022-10-14 08:17    [W:4.090 / U:0.056 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site