From: Leo Yan <leo.yan@linaro.org> To: Timothy Hayes <timothy.hayes@arm.com> Cc: linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, acme@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@kernel.org>, Namhyung Kim <namhyung@kernel.org>, Martin KaFai Lau <kafai@fb.com>, Song Liu <songliubraving@fb.com>, Yonghong Song <yhs@fb.com>, John Fastabend <john.fastabend@gmail.com>, KP Singh <kpsingh@kernel.org>, linux-arm-kernel@lists.infradead.org, netdev@vger.kernel.org, bpf@vger.kernel.org Subject: Re: [PATCH 2/3] perf: arm-spe: Fix SPE events with phys addresses Date: Sun, 24 Apr 2022 20:59:51 +0800 [thread overview] Message-ID: <20220424125951.GD978927@leoy-ThinkPad-X240s> (raw) In-Reply-To: <20220421165205.117662-3-timothy.hayes@arm.com> Hi Timothy, On Thu, Apr 21, 2022 at 05:52:04PM +0100, Timothy Hayes wrote: > This patch corrects a bug whereby SPE collection is invoked with > pa_enable=1 but synthesized events fail to show physical addresses. > > Signed-off-by: Timothy Hayes <timothy.hayes@arm.com> > --- > tools/perf/arch/arm64/util/arm-spe.c | 10 ++++++++++ > tools/perf/util/arm-spe.c | 3 ++- > 2 files changed, 12 insertions(+), 1 deletion(-) > > diff --git a/tools/perf/arch/arm64/util/arm-spe.c b/tools/perf/arch/arm64/util/arm-spe.c > index af4d63af8072..e8b577d33e53 100644 > --- a/tools/perf/arch/arm64/util/arm-spe.c > +++ b/tools/perf/arch/arm64/util/arm-spe.c > @@ -148,6 +148,7 @@ static int arm_spe_recording_options(struct auxtrace_record *itr, > bool privileged = perf_event_paranoid_check(-1); > struct evsel *tracking_evsel; > int err; > + u64 bit; > > sper->evlist = evlist; > > @@ -245,6 +246,15 @@ static int arm_spe_recording_options(struct auxtrace_record *itr, > */ > evsel__set_sample_bit(arm_spe_evsel, DATA_SRC); > > + /* > + * The PHYS_ADDR flag does not affect the driver behaviour, it is used to > + * inform that the resulting output's SPE samples contain physical addresses > + * where applicable. > + */ > + bit = perf_pmu__format_bits(&arm_spe_pmu->format, "pa_enable"); > + if (arm_spe_evsel->core.attr.config & bit) > + evsel__set_sample_bit(arm_spe_evsel, PHYS_ADDR); > + > /* Add dummy event to keep tracking */ > err = parse_events(evlist, "dummy:u", NULL); > if (err) > diff --git a/tools/perf/util/arm-spe.c b/tools/perf/util/arm-spe.c > index 151cc38a171c..1a80151baed9 100644 > --- a/tools/perf/util/arm-spe.c > +++ b/tools/perf/util/arm-spe.c > @@ -1033,7 +1033,8 @@ arm_spe_synth_events(struct arm_spe *spe, struct perf_session *session) > memset(&attr, 0, sizeof(struct perf_event_attr)); > attr.size = sizeof(struct perf_event_attr); > attr.type = PERF_TYPE_HARDWARE; > - attr.sample_type = evsel->core.attr.sample_type & PERF_SAMPLE_MASK; > + attr.sample_type = evsel->core.attr.sample_type & > + (PERF_SAMPLE_MASK | PERF_SAMPLE_PHYS_ADDR); I verified this patch and I can confirm the physical address can be dumped successfully. I have a more general question, seems to me, we need to change the macro PERF_SAMPLE_MASK in the file util/event.h as below, so here doesn't need to 'or' the flag PERF_SAMPLE_PHYS_ADDR anymore. @Arnaldo, @Jiri, could you confirm if this is the right way to move forward? I am not sure why PERF_SAMPLE_MASK doesn't contain the bit PERF_SAMPLE_PHYS_ADDR in current code. diff --git a/tools/perf/util/event.h b/tools/perf/util/event.h index cdd72e05fd28..c905ac32ebad 100644 --- a/tools/perf/util/event.h +++ b/tools/perf/util/event.h @@ -39,7 +39,7 @@ struct perf_event_attr; PERF_SAMPLE_TIME | PERF_SAMPLE_ADDR | \ PERF_SAMPLE_ID | PERF_SAMPLE_STREAM_ID | \ PERF_SAMPLE_CPU | PERF_SAMPLE_PERIOD | \ - PERF_SAMPLE_IDENTIFIER) + PERF_SAMPLE_IDENTIFIER | PERF_SAMPLE_PHYS_ADDR) Thanks, Leo > attr.sample_type |= PERF_SAMPLE_IP | PERF_SAMPLE_TID | > PERF_SAMPLE_PERIOD | PERF_SAMPLE_DATA_SRC | > PERF_SAMPLE_WEIGHT | PERF_SAMPLE_ADDR; > -- > 2.25.1 >
WARNING: multiple messages have this Message-ID (diff)
From: Leo Yan <leo.yan@linaro.org> To: Timothy Hayes <timothy.hayes@arm.com> Cc: linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, acme@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@kernel.org>, Namhyung Kim <namhyung@kernel.org>, Martin KaFai Lau <kafai@fb.com>, Song Liu <songliubraving@fb.com>, Yonghong Song <yhs@fb.com>, John Fastabend <john.fastabend@gmail.com>, KP Singh <kpsingh@kernel.org>, linux-arm-kernel@lists.infradead.org, netdev@vger.kernel.org, bpf@vger.kernel.org Subject: Re: [PATCH 2/3] perf: arm-spe: Fix SPE events with phys addresses Date: Sun, 24 Apr 2022 20:59:51 +0800 [thread overview] Message-ID: <20220424125951.GD978927@leoy-ThinkPad-X240s> (raw) In-Reply-To: <20220421165205.117662-3-timothy.hayes@arm.com> Hi Timothy, On Thu, Apr 21, 2022 at 05:52:04PM +0100, Timothy Hayes wrote: > This patch corrects a bug whereby SPE collection is invoked with > pa_enable=1 but synthesized events fail to show physical addresses. > > Signed-off-by: Timothy Hayes <timothy.hayes@arm.com> > --- > tools/perf/arch/arm64/util/arm-spe.c | 10 ++++++++++ > tools/perf/util/arm-spe.c | 3 ++- > 2 files changed, 12 insertions(+), 1 deletion(-) > > diff --git a/tools/perf/arch/arm64/util/arm-spe.c b/tools/perf/arch/arm64/util/arm-spe.c > index af4d63af8072..e8b577d33e53 100644 > --- a/tools/perf/arch/arm64/util/arm-spe.c > +++ b/tools/perf/arch/arm64/util/arm-spe.c > @@ -148,6 +148,7 @@ static int arm_spe_recording_options(struct auxtrace_record *itr, > bool privileged = perf_event_paranoid_check(-1); > struct evsel *tracking_evsel; > int err; > + u64 bit; > > sper->evlist = evlist; > > @@ -245,6 +246,15 @@ static int arm_spe_recording_options(struct auxtrace_record *itr, > */ > evsel__set_sample_bit(arm_spe_evsel, DATA_SRC); > > + /* > + * The PHYS_ADDR flag does not affect the driver behaviour, it is used to > + * inform that the resulting output's SPE samples contain physical addresses > + * where applicable. > + */ > + bit = perf_pmu__format_bits(&arm_spe_pmu->format, "pa_enable"); > + if (arm_spe_evsel->core.attr.config & bit) > + evsel__set_sample_bit(arm_spe_evsel, PHYS_ADDR); > + > /* Add dummy event to keep tracking */ > err = parse_events(evlist, "dummy:u", NULL); > if (err) > diff --git a/tools/perf/util/arm-spe.c b/tools/perf/util/arm-spe.c > index 151cc38a171c..1a80151baed9 100644 > --- a/tools/perf/util/arm-spe.c > +++ b/tools/perf/util/arm-spe.c > @@ -1033,7 +1033,8 @@ arm_spe_synth_events(struct arm_spe *spe, struct perf_session *session) > memset(&attr, 0, sizeof(struct perf_event_attr)); > attr.size = sizeof(struct perf_event_attr); > attr.type = PERF_TYPE_HARDWARE; > - attr.sample_type = evsel->core.attr.sample_type & PERF_SAMPLE_MASK; > + attr.sample_type = evsel->core.attr.sample_type & > + (PERF_SAMPLE_MASK | PERF_SAMPLE_PHYS_ADDR); I verified this patch and I can confirm the physical address can be dumped successfully. I have a more general question, seems to me, we need to change the macro PERF_SAMPLE_MASK in the file util/event.h as below, so here doesn't need to 'or' the flag PERF_SAMPLE_PHYS_ADDR anymore. @Arnaldo, @Jiri, could you confirm if this is the right way to move forward? I am not sure why PERF_SAMPLE_MASK doesn't contain the bit PERF_SAMPLE_PHYS_ADDR in current code. diff --git a/tools/perf/util/event.h b/tools/perf/util/event.h index cdd72e05fd28..c905ac32ebad 100644 --- a/tools/perf/util/event.h +++ b/tools/perf/util/event.h @@ -39,7 +39,7 @@ struct perf_event_attr; PERF_SAMPLE_TIME | PERF_SAMPLE_ADDR | \ PERF_SAMPLE_ID | PERF_SAMPLE_STREAM_ID | \ PERF_SAMPLE_CPU | PERF_SAMPLE_PERIOD | \ - PERF_SAMPLE_IDENTIFIER) + PERF_SAMPLE_IDENTIFIER | PERF_SAMPLE_PHYS_ADDR) Thanks, Leo > attr.sample_type |= PERF_SAMPLE_IP | PERF_SAMPLE_TID | > PERF_SAMPLE_PERIOD | PERF_SAMPLE_DATA_SRC | > PERF_SAMPLE_WEIGHT | PERF_SAMPLE_ADDR; > -- > 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-04-24 13:00 UTC|newest] Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-04-21 16:52 [PATCH 0/3] perf: arm-spe: Fix addresses of synthesized Arm SPE events Timothy Hayes 2022-04-21 16:52 ` Timothy Hayes 2022-04-21 16:52 ` [PATCH 1/3] perf: arm-spe: Fix addresses of synthesized " Timothy Hayes 2022-04-21 16:52 ` Timothy Hayes 2022-04-24 12:28 ` Leo Yan 2022-04-24 12:28 ` Leo Yan 2022-04-24 15:22 ` Leo Yan 2022-04-24 15:22 ` Leo Yan 2022-04-21 16:52 ` [PATCH 2/3] perf: arm-spe: Fix SPE events with phys addresses Timothy Hayes 2022-04-21 16:52 ` Timothy Hayes 2022-04-24 12:59 ` Leo Yan [this message] 2022-04-24 12:59 ` Leo Yan 2022-04-25 9:12 ` James Clark 2022-04-25 9:12 ` James Clark 2022-04-26 13:19 ` Leo Yan 2022-04-26 13:19 ` Leo Yan 2022-04-21 16:52 ` [PATCH 3/3] perf test: Add perf_event_attr test for Arm SPE Timothy Hayes 2022-04-21 16:52 ` Timothy Hayes 2022-04-24 14:53 ` Leo Yan 2022-04-24 14:53 ` Leo Yan 2022-04-28 13:41 ` Arnaldo Carvalho de Melo 2022-04-28 13:41 ` 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=20220424125951.GD978927@leoy-ThinkPad-X240s \ --to=leo.yan@linaro.org \ --cc=acme@kernel.org \ --cc=alexander.shishkin@linux.intel.com \ --cc=bpf@vger.kernel.org \ --cc=john.fastabend@gmail.com \ --cc=john.garry@huawei.com \ --cc=jolsa@kernel.org \ --cc=kafai@fb.com \ --cc=kpsingh@kernel.org \ --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=netdev@vger.kernel.org \ --cc=songliubraving@fb.com \ --cc=timothy.hayes@arm.com \ --cc=will@kernel.org \ --cc=yhs@fb.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.