perf tools: Parse tracepoints with '-' in system name
authorAlexander Yarygin <yarygin@linux.vnet.ibm.com>
Fri, 25 Apr 2014 15:34:05 +0000 (17:34 +0200)
committerJiri Olsa <jolsa@kernel.org>
Tue, 29 Apr 2014 12:27:23 +0000 (14:27 +0200)
Trace events potentially can have a '-' in their trace system name,
e.g. kvm on s390 defines kvm-s390:* tracepoints.
We could not parse them, because there was no rule for this:
  $ sudo ./perf top -e "kvm-s390:*"
  invalid or unsupported event: 'kvm-s390:*'

This patch adds an extra rule to event_legacy_tracepoint which handles
those cases. Without the patch, perf will not accept such tracepoints in
the -e option.

Signed-off-by: Alexander Yarygin <yarygin@linux.vnet.ibm.com>
Tested-by: Christian Borntraeger <borntraeger@de.ibm.com>
Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
Link: http://lkml.kernel.org/r/1398440047-6641-2-git-send-email-yarygin@linux.vnet.ibm.com
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
tools/perf/util/parse-events.y

index 4eb67ec333f19ce05f3c7ded7e6999e97af99f04..ac9db9f699f3f583e0ebd99baec9aa52eef9e029 100644 (file)
@@ -299,6 +299,18 @@ PE_PREFIX_MEM PE_VALUE sep_dc
 }
 
 event_legacy_tracepoint:
+PE_NAME '-' PE_NAME ':' PE_NAME
+{
+       struct parse_events_evlist *data = _data;
+       struct list_head *list;
+       char sys_name[128];
+       snprintf(&sys_name, 128, "%s-%s", $1, $3);
+
+       ALLOC_LIST(list);
+       ABORT_ON(parse_events_add_tracepoint(list, &data->idx, &sys_name, $5));
+       $$ = list;
+}
+|
 PE_NAME ':' PE_NAME
 {
        struct parse_events_evlist *data = _data;