linux-kernel - Re: [PATCH v1 0/3] perf more ELF machine usage

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aXfUFBfCQ4m_DFrx@x1>
Date: Mon, 26 Jan 2026 17:52:36 -0300
From: Arnaldo Carvalho de Melo <acme@...nel.org>
To: Ian Rogers <irogers@...gle.com>
Cc: Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...hat.com>,
	Namhyung Kim <namhyung@...nel.org>,
	Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
	Jiri Olsa <jolsa@...nel.org>,
	Adrian Hunter <adrian.hunter@...el.com>,
	James Clark <james.clark@...aro.org>,
	Shimin Guo <shimin.guo@...dio.com>, Andi Kleen <ak@...ux.intel.com>,
	Leo Yan <leo.yan@....com>, Yujie Liu <yujie.liu@...el.com>,
	linux-kernel@...r.kernel.org, linux-perf-users@...r.kernel.org
Subject: Re: [PATCH v1 0/3] perf more ELF machine usage

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
>