Re: [PATCHv2 0/6] perf, tool: event parsing enhancements/fixes

[Namhyung Kim <namhyung.kim@lge.com>]

Hi,

On Wed, 25 Apr 2012 18:24:56 +0200, Jiri Olsa wrote:
> Hi,
> adding some enhancements/fixies for perf event parsing.
>
> v2 changes:
>    - rebased to current tip tree
>    - 2/6 - ommitting pmu tests in case there's no pmu sysfs record

Nice! It bothered me somewhat so I was thinking about fixing it.
I guess we have a same problem on x86 rdpmc test too?

Thanks,
Namhyung


>    - 6/6 - default the pmu event name to raw 0x<perf_event_attr::config>
>
> Attached patches:
>   1/6 perf, tool: Split term type into value type and term type
>   2/6 perf, tool: Move parse event automated tests to separated object
>   3/6 perf, tool: Add support for displaying event parser debug info
>   4/6 perf, tool: Use allocated list for each parsed event
>   5/6 perf, tool: Separate 'mem:' event scanner bits
>   6/6 perf, tool: Add hardcoded name term for pmu events
>
> thanks,
> jirka
> ---
>  tools/perf/Makefile                 |   13 +-
>  tools/perf/builtin-test.c           |  552 +------------------------------
>  tools/perf/util/parse-events-test.c |  625 +++++++++++++++++++++++++++++++++++
>  tools/perf/util/parse-events.c      |  114 +++++--
>  tools/perf/util/parse-events.h      |   42 ++-
>  tools/perf/util/parse-events.l      |   26 ++-
>  tools/perf/util/parse-events.y      |   93 ++++--
>  tools/perf/util/pmu.c               |   74 +++--
>  8 files changed, 883 insertions(+), 656 deletions(-)