Messages in this thread | | | Date | Tue, 4 Oct 2022 15:48:35 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCHSET 0/7] perf stat: Small random cleanups (v2) |
| |
Em Fri, Sep 30, 2022 at 01:21:03PM -0700, Namhyung Kim escreveu: > Hello, > > I'm working on perf stat and I found some items to clean up. This time > I removed runtime stats for per-thread aggregation mode which we can simply > use thread map index to compare the shadow stat values in the rt_stat. > > Changes from v1) > * renaming missing parts in headers (James) > * add a comment for map_idx (Ian) > * add Reviewed-by from James > > The code is available at 'perf/stat-cleanup-v2' branch in > > git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git
Applied locally, testing.
- Arnaldo
> Thanks, > Namhyung > > > Namhyung Kim (7): > perf stat: Convert perf_stat_evsel.res_stats array > perf stat: Don't call perf_stat_evsel_id_init() repeatedly > perf stat: Rename saved_value->cpu_map_idx > perf stat: Use thread map index for shadow stat > perf stat: Kill unused per-thread runtime stats > perf stat: Don't compare runtime stat for shadow stats > perf stat: Rename to aggr_cpu_id.thread_idx > > tools/perf/builtin-stat.c | 54 ------ > tools/perf/tests/topology.c | 10 +- > tools/perf/util/cpumap.c | 8 +- > tools/perf/util/cpumap.h | 2 +- > tools/perf/util/stat-display.c | 32 ++-- > tools/perf/util/stat-shadow.c | 320 ++++++++++++++++----------------- > tools/perf/util/stat.c | 20 +-- > tools/perf/util/stat.h | 10 +- > 8 files changed, 189 insertions(+), 267 deletions(-) > > > base-commit: 62e64c9d2fd12839c02f1b3e8b873e7cb34e8720 > -- > 2.38.0.rc1.362.ged0d419d3c-goog
--
- Arnaldo
| |