Re: [PATCH] perf/tool: Fix usage of trace events with '-' in trace system name.

2014-04-21 Thread Alexander Yarygin
At Thu, 17 Apr 2014 13:32:21 +0200, Jiri Olsa wrote: > > On Tue, Mar 25, 2014 at 11:15:29AM +0100, Paolo Bonzini wrote: > > Il 24/03/2014 21:49, Christian Borntraeger ha scritto: > > > event_legacy_tracepoint: > > >+PE_NAME '-' PE_NAME ':' PE_NAME > > >+{ > > >+ struct parse_events_evlist *data =

[PATCH 1/3] perf tools: parse tracepooints with '-' in system name

2014-04-25 Thread Alexander Yarygin
is 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 --- tools/perf/util/parse-events.y | 12 1 file changed, 12 insertions(+) diff --git a/to

[PATCH 3/3] perf-test: Add a test of kvm-390: trace event

2014-04-25 Thread Alexander Yarygin
Add a s390 specific test of a hardcoded trace event with '-' in the name. Signed-off-by: Alexander Yarygin --- tools/perf/tests/parse-events.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/tools/perf/tests/parse-events.c b/tools/perf/tests/parse-events.c index 81dbd5

[PATCH v2 0/3] Fix usage of trace events with '-' in trace system name.

2014-04-25 Thread Alexander Yarygin
Patch 2/3 makes possible to add arch specific tests to perf/tests/parse-events.c. Alexander Yarygin (3): perf tools: parse tracepooints with '-' in system name perf-test: add numeric identifier to evlist_test perf-test: Add a test of kvm-390: trace event tools/perf/tests/p

[PATCH 2/3] perf-test: add numeric identifier to evlist_test

2014-04-25 Thread Alexander Yarygin
removes explicit numbering of array elements. This helps to number tests with gaps. Signed-off-by: Alexander Yarygin --- tools/perf/tests/parse-events.c | 135 ++- 1 file changed, 90 insertions(+), 45 deletions(-) diff --git a/tools/perf/tests/parse-events.c b/to