linux-kernel - Re: [PATCH v5 0/4] perf vendor events: Support PMU events for A64FX

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]

Date:   Mon, 8 Feb 2021 08:57:30 +0000
From:   John Garry <john.garry@...wei.com>
To:     "nakamura.shun@...itsu.com" <nakamura.shun@...itsu.com>,
        "'will@...nel.org'" <will@...nel.org>,
        "'mathieu.poirier@...aro.org'" <mathieu.poirier@...aro.org>,
        "'leo.yan@...aro.org'" <leo.yan@...aro.org>,
        "'peterz@...radead.org'" <peterz@...radead.org>,
        "'mingo@...hat.com'" <mingo@...hat.com>,
        "'acme@...nel.org'" <acme@...nel.org>,
        "'mark.rutland@....com'" <mark.rutland@....com>,
        "'alexander.shishkin@...ux.intel.com'" 
        <alexander.shishkin@...ux.intel.com>,
        "'jolsa@...hat.com'" <jolsa@...hat.com>,
        "'namhyung@...nel.org'" <namhyung@...nel.org>
CC:     "'linux-kernel@...r.kernel.org'" <linux-kernel@...r.kernel.org>,
        "'linux-arm-kernel@...ts.infradead.org'" 
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v5 0/4] perf vendor events: Support PMU events for A64FX

On 08/02/2021 01:53, nakamur'd a.shun@...itsu.com wrote:
> Hi, John
> 
>>>> Apart from that, I think that we're a bit uncertain about patch 3/4
>>> What are your concerns?
>>> I think it's okay for perf to read a new event code with a number at the
>> beginning.
>>
>> The impact of this fix is on {name} and later rules.
>> parse_events.l uses {name} only in one place.
>> The only rule defined after {name} is {name_tag}.
>>
>> I think the point of current fix is that the rules defined after {name} are not
>> misrecognize and the syntax is not broken.
>> {name_tag} starts with ', but {name} does not contain'.
>> Therefore, the corrected {name} does not misrecognize the {name_tag}, and I
>> think there is no problem.
> Do you have any advice?

Hi,

So my series is now on remotes/origin/tmp.perf/core in Arnaldo's acme 
git, so you could resend against that.

https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git

Or again, put my series on perf/core and send against that.

As for patch 3/4, firstly I'd say that it is not a 'fix'. As well, 
please ensure it causes no regression on x86 or arm64. So like "perf 
list" output is same as before (with just that change), and also test 
some same perf events and metrics (x86 only) to ensure that they are ok. 
Please also ensure no regression on "perf test".

BTW, please config your mail client for no text encoding.

Thanks,
John