Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 21 Aug 2020 09:07:58 -0300
From:   Arnaldo Carvalho de Melo <acme@kernel.org>
To:     Will Deacon <will@kernel.org>
Cc:     Mathieu Poirier <mathieu.poirier@linaro.org>,
        john.garry@huawei.com, leo.yan@linaro.org, jolsa@redhat.com,
        mark.rutland@arm.com, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] MAINTAINERS: Add entries for CoreSight and Arm SPE
Message-ID: <20200821120758.GE2667554@kernel.org>
References: <20200820175510.3935932-1-mathieu.poirier@linaro.org>
 <20200821090531.GC20255@willie-the-truck>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200821090531.GC20255@willie-the-truck>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Em Fri, Aug 21, 2020 at 10:05:31AM +0100, Will Deacon escreveu:
> On Thu, Aug 20, 2020 at 11:55:10AM -0600, Mathieu Poirier wrote:
> > Add entries for perf tools elements related to the support of Arm CoreSight
> > and Arm SPE.  Also lump in Arm and Arm64 architecture files to provide
> > coverage.
> > 
> > Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
> > ---
> > V2:
> > - Completed fileset for SPE.
> > - Added Arm and Arm64 architecture files.
> > 
> >  MAINTAINERS | 8 +++++++-
> >  1 file changed, 7 insertions(+), 1 deletion(-)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index deaafb617361..e76f7bb014ce 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -13569,12 +13569,18 @@ F:	kernel/events/*
> >  F:	tools/lib/perf/
> >  F:	tools/perf/
> >  
> > -PERFORMANCE EVENTS SUBSYSTEM ARM64 PMU EVENTS
> > +PERFORMANCE EVENTS SUBSYSTEM ARM64
> 
> I'd probably prefer to go with TOOLING instead of SUBSYSTEM, since the
> kernel parts are covered by the "ARM PMU PROFILING AND DEBUGGING" entry.

Yeah, I think its appropriate to have TOOLING instead of SUBSYSTEM, that
is more commonly used when referring to the kernel.

I'm applying with that change, locally, with your and John's Acked-by,
please holler if that is somehow controversial.

- Arnaldo
 
> >  R:	John Garry <john.garry@huawei.com>
> >  R:	Will Deacon <will@kernel.org>
> > +R:	Mathieu Poirier <mathieu.poirier@linaro.org>
> > +R:	Leo Yan <leo.yan@linaro.org>
> >  L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> >  S:	Supported
> > +F:	tools/build/feature/test-libopencsd.c
> > +F:	tools/perf/arch/arm*/
> >  F:	tools/perf/pmu-events/arch/arm64/
> > +F:	tools/perf/util/arm-spe*
> > +F:	tools/perf/util/cs-etm*
> 
> Either way,
> 
> Acked-by: Will Deacon <will@kernel.org>
> 
> Thanks,
> 
> Will

-- 

- Arnaldo