RE: [PATCH v4 3/4] perf tools: Fix lexical definition of event name

2021-02-02 Thread nakamura.s...@fujitsu.com
(I missed to include the quotation in the text. Will resend. Sorry about that.) Hi, John > > Fix the lexical definition of event name so that the numbers are > > recognizable. > > > > A64FX defines an event name that starts with a number. > > - 0inst_commit > > - 1inst_commit > > - 2inst_c

RE: [PATCH v4 3/4] perf tools: Fix lexical definition of event name

2021-02-02 Thread nakamura.s...@fujitsu.com
Hi, John I have applied this patch and confirmed that the event can be specified correctly. Event names are defined in the A64FX specification. I want to deal with it at that time if there is a problem. So I would like to resend the v5 patch and wait for advice from others. Best Regards Shunsuk

Re: [PATCH v4 3/4] perf tools: Fix lexical definition of event name

2021-02-01 Thread John Garry
On 01/02/2021 11:13, Shunsuke Nakamura wrote: Fix the lexical definition of event name so that the numbers are recognizable. A64FX defines an event name that starts with a number. - 0inst_commit - 1inst_commit - 2inst_commit - 3inst_commit - 4inst_commit Signed-off-by: Shunsuke Nakamu