From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Namhyung Kim <namhyung@kernel.org>
Cc: Ingo Molnar <mingo@kernel.org>,
Thomas Gleixner <tglx@linutronix.de>,
James Clark <james.clark@linaro.org>,
Jiri Olsa <jolsa@kernel.org>, Ian Rogers <irogers@google.com>,
Adrian Hunter <adrian.hunter@intel.com>,
Kan Liang <kan.liang@linux.intel.com>,
Clark Williams <williams@redhat.com>,
linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org,
Arnaldo Carvalho de Melo <acme@kernel.org>
Subject: [PATCHES perf-tools-next v5 0/7] Cleanups and fixes
Date: Wed, 8 Apr 2026 14:28:38 -0300 [thread overview]
Message-ID: <20260408172846.96360-1-acme@kernel.org> (raw)
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.
This also adds checks for perf.data headers that were suggested
by sashiko.
Thanks a lot,
- Arnaldo
v5: Address more comments and add perf.data header checks.
v4: Address some more Sashiko comments, see patch comments for details.
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 (7):
perf symbols: Make variable receiving result strrchr() const
perf util: Kill die() prototype, dead for a long time
perf bench: Constify tables
perf header: Use a max number of command line args
perf header: Do validation of perf.data HEADER_CPU_DOMAIN_INFO
perf tools: Use calloc() where applicable
perf tools: Replace basename() calls with perf_basename()
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-daemon.c | 4 +-
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/annotate.c | 3 +-
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-convert-json.c | 4 +-
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/dsos.c | 32 ++----
tools/perf/util/event.c | 1 -
tools/perf/util/evlist.c | 3 +-
tools/perf/util/header.c | 35 ++++--
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 | 20 ++--
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 | 12 +-
tools/perf/util/stat-shadow.c | 1 -
tools/perf/util/symbol.c | 2 +-
tools/perf/util/symbol.h | 16 +--
tools/perf/util/unwind-libunwind-local.c | 1 -
tools/perf/util/util.c | 10 +-
tools/perf/util/util.h | 3 +-
tools/perf/util/values.c | 8 +-
69 files changed, 251 insertions(+), 280 deletions(-)
--
2.53.0
next reply other threads:[~2026-04-08 17:29 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-08 17:28 Arnaldo Carvalho de Melo [this message]
2026-04-08 17:28 ` [PATCH 1/8] perf symbols: Make variable receiving result strrchr() const Arnaldo Carvalho de Melo
2026-04-08 17:28 ` [PATCH 2/8] perf util: Kill die() prototype, dead for a long time Arnaldo Carvalho de Melo
2026-04-08 17:28 ` [PATCH 3/8] perf tools: Make more global variables static Arnaldo Carvalho de Melo
2026-04-08 17:28 ` [PATCH 4/8] perf bench: Constify tables Arnaldo Carvalho de Melo
2026-04-08 17:28 ` [PATCH 5/8] perf header: Use a max number of command line args Arnaldo Carvalho de Melo
2026-04-08 18:17 ` sashiko-bot
2026-04-08 17:28 ` [PATCH 6/8] perf header: Do validation of perf.data HEADER_CPU_DOMAIN_INFO Arnaldo Carvalho de Melo
2026-04-08 18:38 ` sashiko-bot
2026-04-08 17:28 ` [PATCH 7/8] perf tools: Use calloc() where applicable Arnaldo Carvalho de Melo
2026-04-08 17:28 ` [PATCH 8/8] perf tools: Replace basename() calls with perf_basename() Arnaldo Carvalho de Melo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260408172846.96360-1-acme@kernel.org \
--to=acme@kernel.org \
--cc=adrian.hunter@intel.com \
--cc=irogers@google.com \
--cc=james.clark@linaro.org \
--cc=jolsa@kernel.org \
--cc=kan.liang@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-perf-users@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=namhyung@kernel.org \
--cc=tglx@linutronix.de \
--cc=williams@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox