Thread (14 messages) flat view 14 messages, 3 authors, 2021-02-02

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

From: John Garry <hidden>
Date: 2021-02-01 14:38:31
Also in: lkml

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 Nakamura <redacted>
eh... I'm not sure if you will encounter other problems, so may be 
prudent to change the event name to not start with a number. Someone 
else may also advise on this.
quoted hunk ↗ jump to hunk
---
  tools/perf/util/parse-events.l | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/util/parse-events.l b/tools/perf/util/parse-events.l
index 9db5097..e6bbbd0 100644
--- a/tools/perf/util/parse-events.l
+++ b/tools/perf/util/parse-events.l
@@ -205,7 +205,7 @@ bpf_source	[^,{}]+\.c[a-zA-Z0-9._]*
  num_dec		[0-9]+
  num_hex		0x[a-fA-F0-9]+
  num_raw_hex	[a-fA-F0-9]+
-name		[a-zA-Z_*?\[\]][a-zA-Z0-9_*?.\[\]]*
+name		[a-zA-Z0-9_*?\[\]][a-zA-Z0-9_*?.\[\]]*
  name_tag	[\'][a-zA-Z_*?\[\]][a-zA-Z0-9_*?\-,\.\[\]:=]*[\']
  name_minus	[a-zA-Z_*?][a-zA-Z0-9\-_*?.:]*
  drv_cfg_term	[a-zA-Z0-9_\.]+(=[a-zA-Z0-9_*?\.:]+)?

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help