Re: [PATCHv3 0/3] perf tool: Add new event group management

From: Namhyung Kim
Date: Sun Jul 01 2012 - 21:57:56 EST


Hi, Jiri

On Fri, 29 Jun 2012 11:08:24 +0200, Jiri Olsa wrote:
> hi,
> adding support for creating event groups based on the way they
> are specified on the command line.
>
> This patchset adds the '{}' style grammar to express event group,
> allowing so far only the 'event modifier' as group modifier.
>
> The sample/leader modifier changes are ready, but I'm getting
> some weird numbers in tests, so I'll send it later. Meanwhile,
> it'd be nice to have at least this patchset in.. ;)
>

Just a question, is there a way to know about the grouping at perf
report time?

Thanks,
Namhyung


>
> Attached patches:
> 1/3 perf, tool: Add support to parse event group syntax
> 2/3 perf, tool: Enable grouping logic for parsed events
> 3/3 perf, test: Add automated tests for event group parsing
>
> wbr,
> jirka
> ---
> tools/perf/builtin-record.c | 13 +--
> tools/perf/builtin-stat.c | 13 +--
> tools/perf/builtin-test.c | 8 +-
> tools/perf/builtin-top.c | 12 +--
> tools/perf/util/evlist.c | 20 ++---
> tools/perf/util/evlist.h | 3 +-
> tools/perf/util/evsel.c | 51 ++++++++----
> tools/perf/util/evsel.h | 11 +--
> tools/perf/util/parse-events-test.c | 152 +++++++++++++++++++++++++++++++++++
> tools/perf/util/parse-events.c | 32 +++++++-
> tools/perf/util/parse-events.h | 5 +-
> tools/perf/util/parse-events.l | 2 +
> tools/perf/util/parse-events.y | 93 ++++++++++++++++++---
> tools/perf/util/python.c | 7 +-
> 14 files changed, 341 insertions(+), 81 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/