From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Ian Rogers <irogers@google.com>
Cc: Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@redhat.com>,
Namhyung Kim <namhyung@kernel.org>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Jiri Olsa <jolsa@kernel.org>,
Adrian Hunter <adrian.hunter@intel.com>,
James Clark <james.clark@linaro.org>,
Shimin Guo <shimin.guo@skydio.com>,
Andi Kleen <ak@linux.intel.com>, Leo Yan <leo.yan@arm.com>,
Yujie Liu <yujie.liu@intel.com>,
linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org
Subject: Re: [PATCH v1 0/3] perf more ELF machine usage
Date: Mon, 26 Jan 2026 17:52:36 -0300 [thread overview]
Message-ID: <aXfUFBfCQ4m_DFrx@x1> (raw)
In-Reply-To: <20260124001611.1332019-1-irogers@google.com>
On Fri, Jan 23, 2026 at 04:16:08PM -0800, Ian Rogers wrote:
> Switch 3 more cases where the perf_env arch was being used to
> determine the architecture type and switch to using the ELF
> machine. This includes the addition of minimal instruction parsing in
> perf script for architectures other than x86, or x86 when
> cross-platform.
>
> Note: this series will conflict with the e_flags series:
> https://lore.kernel.org/lkml/20260123222209.1181249-1-irogers@google.com/
> that adds an additional out argument to thread__e_machine. In all the
> cases here that argument should be NULL. So:
> thread__e_machine(thread, machine)
> becomes:
> thread__e_machine(thread, machine, /*e_flags=*/NULL)
This one isn't applying.
- Arnaldo
> Ian Rogers (3):
> perf script: Fix script_fetch_insn for more than just x86
> perf callchain: Switch callchain_param_setup from an arch to an
> e_machine
> perf thread-stack: Switch thread_stack__init to use e_machine
>
> tools/perf/arch/x86/util/Build | 1 -
> tools/perf/arch/x86/util/archinsn.c | 27 -------
> tools/perf/builtin-report.c | 6 +-
> tools/perf/builtin-script.c | 20 +-----
> .../scripts/python/Perf-Trace-Util/Context.c | 2 +-
> tools/perf/tests/dlfilter-test.c | 1 -
> tools/perf/util/archinsn.h | 12 ----
> tools/perf/util/callchain.c | 5 +-
> tools/perf/util/callchain.h | 2 +-
> tools/perf/util/dlfilter.c | 3 +-
> tools/perf/util/sample.c | 71 +++++++++++++++++++
> tools/perf/util/sample.h | 7 ++
> tools/perf/util/thread-stack.c | 4 +-
> tools/perf/util/trace-event-scripting.c | 16 -----
> tools/perf/util/trace-event.h | 3 -
> 15 files changed, 91 insertions(+), 89 deletions(-)
> delete mode 100644 tools/perf/arch/x86/util/archinsn.c
> delete mode 100644 tools/perf/util/archinsn.h
>
> --
> 2.52.0.457.g6b5491de43-goog
>
next prev parent reply other threads:[~2026-01-26 20:52 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-24 0:16 [PATCH v1 0/3] perf more ELF machine usage Ian Rogers
2026-01-24 0:16 ` [PATCH v1 1/3] perf script: Fix script_fetch_insn for more than just x86 Ian Rogers
2026-01-24 0:16 ` [PATCH v1 2/3] perf callchain: Switch callchain_param_setup from an arch to an e_machine Ian Rogers
2026-01-24 0:16 ` [PATCH v1 3/3] perf thread-stack: Switch thread_stack__init to use e_machine Ian Rogers
2026-01-26 20:52 ` Arnaldo Carvalho de Melo [this message]
2026-01-26 20:53 ` [PATCH v1 0/3] perf more ELF machine usage Arnaldo Carvalho de Melo
2026-01-26 21:35 ` Arnaldo Carvalho de Melo
2026-01-26 22:07 ` Ian Rogers
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=aXfUFBfCQ4m_DFrx@x1 \
--to=acme@kernel.org \
--cc=adrian.hunter@intel.com \
--cc=ak@linux.intel.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=irogers@google.com \
--cc=james.clark@linaro.org \
--cc=jolsa@kernel.org \
--cc=leo.yan@arm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-perf-users@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=namhyung@kernel.org \
--cc=peterz@infradead.org \
--cc=shimin.guo@skydio.com \
--cc=yujie.liu@intel.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