Re: [PATCH v7 0/4] Add support for ThunderX2 pmu events using json files

[Zhangshaokun]

Hi Will,

On 2017/10/12 18:58, Will Deacon wrote:
> On Thu, Oct 12, 2017 at 04:20:06PM +0530, Ganapatrao Kulkarni wrote:
>> Hi Robert,
>>
>> On Thu, Oct 12, 2017 at 4:08 PM, Robert Richter <rric@xxxxxxxxxx> wrote:
>>> On 12.10.17 16:04:15, Ganapatrao Kulkarni wrote:
>>>
>>>>  tools/perf/arch/arm64/util/Build                   |  1 +
>>>>  tools/perf/arch/arm64/util/header.c                | 65 ++++++++++++++++++++++
>>>>  tools/perf/arch/powerpc/util/header.c              |  2 +-
>>>>  tools/perf/arch/x86/util/header.c                  |  2 +-
>>>>  tools/perf/pmu-events/arch/arm64/mapfile.csv       | 15 +++++
>>>>  .../arch/arm64/thunderx2/thunderx2-imp-def.json    | 62 +++++++++++++++++++++
>>>
>>> I guess the dir needs to be named 'cavium'.
>>
>> i dont think so ,  while compilation arch type is detected, it has to be arm64
>> refer SRCARCH in pmu-events/Build
> 
> The full name should be:
> 
> .../arch/arm64/cavium/thunderx2-imp-def.json
> 

It works for us (HiSilicon) as .../arch/arm64/hisilicon/hip08-imp-def.json.

Thanks,
Shaokun

> Will
> 
> .
>