[PATCH v7 2/4] perf tools: Add lexical definition of event name

Jiri Olsa jolsa at redhat.com
Sat Mar 6 19:14:13 GMT 2021


On Fri, Mar 05, 2021 at 10:09:20AM +0000, nakamura.shun at fujitsu.com wrote:
> Hi, John
> 
> > > From: Shunsuke Nakamura <nakamura.shun at jp.fujitsu.com>
> > > 
> > > Add the lexical definition of event name so that the numbers are recognizable.
> > > 
> > 
> > Sorry for the hassle, but it's not clear that we got a definitive 
> > conclusion on this patch from v6 series:
> > 
> > https://lore.kernel.org/lkml/YCuo01Wxxe%2FMCrC5@krava/
> > 
> > Would it be so much worse for you to change format of these event names 
> > to not start with a numberic?
> I was trying to match the specifications published by Fujitsu exactly.
> We also thought that being able to parse event names starting with a number would be an improvement for perf.
> We've discussed it with the team, but we decided to change the event name.

hum, I'm not sure what's the final decision in here.. are you keeping
numbers in the start? would be much easier if you did not ;-)

jirka




More information about the linux-arm-kernel mailing list