Re: [PATCH v2] perf/cxlpmu: Support missing events in 3.1 spec

[Jonathan Cameron <Jonathan.Cameron@Huawei.com>]

On Wed, 9 Oct 2024 19:45:05 -0700
Davidlohr Bueso <dave@stgolabs.net> wrote:

> On Fri, 04 Oct 2024, Jonathan Cameron wrote:\n
> 
> >Driver is in perf though so you need to +CC perf maintainers as
> >they will probably pick this up directly.  
> 
> Doing the v3 to add the typo Alison pointed out, I remembered
> why I didn't Cc a perf list. Which is it? it is not clear to me
> exactly who is responsible to pick this up outside of the cxl
> world:
> 
> $ ./scripts/get_maintainer.pl 0001-perf-cxlpmu-Support-missing-events-in-3.1-spec.patch
> Jonathan Cameron <jonathan.cameron@huawei.com> (maintainer:COMPUTE EXPRESS LINK PMU (CPMU))
> Will Deacon <will@kernel.org> (maintainer:ARM PMU PROFILING AND DEBUGGING)
> Mark Rutland <mark.rutland@arm.com> (maintainer:ARM PMU PROFILING AND DEBUGGING)
> linux-cxl@vger.kernel.org (open list:COMPUTE EXPRESS LINK PMU (CPMU))
> linux-arm-kernel@lists.infradead.org (moderated list:ARM PMU PROFILING AND DEBUGGING)
> linux-kernel@vger.kernel.org (open list)
> 
> ... Yeah I guess it's Will and Mark, but arm seems really
> misleading and out of place for this. Imo both the cxl driver
> and pmu code should have the same maintainer scope.

There was a discussion ages ago about changing that maintainer entry
to make it more generic (drivers/perf).  With a few exceptions the reality
is that Will and Mark end up reviewing everything in there :)

Generally for PMU drivers the perf specific aspects end up more complex
than the hardware side. That's why we put the driver in drivers/perf.

I don't think either Mark / Will / CXL maintainers really care a lot
about who does the patch wrangling on simple patches like this beyond
if it is from perf side no one wonders what the CXL maintainers are doing
touching code in drivers/perf.

Jonathan


> 
> But even if this does not change, I think the maintainer entries
> should at least be labeled something more generic.
> 
> Thanks,
> Davidlohr