linux-kernel - Re: [PATCH 0/4] HiSilicon hip08 uncore PMU events additions

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <6990f60b-2df4-b20c-2777-090bc4d9b6db@huawei.com>
Date:   Mon, 7 Oct 2019 14:40:39 +0100
From:   John Garry <john.garry@...wei.com>
To:     Arnaldo Carvalho de Melo <arnaldo.melo@...il.com>
CC:     Peter Zijlstra <peterz@...radead.org>,
        Ingo Molnar <mingo@...nel.org>,
        Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
        Jiri Olsa <jolsa@...nel.org>,
        Namhyung Kim <namhyung@...nel.org>,
        <linux-kernel@...r.kernel.org>, <linuxarm@...wei.com>,
        <will@...nel.org>, <mark.rutland@....com>,
        <zhangshaokun@...ilicon.com>, James Clark <James.Clark@....com>,
        Ganapatrao Kulkarni <gklkml16@...il.com>,
        "William Cohen" <wcohen@...hat.com>,
        Florian Fainelli <f.fainelli@...il.com>
Subject: Re: [PATCH 0/4] HiSilicon hip08 uncore PMU events additions

>
>> As an alternative, how about just add a maintainers entry for reviewers per
>> arch? As a start, I don't mind being added there for arm64:
>>
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -12767,6 +12767,10 @@ F:     arch/*/events/*
>>  F:     arch/*/events/*/*
>>  F:     tools/perf/
>>
>> +PERFORMANCE EVENTS SUBSYSTEM ARM64 PMU EVENTS
>> +R:     John Garry <john.garry@...wei.com>
>> +F:     tools/perf/pmu-events/arch/arm64
>> +
>>
>> Patches per-arch should have some nod/tag from a member of the respective
>> list. Or at very least be cc'ed :)
>
> Another Ok, please send a formal patch, and it would be really nice if
> the other ARM folks would... Ack that ;-) :-) And provide extra entries
> for the other pmu-events directories or even for specific files, which
> is a possibility, right?

ok, can do. Will has kindly agreed to have his name added to the 
MAINTAINERS entry (thanks). Other Cc's from ARM community may also be 
interested to be included - shout if so.

So I'll just include tools/perf/pmu-events/arch/arm64 for now.

The code in tools/perf/pmu-events/. should be generic for all 
architectures, while I'd say tools/perf/arch/arm64 is not strictly related.

Thanks,
John

>
> On my side I'll script a bit more and make sure that a post commit hook
> warns me if the right tag is not present.
>
> - Arnaldo
>
> .
>