[PATCHES v3 perf-tools-next 0/4] Cleanups and a fix

Arnaldo Carvalho de Melo posted 4 patches 2 months, 1 week ago
tools/perf/arch/arm/util/auxtrace.c           |   6 +-
tools/perf/arch/common.c                      |  22 ++--
tools/perf/arch/powerpc/util/auxtrace.c       |   1 +
tools/perf/arch/sh/include/dwarf-regs-table.h |   2 +-
tools/perf/arch/x86/tests/amd-ibs-period.c    |   3 +-
tools/perf/arch/x86/tests/dwarf-unwind.c      |  11 +-
tools/perf/arch/x86/util/pmu.c                |   1 -
tools/perf/bench/breakpoint.c                 |   4 +-
tools/perf/bench/mem-functions.c              |   2 +-
tools/perf/bench/numa.c                       |  15 +--
tools/perf/bench/sched-messaging.c            |   2 +-
tools/perf/bench/uprobe.c                     |   2 +-
tools/perf/builtin-annotate.c                 |   1 -
tools/perf/builtin-bench.c                    |  42 +++----
tools/perf/builtin-c2c.c                      |  13 ++-
tools/perf/builtin-config.c                   |   2 +-
tools/perf/builtin-data.c                     |   8 +-
tools/perf/builtin-diff.c                     |   6 +-
tools/perf/builtin-ftrace.c                   |   1 +
tools/perf/builtin-kmem.c                     |   2 +-
tools/perf/builtin-kwork.c                    |  14 +--
tools/perf/builtin-record.c                   |  10 +-
tools/perf/builtin-sched.c                    |   6 +-
tools/perf/builtin-script.c                   |  10 +-
tools/perf/builtin-stat.c                     |   2 +-
tools/perf/builtin-top.c                      |   5 +-
tools/perf/builtin-trace.c                    |   4 +-
tools/perf/jvmti/libjvmti.c                   |   5 +-
tools/perf/tests/bp_signal.c                  |   2 +-
tools/perf/tests/code-reading.c               |   1 +
tools/perf/tests/dso-data.c                   |   2 +-
tools/perf/tests/thread-map.c                 |   1 -
tools/perf/tests/wp.c                         |   6 +-
tools/perf/util/annotate-arch/annotate-x86.c  |   1 +
tools/perf/util/block-range.c                 |   2 +-
tools/perf/util/bpf-event.c                   |   2 +-
tools/perf/util/bpf_counter.c                 |   4 +-
tools/perf/util/bpf_counter_cgroup.c          |   1 -
tools/perf/util/bpf_off_cpu.c                 |   2 +-
tools/perf/util/data-convert-bt.c             |   2 +-
tools/perf/util/data.c                        |   2 +-
tools/perf/util/db-export.c                   |   1 -
tools/perf/util/debug.c                       |   2 +-
tools/perf/util/debuginfo.c                   |  19 ++--
tools/perf/util/disasm.c                      |   1 +
tools/perf/util/event.c                       |   1 -
tools/perf/util/evlist.c                      |   3 +-
tools/perf/util/header.c                      |  18 +--
tools/perf/util/hist.c                        |   2 +-
tools/perf/util/mem2node.c                    |   2 +-
tools/perf/util/pmus.c                        |   2 +-
tools/perf/util/powerpc-vpadtl.c              |   1 +
tools/perf/util/probe-event.c                 |  17 ++-
tools/perf/util/probe-file.c                  |   2 +-
tools/perf/util/probe-finder.c                |   8 +-
tools/perf/util/session.c                     |   2 +-
tools/perf/util/sort.c                        | 104 +++++++++---------
tools/perf/util/srcline.c                     |   1 +
tools/perf/util/stat-shadow.c                 |   1 -
tools/perf/util/symbol.h                      |   1 +
tools/perf/util/unwind-libunwind-local.c      |   1 -
tools/perf/util/util.c                        |   2 -
tools/perf/util/values.c                      |   8 +-
63 files changed, 203 insertions(+), 226 deletions(-)
[PATCHES v3 perf-tools-next 0/4] Cleanups and a fix
Posted by Arnaldo Carvalho de Melo 2 months, 1 week ago
From: Arnaldo Carvalho de Melo <acme@redhat.com>

Hi,

        Here are some cleanups, a refresh for a patch Ian sent and fell
thru the cracks, using calloc where applicable, constify some arrays and
add a missing header that was making the build fail on some musl
systems.

Thanks a lot,

- Arnaldo

