linux-kernel - Re: [PATCH v2 0/5] perf arm-spe: Introduce metadata version 2

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: <ZvZMXpLnyzhFMrLb@google.com>
Date: Thu, 26 Sep 2024 23:10:38 -0700
From: Namhyung Kim <namhyung@...nel.org>
To: Leo Yan <leo.yan@....com>
Cc: Arnaldo Carvalho de Melo <acme@...nel.org>,
	James Clark <james.clark@...aro.org>,
	Mike Leach <mike.leach@...aro.org>, Will Deacon <will@...nel.org>,
	Mark Rutland <mark.rutland@....com>, Jiri Olsa <jolsa@...nel.org>,
	Ian Rogers <irogers@...gle.com>,
	Adrian Hunter <adrian.hunter@...el.com>,
	"Liang, Kan" <kan.liang@...ux.intel.com>,
	linux-arm-kernel@...ts.infradead.org,
	linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/5] perf arm-spe: Introduce metadata version 2

Hi Leo,

On Thu, Sep 26, 2024 at 05:20:49PM +0100, Leo Yan wrote:
> On 9/14/24 22:54, Leo Yan wrote:
> > This patch series enhances Arm SPE metadata in the Perf file to a
> > version 2 format and maintains backward compatibility for metadata v1.
> > 
> > The version 2 metadata contains a global structure with fields for
> > metadata header version number, header size, CPU numbers. It also
> > expands to store per-CPU metadata, which includes the CPU logical
> > number in the kernel, MIDR, and capacity values associated with Arm SPE.
> > 
> > This patch set has been tested the perf to decode the Arm SPE metadata
> > v1 and v2.
> > 
> > Changes from v1:
> > - Dropped LDS bit exposing from Arm SPE driver (Will Deacon).
> > - To simplify the change, this series did not include multiple AUX event
> >    support.
> 
> Hi Arnaldo, Namhyung,
> 
> Gentle ping. There is a bit backlog for Arm SPE patches, so I would
> like to bring up this series and the series [1] for the Arm SPE data
> source refactoring (which is dependent on the current series).
> 
> Please kindly review and pick it up if it is fine for you.

While it seems like general changes, I'd like to see some ARM folks
reviewing this series.

Thanks,
Namhyung

> 
> [1] https://lore.kernel.org/linux-perf-users/20240914220901.756177-1-leo.yan@arm.com/
> 
> 
> > Leo Yan (5):
> >    perf arm-spe: Define metadata header version 2
> >    perf arm-spe: Calculate meta data size
> >    perf arm-spe: Save per CPU information in metadata
> >    perf arm-spe: Support metadata version 2
> >    perf arm-spe: Dump metadata with version 2
> > 
> >   tools/perf/arch/arm64/util/arm-spe.c | 106 ++++++++++++++++++-
> >   tools/perf/util/arm-spe.c            | 151 +++++++++++++++++++++++++--
> >   tools/perf/util/arm-spe.h            |  38 ++++++-
> >   3 files changed, 281 insertions(+), 14 deletions(-)
> >