Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 28 Apr 2022 10:41:42 -0300
From:   Arnaldo Carvalho de Melo <acme@kernel.org>
To:     Leo Yan <leo.yan@linaro.org>
Cc:     Timothy Hayes <timothy.hayes@arm.com>,
        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@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 3/3] perf test: Add perf_event_attr test for Arm SPE
Message-ID: <YmqZluTEKAgg4AU0@kernel.org>
References: <20220421165205.117662-1-timothy.hayes@arm.com>
 <20220421165205.117662-4-timothy.hayes@arm.com>
 <20220424145307.GE978927@leoy-ThinkPad-X240s>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220424145307.GE978927@leoy-ThinkPad-X240s>
Precedence: bulk

Em Sun, Apr 24, 2022 at 10:53:07PM +0800, Leo Yan escreveu:
> On Thu, Apr 21, 2022 at 05:52:05PM +0100, Timothy Hayes wrote:
> > Adds a perf_event_attr test for Arm SPE in which the presence of
> > physical addresses are checked when SPE unit is run with pa_enable=1.
> > 
> > Signed-off-by: Timothy Hayes <timothy.hayes@arm.com>
> 
> Reviewed-by: Leo Yan <leo.yan@linaro.org>
> Tested-by: Leo Yan <leo.yan@linaro.org>

Thanks, applied the set to perf/urgent.

- Arnaldo
 
> > ---
> >  tools/perf/tests/attr/README                         |  1 +
> >  .../perf/tests/attr/test-record-spe-physical-address | 12 ++++++++++++
> >  2 files changed, 13 insertions(+)
> >  create mode 100644 tools/perf/tests/attr/test-record-spe-physical-address
> > 
> > diff --git a/tools/perf/tests/attr/README b/tools/perf/tests/attr/README
> > index 454505d343fa..eb3f7d4bb324 100644
> > --- a/tools/perf/tests/attr/README
> > +++ b/tools/perf/tests/attr/README
> > @@ -60,6 +60,7 @@ Following tests are defined (with perf commands):
> >    perf record -R kill                           (test-record-raw)
> >    perf record -c 2 -e arm_spe_0// -- kill       (test-record-spe-period)
> >    perf record -e arm_spe_0/period=3/ -- kill    (test-record-spe-period-term)
> > +  perf record -e arm_spe_0/pa_enable=1/ -- kill (test-record-spe-physical-address)
> >    perf stat -e cycles kill                      (test-stat-basic)
> >    perf stat kill                                (test-stat-default)
> >    perf stat -d kill                             (test-stat-detailed-1)
> > diff --git a/tools/perf/tests/attr/test-record-spe-physical-address b/tools/perf/tests/attr/test-record-spe-physical-address
> > new file mode 100644
> > index 000000000000..7ebcf5012ce3
> > --- /dev/null
> > +++ b/tools/perf/tests/attr/test-record-spe-physical-address
> > @@ -0,0 +1,12 @@
> > +[config]
> > +command = record
> > +args    = --no-bpf-event -e arm_spe_0/pa_enable=1/ -- kill >/dev/null 2>&1
> > +ret     = 1
> > +arch    = aarch64
> > +
> > +[event-10:base-record-spe]
> > +# 622727 is the decimal of IP|TID|TIME|CPU|IDENTIFIER|DATA_SRC|PHYS_ADDR
> > +sample_type=622727
> > +
> > +# dummy event
> > +[event-1:base-record-spe]
> > \ No newline at end of file
> > -- 
> > 2.25.1
> > 

-- 

- Arnaldo