From: Nikita Shubin <nikita.shubin@maquefel.me> To: John Garry <john.garry@huawei.com> Cc: "Genevieve Chan" <genevieve.chan@starfivetech.com>, "João Mário Domingos" <joao.mario@tecnico.ulisboa.pt>, "Nikita Shubin" <n.shubin@yadro.com>, "Peter Zijlstra" <peterz@infradead.org>, "Ingo Molnar" <mingo@redhat.com>, "Arnaldo Carvalho de Melo" <acme@kernel.org>, "Mark Rutland" <mark.rutland@arm.com>, "Alexander Shishkin" <alexander.shishkin@linux.intel.com>, "Jiri Olsa" <jolsa@kernel.org>, "Namhyung Kim" <namhyung@kernel.org>, "Paul Walmsley" <paul.walmsley@sifive.com>, "Palmer Dabbelt" <palmer@dabbelt.com>, "Albert Ou" <aou@eecs.berkeley.edu>, linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, linux-riscv@lists.infradead.org, "Atish Patra" <atishp@atishpatra.org> Subject: Re: [PATCH v3 3/4] RISC-V: Added generic pmu-events mapfile Date: Wed, 8 Jun 2022 17:41:07 +0300 [thread overview] Message-ID: <20220608174107.7b94e545@redslave.neermore.group> (raw) In-Reply-To: <9e981523-6032-fd24-08f9-0ef771cd1c2e@huawei.com> Hello, John. On Wed, 8 Jun 2022 11:45:53 +0100 John Garry <john.garry@huawei.com> wrote: > On 07/06/2022 14:16, Nikita Shubin wrote: > > From: João Mário Domingos <joao.mario@tecnico.ulisboa.pt> > > > > The pmu-events now supports custom events for RISC-V, plus the > > cycle, time and instret events were defined. > > > > Signed-off-by: João Mário Domingos <joao.mario@tecnico.ulisboa.pt> > > Tested-by: Nikita Shubin <n.shubin@yadro.com> > > --- > > tools/perf/pmu-events/arch/riscv/mapfile.csv | 14 +++++++++++++ > > .../pmu-events/arch/riscv/riscv-generic.json | 20 > > +++++++++++++++++++ 2 files changed, 34 insertions(+) > > create mode 100644 tools/perf/pmu-events/arch/riscv/mapfile.csv > > create mode 100644 > > tools/perf/pmu-events/arch/riscv/riscv-generic.json > > > > diff --git a/tools/perf/pmu-events/arch/riscv/mapfile.csv > > b/tools/perf/pmu-events/arch/riscv/mapfile.csv new file mode 100644 > > index 000000000000..4f2aa199d9cb > > --- /dev/null > > +++ b/tools/perf/pmu-events/arch/riscv/mapfile.csv > > @@ -0,0 +1,14 @@ > > +# Format: > > +# MIDR,Version,JSON/file/pathname,Type > > +# > > +# where > > +# MIDR Processor version > > ARM, no? I've messed with --cc-cmd badly and didn't include every one in cover letter, sorry for that, attaching link to cover letter: https://lore.kernel.org/all/20220607131648.29439-1-nikita.shubin@maquefel.me/ They are ARM inspired indeed. > > > +# Variant[23:20] and Revision [3:0] should be zero. > > +# Version could be used to track version of JSON file > > +# but currently unused. > > +# JSON/file/pathname is the path to JSON file, relative > > +# to tools/perf/pmu-events/arch/riscv/. > > +# Type is core, uncore etc > > +# > > +# > > +#Family-model,Version,Filename,EventType > > diff --git a/tools/perf/pmu-events/arch/riscv/riscv-generic.json > > b/tools/perf/pmu-events/arch/riscv/riscv-generic.json new file mode > > 100644 index 000000000000..013e50efad99 > > --- /dev/null > > +++ b/tools/perf/pmu-events/arch/riscv/riscv-generic.json > > where or how are these referenced? Currently they are not referenced in this version of series at all, their purpose is to be used like "ArchStdEvent". Through any RISCV implementation should have at least these 3 events. Yours, Nikita Shubin. > > > @@ -0,0 +1,20 @@ > > +[ > > + { > > + "PublicDescription": "CPU Cycles", > > + "EventCode": "0x00", > > + "EventName": "riscv_cycles", > > + "BriefDescription": "CPU cycles RISC-V generic counter" > > + }, > > + { > > + "PublicDescription": "CPU Time", > > + "EventCode": "0x01", > > + "EventName": "riscv_time", > > + "BriefDescription": "CPU time RISC-V generic counter" > > + }, > > + { > > + "PublicDescription": "CPU Instructions", > > + "EventCode": "0x02", > > + "EventName": "riscv_instret", > > + "BriefDescription": "CPU retired instructions RISC-V generic > > counter" > > + } > > +] > > \ No newline at end of file > _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
WARNING: multiple messages have this Message-ID (diff)
From: Nikita Shubin <nikita.shubin@maquefel.me> To: John Garry <john.garry@huawei.com> Cc: "Genevieve Chan" <genevieve.chan@starfivetech.com>, "João Mário Domingos" <joao.mario@tecnico.ulisboa.pt>, "Nikita Shubin" <n.shubin@yadro.com>, "Peter Zijlstra" <peterz@infradead.org>, "Ingo Molnar" <mingo@redhat.com>, "Arnaldo Carvalho de Melo" <acme@kernel.org>, "Mark Rutland" <mark.rutland@arm.com>, "Alexander Shishkin" <alexander.shishkin@linux.intel.com>, "Jiri Olsa" <jolsa@kernel.org>, "Namhyung Kim" <namhyung@kernel.org>, "Paul Walmsley" <paul.walmsley@sifive.com>, "Palmer Dabbelt" <palmer@dabbelt.com>, "Albert Ou" <aou@eecs.berkeley.edu>, linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, linux-riscv@lists.infradead.org, "Atish Patra" <atishp@atishpatra.org> Subject: Re: [PATCH v3 3/4] RISC-V: Added generic pmu-events mapfile Date: Wed, 8 Jun 2022 17:41:07 +0300 [thread overview] Message-ID: <20220608174107.7b94e545@redslave.neermore.group> (raw) In-Reply-To: <9e981523-6032-fd24-08f9-0ef771cd1c2e@huawei.com> Hello, John. On Wed, 8 Jun 2022 11:45:53 +0100 John Garry <john.garry@huawei.com> wrote: > On 07/06/2022 14:16, Nikita Shubin wrote: > > From: João Mário Domingos <joao.mario@tecnico.ulisboa.pt> > > > > The pmu-events now supports custom events for RISC-V, plus the > > cycle, time and instret events were defined. > > > > Signed-off-by: João Mário Domingos <joao.mario@tecnico.ulisboa.pt> > > Tested-by: Nikita Shubin <n.shubin@yadro.com> > > --- > > tools/perf/pmu-events/arch/riscv/mapfile.csv | 14 +++++++++++++ > > .../pmu-events/arch/riscv/riscv-generic.json | 20 > > +++++++++++++++++++ 2 files changed, 34 insertions(+) > > create mode 100644 tools/perf/pmu-events/arch/riscv/mapfile.csv > > create mode 100644 > > tools/perf/pmu-events/arch/riscv/riscv-generic.json > > > > diff --git a/tools/perf/pmu-events/arch/riscv/mapfile.csv > > b/tools/perf/pmu-events/arch/riscv/mapfile.csv new file mode 100644 > > index 000000000000..4f2aa199d9cb > > --- /dev/null > > +++ b/tools/perf/pmu-events/arch/riscv/mapfile.csv > > @@ -0,0 +1,14 @@ > > +# Format: > > +# MIDR,Version,JSON/file/pathname,Type > > +# > > +# where > > +# MIDR Processor version > > ARM, no? I've messed with --cc-cmd badly and didn't include every one in cover letter, sorry for that, attaching link to cover letter: https://lore.kernel.org/all/20220607131648.29439-1-nikita.shubin@maquefel.me/ They are ARM inspired indeed. > > > +# Variant[23:20] and Revision [3:0] should be zero. > > +# Version could be used to track version of JSON file > > +# but currently unused. > > +# JSON/file/pathname is the path to JSON file, relative > > +# to tools/perf/pmu-events/arch/riscv/. > > +# Type is core, uncore etc > > +# > > +# > > +#Family-model,Version,Filename,EventType > > diff --git a/tools/perf/pmu-events/arch/riscv/riscv-generic.json > > b/tools/perf/pmu-events/arch/riscv/riscv-generic.json new file mode > > 100644 index 000000000000..013e50efad99 > > --- /dev/null > > +++ b/tools/perf/pmu-events/arch/riscv/riscv-generic.json > > where or how are these referenced? Currently they are not referenced in this version of series at all, their purpose is to be used like "ArchStdEvent". Through any RISCV implementation should have at least these 3 events. Yours, Nikita Shubin. > > > @@ -0,0 +1,20 @@ > > +[ > > + { > > + "PublicDescription": "CPU Cycles", > > + "EventCode": "0x00", > > + "EventName": "riscv_cycles", > > + "BriefDescription": "CPU cycles RISC-V generic counter" > > + }, > > + { > > + "PublicDescription": "CPU Time", > > + "EventCode": "0x01", > > + "EventName": "riscv_time", > > + "BriefDescription": "CPU time RISC-V generic counter" > > + }, > > + { > > + "PublicDescription": "CPU Instructions", > > + "EventCode": "0x02", > > + "EventName": "riscv_instret", > > + "BriefDescription": "CPU retired instructions RISC-V generic > > counter" > > + } > > +] > > \ No newline at end of file >
next prev parent reply other threads:[~2022-06-08 14:41 UTC|newest] Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-06-07 13:16 [PATCH v3 0/4] Introduce pmu-events support for HiFive Unmatched Nikita Shubin 2022-06-07 13:16 ` [PATCH v3 1/4] RISC-V: Create unique identification for SoC PMU Nikita Shubin 2022-06-07 13:16 ` Nikita Shubin 2022-06-07 13:16 ` Nikita Shubin 2022-06-07 16:37 ` Atish Patra 2022-06-07 16:37 ` Atish Patra 2022-06-07 16:37 ` Atish Patra 2022-06-08 8:47 ` Nikita Shubin 2022-06-08 8:47 ` Nikita Shubin 2022-06-08 8:47 ` Nikita Shubin 2022-06-14 10:16 ` Sunil V L 2022-06-14 10:16 ` Sunil V L 2022-06-14 10:16 ` Sunil V L 2022-06-07 13:16 ` [PATCH v3 2/4] RISC-V: Support CPUID for risc-v in perf Nikita Shubin 2022-06-07 13:16 ` Nikita Shubin 2022-06-07 13:16 ` Nikita Shubin 2022-06-09 13:54 ` Will Deacon 2022-06-09 13:54 ` Will Deacon 2022-06-09 13:54 ` Will Deacon 2022-06-07 13:16 ` [PATCH v3 3/4] RISC-V: Added generic pmu-events mapfile Nikita Shubin 2022-06-07 13:16 ` Nikita Shubin 2022-06-08 10:45 ` John Garry 2022-06-08 10:45 ` John Garry 2022-06-08 14:41 ` Nikita Shubin [this message] 2022-06-08 14:41 ` Nikita Shubin 2022-06-08 15:51 ` John Garry 2022-06-08 15:51 ` John Garry 2022-06-07 13:16 ` [PATCH v3 4/4] RISC-V: Added HiFive Unmatched PMU events Nikita Shubin 2022-06-07 13:16 ` Nikita Shubin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20220608174107.7b94e545@redslave.neermore.group \ --to=nikita.shubin@maquefel.me \ --cc=acme@kernel.org \ --cc=alexander.shishkin@linux.intel.com \ --cc=aou@eecs.berkeley.edu \ --cc=atishp@atishpatra.org \ --cc=genevieve.chan@starfivetech.com \ --cc=joao.mario@tecnico.ulisboa.pt \ --cc=john.garry@huawei.com \ --cc=jolsa@kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-perf-users@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=mark.rutland@arm.com \ --cc=mingo@redhat.com \ --cc=n.shubin@yadro.com \ --cc=namhyung@kernel.org \ --cc=palmer@dabbelt.com \ --cc=paul.walmsley@sifive.com \ --cc=peterz@infradead.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.