v3: Address https://sashiko.dev review, removed a malloc->calloc
conversion, flawed, and removed the changes to trace-event-scripting.c,
that made the build fail in some situation, see the committer notes
in the patch for more details.

v2: remove the commenting of gen_vmlinux in tools/perf/tests/make as
noticed by Ian Rogers.

Arnaldo Carvalho de Melo (3):
  perf bench: Constify tables
  perf tools: Use calloc() were applicable
  perf symbol: Add missing libgen.h include to get basename() prototype

Ian Rogers (1):
  perf tools: Make more global variables static

 tools/perf/arch/arm/util/auxtrace.c           |   6 +-
 tools/perf/arch/common.c                      |  22 ++--
 tools/perf/arch/powerpc/util/auxtrace.c       |   1 +
 tools/perf/arch/sh/include/dwarf-regs-table.h |   2 +-
 tools/perf/arch/x86/tests/amd-ibs-period.c    |   3 +-
 tools/perf/arch/x86/tests/dwarf-unwind.c      |  11 +-
 tools/perf/arch/x86/util/pmu.c                |   1 -
 tools/perf/bench/breakpoint.c                 |   4 +-
 tools/perf/bench/mem-functions.c              |   2 +-
 tools/perf/bench/numa.c                       |  15 +--
 tools/perf/bench/sched-messaging.c            |   2 +-
 tools/perf/bench/uprobe.c                     |   2 +-
 tools/perf/builtin-annotate.c                 |   1 -
 tools/perf/builtin-bench.c                    |  42 +++----
 tools/perf/builtin-c2c.c                      |  13 ++-
 tools/perf/builtin-config.c                   |   2 +-
 tools/perf/builtin-data.c                     |   8 +-
 tools/perf/builtin-diff.c                     |   6 +-
 tools/perf/builtin-ftrace.c                   |   1 +
 tools/perf/builtin-kmem.c                     |   2 +-
 tools/perf/builtin-kwork.c                    |  14 +--
 tools/perf/builtin-record.c                   |  10 +-
 tools/perf/builtin-sched.c                    |   6 +-
 tools/perf/builtin-script.c                   |  10 +-
 tools/perf/builtin-stat.c                     |   2 +-
 tools/perf/builtin-top.c                      |   5 +-
 tools/perf/builtin-trace.c                    |   4 +-
 tools/perf/jvmti/libjvmti.c                   |   5 +-
 tools/perf/tests/bp_signal.c                  |   2 +-
 tools/perf/tests/code-reading.c               |   1 +
 tools/perf/tests/dso-data.c                   |   2 +-
 tools/perf/tests/thread-map.c                 |   1 -
 tools/perf/tests/wp.c                         |   6 +-
 tools/perf/util/annotate-arch/annotate-x86.c  |   1 +
 tools/perf/util/block-range.c                 |   2 +-
 tools/perf/util/bpf-event.c                   |   2 +-
 tools/perf/util/bpf_counter.c                 |   4 +-
 tools/perf/util/bpf_counter_cgroup.c          |   1 -
 tools/perf/util/bpf_off_cpu.c                 |   2 +-
 tools/perf/util/data-convert-bt.c             |   2 +-
 tools/perf/util/data.c                        |   2 +-
 tools/perf/util/db-export.c                   |   1 -
 tools/perf/util/debug.c                       |   2 +-
 tools/perf/util/debuginfo.c                   |  19 ++--
 tools/perf/util/disasm.c                      |   1 +
 tools/perf/util/event.c                       |   1 -
 tools/perf/util/evlist.c                      |   3 +-
 tools/perf/util/header.c                      |  18 +--
 tools/perf/util/hist.c                        |   2 +-
 tools/perf/util/mem2node.c                    |   2 +-
 tools/perf/util/pmus.c                        |   2 +-
 tools/perf/util/powerpc-vpadtl.c              |   1 +
 tools/perf/util/probe-event.c                 |  17 ++-
 tools/perf/util/probe-file.c                  |   2 +-
 tools/perf/util/probe-finder.c                |   8 +-
 tools/perf/util/session.c                     |   2 +-
 tools/perf/util/sort.c                        | 104 +++++++++---------
 tools/perf/util/srcline.c                     |   1 +
 tools/perf/util/stat-shadow.c                 |   1 -
 tools/perf/util/symbol.h                      |   1 +
 tools/perf/util/unwind-libunwind-local.c      |   1 -
 tools/perf/util/util.c                        |   2 -
 tools/perf/util/values.c                      |   8 +-
 63 files changed, 203 insertions(+), 226 deletions(-)

-- 
2.53.0