From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751017AbdJRRtb (ORCPT ); Wed, 18 Oct 2017 13:49:31 -0400 Received: from mail-lf0-f67.google.com ([209.85.215.67]:53996 "EHLO mail-lf0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750730AbdJRRt3 (ORCPT ); Wed, 18 Oct 2017 13:49:29 -0400 X-Google-Smtp-Source: ABhQp+SnNEko7tJao1VJ3VTN6Ft8KaoZPWhBHGmi91DoBdXk2EZT1Hszj/ZwLvu+XHBiSuBKcj4eCG3e6I1mu0Oj+Kg= MIME-Version: 1.0 In-Reply-To: <20171016183222.25750-1-ganapatrao.kulkarni@cavium.com> References: <20171016183222.25750-1-ganapatrao.kulkarni@cavium.com> From: Ganapatrao Kulkarni Date: Wed, 18 Oct 2017 23:19:28 +0530 Message-ID: Subject: Re: [PATCH v9 0/5] Add support for ThunderX2 pmu events using json files To: Ganapatrao Kulkarni Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Will Deacon , catalin.marinas@arm.com, mark.rutland@arm.com, Arnaldo Carvalho de Melo , Alexander Shishkin , Peter Zijlstra , Ingo Molnar , jnair@caviumnetworks.com, Zhangshaokun , Jonathan.Cameron@huawei.com, Robert Richter Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Arnaldo, Will, can this be queued to next? On Tue, Oct 17, 2017 at 12:02 AM, Ganapatrao Kulkarni wrote: > Extending json/jevent framework for parsing arm64 event files. > Adding jevents for ThunderX2 implementation defined PMU events. > > v9: > - Rebased to [3] and resolved conficts in PATCH 1/5 and reworked PATCH 3/5. > - Added PATCH 5 to fix segmentation fault in perf_pmu__find_map > > [3] https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/log/?h=perf/core > > v8: > - Rename thunderx2 directory to cavium > > v7: > - Addressed some more comments from Will Deacon. > > v6 : Addressed comments [1] [2] > > [1] https://patchwork.kernel.org/patch/9903099/ > [2] https://patchwork.kernel.org/patch/9853899/ > > v5: > - Addressed comments from Arnaldo. > - Rebased to 4.13-rc5 > > v4: > - Rebased to 4.13-rc1 > > v3: > - Addressed comments from Will Deacon and Jayachandran C. > - Rebased to 4.12-rc1 > > v2: > - Updated as per Mark Rutland's suggestions. > - Added provision for get_cpuid_str to get cpu id string > from associated cpus of pmu core device. > > v1: Initial patchset. > > > Ganapatrao Kulkarni (5): > perf utils: passing pmu as a parameter to function get_cpuid_str > perf tools arm64: Add support for get_cpuid_str function. > perf utils: use pmu->is_uncore to detect PMU CORE/UNCORE devices > perf vendor events arm64: Add ThunderX2 implementation defined pmu > core events > perf utils: add check for valid cpuid in function perf_pmu__find_map > > 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 +- > .../arch/arm64/cavium/thunderx2-imp-def.json | 62 +++++++++++++++++++++ > tools/perf/pmu-events/arch/arm64/mapfile.csv | 15 +++++ > tools/perf/util/header.h | 3 +- > tools/perf/util/metricgroup.c | 4 +- > tools/perf/util/pmu.c | 39 ++++++++----- > tools/perf/util/pmu.h | 2 +- > 10 files changed, 174 insertions(+), 21 deletions(-) > create mode 100644 tools/perf/arch/arm64/util/header.c > create mode 100644 tools/perf/pmu-events/arch/arm64/cavium/thunderx2-imp-def.json > create mode 100644 tools/perf/pmu-events/arch/arm64/mapfile.csv > > -- > 2.9.4 > thanks Ganapat From mboxrd@z Thu Jan 1 00:00:00 1970 From: gklkml16@gmail.com (Ganapatrao Kulkarni) Date: Wed, 18 Oct 2017 23:19:28 +0530 Subject: [PATCH v9 0/5] Add support for ThunderX2 pmu events using json files In-Reply-To: <20171016183222.25750-1-ganapatrao.kulkarni@cavium.com> References: <20171016183222.25750-1-ganapatrao.kulkarni@cavium.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Arnaldo, Will, can this be queued to next? On Tue, Oct 17, 2017 at 12:02 AM, Ganapatrao Kulkarni wrote: > Extending json/jevent framework for parsing arm64 event files. > Adding jevents for ThunderX2 implementation defined PMU events. > > v9: > - Rebased to [3] and resolved conficts in PATCH 1/5 and reworked PATCH 3/5. > - Added PATCH 5 to fix segmentation fault in perf_pmu__find_map > > [3] https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/log/?h=perf/core > > v8: > - Rename thunderx2 directory to cavium > > v7: > - Addressed some more comments from Will Deacon. > > v6 : Addressed comments [1] [2] > > [1] https://patchwork.kernel.org/patch/9903099/ > [2] https://patchwork.kernel.org/patch/9853899/ > > v5: > - Addressed comments from Arnaldo. > - Rebased to 4.13-rc5 > > v4: > - Rebased to 4.13-rc1 > > v3: > - Addressed comments from Will Deacon and Jayachandran C. > - Rebased to 4.12-rc1 > > v2: > - Updated as per Mark Rutland's suggestions. > - Added provision for get_cpuid_str to get cpu id string > from associated cpus of pmu core device. > > v1: Initial patchset. > > > Ganapatrao Kulkarni (5): > perf utils: passing pmu as a parameter to function get_cpuid_str > perf tools arm64: Add support for get_cpuid_str function. > perf utils: use pmu->is_uncore to detect PMU CORE/UNCORE devices > perf vendor events arm64: Add ThunderX2 implementation defined pmu > core events > perf utils: add check for valid cpuid in function perf_pmu__find_map > > 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 +- > .../arch/arm64/cavium/thunderx2-imp-def.json | 62 +++++++++++++++++++++ > tools/perf/pmu-events/arch/arm64/mapfile.csv | 15 +++++ > tools/perf/util/header.h | 3 +- > tools/perf/util/metricgroup.c | 4 +- > tools/perf/util/pmu.c | 39 ++++++++----- > tools/perf/util/pmu.h | 2 +- > 10 files changed, 174 insertions(+), 21 deletions(-) > create mode 100644 tools/perf/arch/arm64/util/header.c > create mode 100644 tools/perf/pmu-events/arch/arm64/cavium/thunderx2-imp-def.json > create mode 100644 tools/perf/pmu-events/arch/arm64/mapfile.csv > > -- > 2.9.4 > thanks Ganapat