From: Leo Yan <leo.yan@linaro.org>
To: German Gomez <german.gomez@arm.com>,
Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org,
John Garry <john.garry@huawei.com>, Will Deacon <will@kernel.org>,
Mathieu Poirier <mathieu.poirier@linaro.org>,
Mark Rutland <mark.rutland@arm.com>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Jiri Olsa <jolsa@redhat.com>, Namhyung Kim <namhyung@kernel.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 0/3] perf arm-spe: Add snapshot mode support
Date: Thu, 11 Nov 2021 16:46:21 +0800 [thread overview]
Message-ID: <20211111084621.GC106401@leoy-ThinkPad-X240s> (raw)
In-Reply-To: <20211109163009.92072-1-german.gomez@arm.com>
Hi Arnaldo,
On Tue, Nov 09, 2021 at 04:30:06PM +0000, German Gomez wrote:
> This patchset adds snapshot mode support for arm-spe.
>
> - [PATCH 1/3] implements the minimal callbacks to support recording in
> snapshot mode.
> - [PATCH 2/3] implements the find_snapshot callback in order to handle
> wrap-arounds in the AUX buffer.
> - [PATCH 3/3] adds a test for spe snapshot mode.
I have verified this patch set on Hisilicon D06 board, please consider
to pick up:
root@ubuntu:/home/leoy/linux/tools/perf# ./perf test -v 85
85: Check Arm SPE trace data recording and synthesized samples :
--- start ---
test child forked, pid 17083
Recording trace with snapshot mode /tmp/__perf_test.perf.data.MI2iX
Looking at perf.data file for dumping samples:
Looking at perf.data file for reporting samples:
SPE snapshot testing: PASS
test child finished with 0
---- end ----
Check Arm SPE trace data recording and synthesized samples: Ok
BTW, you could see German has another patch set for enabling pid/tid
for Arm SPE tracing [1]. I confirmed that the pid/tid patch set and
current patch set have no conflit, and don't need worry the dependency
between these two patch sets (so you could apply two patch sets in any
ordering).
Thanks,
Leo
[1] https://lore.kernel.org/lkml/20211111072714.GB102075@leoy-ThinkPad-X240s/T/#m932fd26d54278208331d4e3c32597cf63b6d4341
next prev parent reply other threads:[~2021-11-11 8:46 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-09 16:30 [PATCH v2 0/3] perf arm-spe: Add snapshot mode support German Gomez
2021-11-09 16:30 ` [PATCH v2 1/3] " German Gomez
2021-11-09 16:30 ` [PATCH v2 2/3] perf arm-spe: Implement find_snapshot callback German Gomez
2021-11-09 16:30 ` [PATCH v2 3/3] perf arm-spe: Snapshot mode test German Gomez
2021-11-11 8:32 ` Leo Yan
2021-11-10 23:52 ` [PATCH v2 0/3] perf arm-spe: Add snapshot mode support Namhyung Kim
2021-11-11 8:46 ` Leo Yan [this message]
2021-11-11 14:54 ` 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=20211111084621.GC106401@leoy-ThinkPad-X240s \
--to=leo.yan@linaro.org \
--cc=acme@kernel.org \
--cc=alexander.shishkin@linux.intel.com \
--cc=german.gomez@arm.com \
--cc=john.garry@huawei.com \
--cc=jolsa@redhat.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-perf-users@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mathieu.poirier@linaro.org \
--cc=namhyung@kernel.org \
--cc=will@kernel.org \
/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