On 22/01/2021 14:51, Arnaldo Carvalho de Melo wrote: > Em Tue, Jan 19, 2021 at 04:46:51PM +0200, James Clark escreveu: >> From: Leo Yan <leo....@linaro.org> >> >> This patch is to enable sample type PERF_SAMPLE_DATA_SRC for Arm SPE in >> the perf data, when output the tracing data, it tells tools that it >> contains data source in the memory event. >> >> Signed-off-by: Leo Yan <leo....@linaro.org> >> Signed-off-by: James Clark <james.cl...@arm.com> > > I see two Signed-off-by, ok, any Reviewed-by? > > - Arnaldo Hi Arnaldo, I have submitted v2 and added my reviewed-by and tested-by. I didn't change any of the authors as Leo suggested because I only modified the last two patches which we dropped anyway to not show any misleading PID data when run from a container. Thanks James > >> Cc: Peter Zijlstra <pet...@infradead.org> >> Cc: Ingo Molnar <mi...@redhat.com> >> Cc: Arnaldo Carvalho de Melo <a...@kernel.org> >> Cc: Mark Rutland <mark.rutl...@arm.com> >> Cc: Alexander Shishkin <alexander.shish...@linux.intel.com> >> Cc: Jiri Olsa <jo...@redhat.com> >> Cc: Namhyung Kim <namhy...@kernel.org> >> Cc: John Garry <john.ga...@huawei.com> >> Cc: Will Deacon <w...@kernel.org> >> Cc: Mathieu Poirier <mathieu.poir...@linaro.org> >> Cc: Al Grant <al.gr...@arm.com> >> Cc: Andre Przywara <andre.przyw...@arm.com> >> Cc: Wei Li <liwei...@huawei.com> >> Cc: Tan Xiaojun <tanxiao...@huawei.com> >> Cc: Adrian Hunter <adrian.hun...@intel.com> >> --- >> tools/perf/util/arm-spe.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tools/perf/util/arm-spe.c b/tools/perf/util/arm-spe.c >> index 8901a1656a41..b134516e890b 100644 >> --- a/tools/perf/util/arm-spe.c >> +++ b/tools/perf/util/arm-spe.c >> @@ -803,7 +803,7 @@ arm_spe_synth_events(struct arm_spe *spe, struct >> perf_session *session) >> attr.type = PERF_TYPE_HARDWARE; >> attr.sample_type = evsel->core.attr.sample_type & PERF_SAMPLE_MASK; >> attr.sample_type |= PERF_SAMPLE_IP | PERF_SAMPLE_TID | >> - PERF_SAMPLE_PERIOD; >> + PERF_SAMPLE_PERIOD | PERF_SAMPLE_DATA_SRC; >> if (spe->timeless_decoding) >> attr.sample_type &= ~(u64)PERF_SAMPLE_TIME; >> else >> -- >> 2.28.0 >> >
