diff options
author | Jin Yao <yao.jin@linux.intel.com> | 2021-05-14 20:29:47 +0800 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2021-05-17 10:55:10 -0300 |
commit | f7d74ce32fc1b9b3cbf58c015009d1f616e60c23 (patch) | |
tree | c1cf448b3a33bab5d2da68e7678faea9ac5dfbe7 /tools/perf/util/env.c | |
parent | 1ac9e0b5731ad732ddc045cfcfd6739e4e12f3b5 (diff) | |
download | lwn-f7d74ce32fc1b9b3cbf58c015009d1f616e60c23.tar.gz lwn-f7d74ce32fc1b9b3cbf58c015009d1f616e60c23.zip |
perf header: Support HYBRID_TOPOLOGY feature
It is useful to let the user know about the hybrid topology.
Add the HYBRID_TOPOLOGY feature in header to indicate the core CPUs
and the atom CPUs.
With this patch a perf.data generated on a hybrid platform reports
the hybrid CPU list:
root@otcpl-adl-s-2:~# perf report --header-only -I
...
# hybrid cpu system:
# cpu_core cpu list : 0-15
# cpu_atom cpu list : 16-23
For a perf.data generated on a non-hybrid platform, reports a message
that HYBRID_TOPOLOGY is missing:
root@kbl-ppc:~# perf report --header-only -I
...
# missing features: TRACING_DATA BRANCH_STACK GROUP_DESC AUXTRACE STAT CLOCKID DIR_FORMAT COMPRESSED CLOCK_DATA HYBRID_TOPOLOGY
Signed-off-by: Jin Yao <yao.jin@linux.intel.com>
Acked-by: Jiri Olsa <jolsa@redhat.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jin Yao <yao.jin@intel.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lore.kernel.org/lkml/20210514122948.9472-2-yao.jin@linux.intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/util/env.c')
-rw-r--r-- | tools/perf/util/env.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tools/perf/util/env.c b/tools/perf/util/env.c index 9130f6fad8d5..744ae87b5bfa 100644 --- a/tools/perf/util/env.c +++ b/tools/perf/util/env.c @@ -202,6 +202,12 @@ void perf_env__exit(struct perf_env *env) for (i = 0; i < env->nr_memory_nodes; i++) zfree(&env->memory_nodes[i].set); zfree(&env->memory_nodes); + + for (i = 0; i < env->nr_hybrid_nodes; i++) { + zfree(&env->hybrid_nodes[i].pmu_name); + zfree(&env->hybrid_nodes[i].cpus); + } + zfree(&env->hybrid_nodes); } void perf_env__init(struct perf_env *env __maybe_unused) |