[lkml]   [2020]   [Nov]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Subject[PATCH 00/13 v4] perf tools: fix perf stat with large socket IDs
v3 had a mistake in a couple of my signed off lines so I have fixed them
in v4.

v3 breaks up the previous v2 patchset into smaller atomic commits.
The end result is the same as the previous patchset apart from
some minor refactoring, asserting on an empty header and
calling cpu__setup_cpunode_map() in the topology self test.

Testing done:

Tested --per-core, --per-thread, --per-die, --per-node 'perf
stat' outputs on Arm ThunderX2 and Intel KNL.

Also tested 'perf stat record' and 'perf stat report --input'
with recordings from a version of perf before this patchset
to confirm that the output was the same.

Signed-off-by: James Clark <>
Cc: Peter Zijlstra <>
Cc: Ingo Molnar <>
Cc: Arnaldo Carvalho de Melo <>
Cc: Mark Rutland <>
Cc: Alexander Shishkin <>
Cc: Jiri Olsa <>
Cc: Namhyung Kim <>
Cc: Thomas Richter <>
Cc: John Garry <>

James Clark (13):
perf tools: Improve topology test
perf tools: Use allocator for perf_cpu_map
perf tools: Add new struct for cpu aggregation
perf tools: Replace aggregation ID with a struct
perf tools: add new map type for aggregation
perf tools: drop in cpu_aggr_map struct
perf tools: restrict visibility of functions
perf tools: Start using cpu_aggr_id in map
perf tools: Add separate node member
perf tools: Add separate socket member
perf tools: Add separate die member
perf tools: Add separate core member
perf tools: add thread field

tools/perf/builtin-stat.c | 128 +++++++++++++------------
tools/perf/tests/topology.c | 58 +++++++++--
tools/perf/util/cpumap.c | 170 ++++++++++++++++++++++-----------
tools/perf/util/cpumap.h | 55 ++++++-----
tools/perf/util/stat-display.c | 106 +++++++++++---------
tools/perf/util/stat.c | 2 +-
tools/perf/util/stat.h | 9 +-
7 files changed, 332 insertions(+), 196 deletions(-)


 \ /
  Last update: 2020-11-13 18:28    [W:0.108 / U:6.952 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site