From: Ravi Bangoria <ravi.bangoria@amd.com>
To: Peter Zijlstra <peterz@infradead.org>,
Arnaldo Carvalho de Melo <acme@kernel.org>,
Namhyung Kim <namhyung@kernel.org>
Cc: Ravi Bangoria <ravi.bangoria@amd.com>,
Ingo Molnar <mingo@redhat.com>,
Stephane Eranian <eranian@google.com>,
Ian Rogers <irogers@google.com>,
"Kan Liang" <kan.liang@linux.intel.com>,
James Clark <james.clark@linaro.org>, "Leo Yan" <leo.yan@arm.com>,
Joe Mario <jmario@redhat.com>, <linux-kernel@vger.kernel.org>,
<linux-perf-users@vger.kernel.org>,
"Santosh Shukla" <santosh.shukla@amd.com>,
Ananth Narayan <ananth.narayan@amd.com>,
Sandipan Das <sandipan.das@amd.com>
Subject: [PATCH 0/4] perf/amd/ibs: Introduce load/store SW filter
Date: Thu, 29 May 2025 12:34:52 +0000 [thread overview]
Message-ID: <20250529123456.1801-1-ravi.bangoria@amd.com> (raw)
Since current IBS OP PMU does not have the capability to tag only load/
stores instructions, tools like perf mem/c2c ends up recording lots of
unwanted samples. See for example [1]. All these unwanted samples are:
1) Passed from kernel to userspace
2) Stored in the perf.data file
3) Processed every time while running perf mem/c2c report
4) Skews perf mem/c2c overhead computations
So, better to get rid of them early in the IBS NMI handler itself.
Although SW filter is not an ideal solution, it's better compared to
no filter at all.
[1]: https://lore.kernel.org/r/aBTyYv_yXCPkn2d0@x1
Patches are prepated on perf-tools-next/perf-tools-next (4d9b5146f0d9).
Ravi Bangoria (4):
perf/amd/ibs: Add load/store SW filters to IBS OP PMU
perf mem/c2c amd: Wire IBS OP PMU load/store SW filter
perf test amd: Add IBS load/store swfilt tests
perf doc amd: Update perf-amd-ibs man page
arch/x86/events/amd/ibs.c | 73 ++++++++++++++++++++--
include/linux/perf_event.h | 14 +++++
tools/perf/Documentation/perf-amd-ibs.txt | 72 +++++++++++++++------
tools/perf/arch/x86/util/mem-events.c | 12 ++++
tools/perf/arch/x86/util/mem-events.h | 2 +
tools/perf/arch/x86/util/pmu.c | 17 +++--
tools/perf/tests/shell/amd-ibs-swfilt.sh | 26 ++++++++
tools/perf/tests/shell/test_data_symbol.sh | 26 ++++----
tools/perf/util/mem-events.c | 2 +-
9 files changed, 205 insertions(+), 39 deletions(-)
--
2.43.0
next reply other threads:[~2025-05-29 12:35 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-29 12:34 Ravi Bangoria [this message]
2025-05-29 12:34 ` [PATCH 1/4] perf/amd/ibs: Add load/store SW filters to IBS OP PMU Ravi Bangoria
2025-05-31 7:53 ` Ingo Molnar
2025-06-03 6:52 ` Namhyung Kim
2025-06-10 5:12 ` Ravi Bangoria
2025-05-29 12:34 ` [PATCH 2/4] perf mem/c2c amd: Wire IBS OP PMU load/store SW filter Ravi Bangoria
2025-05-29 12:34 ` [PATCH 3/4] perf test amd: Add IBS load/store swfilt tests Ravi Bangoria
2025-05-29 12:34 ` [PATCH 4/4] perf doc amd: Update perf-amd-ibs man page Ravi Bangoria
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=20250529123456.1801-1-ravi.bangoria@amd.com \
--to=ravi.bangoria@amd.com \
--cc=acme@kernel.org \
--cc=ananth.narayan@amd.com \
--cc=eranian@google.com \
--cc=irogers@google.com \
--cc=james.clark@linaro.org \
--cc=jmario@redhat.com \
--cc=kan.liang@linux.intel.com \
--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=sandipan.das@amd.com \
--cc=santosh.shukla@amd.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;
as well as URLs for NNTP newsgroup(s).