linux-kernel - Re: [PATCH v2] MAINTAINERS: Add entries for CoreSight and Arm SPE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20200821120758.GE2667554@kernel.org>
Date:   Fri, 21 Aug 2020 09:07:58 -0300
From:   Arnaldo Carvalho de Melo <acme@...nel.org>
To:     Will Deacon <will@...nel.org>
Cc:     Mathieu Poirier <mathieu.poirier@...aro.org>,
        john.garry@...wei.com, leo.yan@...aro.org, jolsa@...hat.com,
        mark.rutland@....com, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] MAINTAINERS: Add entries for CoreSight and Arm SPE

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@...aro.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@...wei.com>
> >  R:	Will Deacon <will@...nel.org>
> > +R:	Mathieu Poirier <mathieu.poirier@...aro.org>
> > +R:	Leo Yan <leo.yan@...aro.org>
> >  L:	linux-arm-kernel@...ts.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@...nel.org>
> 
> Thanks,
> 
> Will

-- 

- Arnaldo