Re: [PATCH] perf vendor events arm64: Add Cortex-A72 events

From: Florian Fainelli
Date: Tue Apr 09 2019 - 10:30:59 EST




On 4/8/2019 9:47 AM, Will Deacon wrote:
> Hi Florian,
>
> On Sun, Apr 07, 2019 at 02:34:22PM -0700, Florian Fainelli wrote:
>> The Cortex-A72 supports all ARMv8 recommended events up to the
>> RC_ST_SPEC (0x91) event, create an appropriate JSON file for mapping
>> those events and update the mapfile.csv for matching the Cortex-A72 MIDR
>> to that file.
>>
>> Signed-off-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
>> ---
>> .../arm64/arm/cortex-a72/core-imp-def.json | 206 ++++++++++++++++++
>> tools/perf/pmu-events/arch/arm64/mapfile.csv | 1 +
>> 2 files changed, 207 insertions(+)
>> create mode 100644 tools/perf/pmu-events/arch/arm64/arm/cortex-a72/core-imp-def.json
>
> It would be worth checking how many of these events are common between a57
> and a72, as they may be able to share much of the json file.

Makes sense, will check the Cortex-A57 TRM and re-submit accordingly.
Thanks!
--
Florian