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

[Namhyung Kim <namhyung@kernel.org>]

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(-)