Messages in this thread | | | Subject | Re: [PATCH 2/2] perf vendor events riscv: add T-HEAD C9xx JSON file | From | Nikita Shubin <> | Date | Tue, 16 May 2023 18:06:39 +0300 |
| |
On Tue, 2023-05-16 at 17:45 +0800, Inochi Amaoto wrote: > > Do c906 and c910 have same HPM events ? > > > > https://github.com/T-head-Semi/openc906/blob/main/C906_RTL_FACTORY/gen_rtl/pmu/rtl/aq_hpcp_top.v > > > > and > > > > > > https://github.com/T-head-Semi/openc910/blob/main/C910_RTL_FACTORY/gen_rtl/pmu/rtl/ct_hpcp_top.v > > > > Look different to me - am i missing something ? > > Yes, they as different, but event ids are compatible. See [1] p.99 > and [2] p.73. > > It seems I forgot extra event index from C906. I will fix in the v2. > > [1] > https://github.com/T-head-Semi/openc910/blob/main/doc/%E7%8E%84%E9%93%81C910%E7%94%A8%E6%88%B7%E6%89%8B%E5%86%8C.pdf > [2] > https://github.com/T-head-Semi/openc906/blob/main/doc/%E7%8E%84%E9%93%81C906%E7%94%A8%E6%88%B7%E6%89%8B%E5%86%8C.pdf
They are not the same - they are different in many ways. And c906 list seems more complete to me.
I think you should drop wildcard (it shouldn't be used anyway) then and name it c906.
"0x5b7-0x0-0x0,v1,t-head/c906,core" would match both c906 and c910.
> > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv
| |