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

[Ganapatrao Kulkarni]

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
>
> -Robert
>
>>  tools/perf/util/header.h                           |  3 +-
>>  tools/perf/util/pmu.c                              | 52 +++++++++++++----
>>  8 files changed, 189 insertions(+), 13 deletions(-)

thanks
Ganapat