Subject: Re: [PATCH v13 12/14] perf, tools: Add support for event list topics

From: Jiri Olsa <[email-masked]>
Date: Wed, 3 Jun 2015 12:32:30 +0200

On Tue, Jun 02, 2015 at 12:16:41PM -0700, Sukadev Bhattiprolu wrote:

SNIP

>        [Speculative and retired macro-conditional branches]
>   br_inst_exec.all_direct_jmp
>        [Speculative and retired macro-unconditional branches excluding calls and indirects]
>   br_inst_exec.all_direct_near_call
>        [Speculative and retired direct near calls]
>   br_inst_exec.all_indirect_jump_non_call_ret
> 
> Signed-off-by: Andi Kleen <[email-masked]>
> Signed-off-by: Sukadev Bhattiprolu <[email-masked]>
> 
> Changelog[v2]
> 	Dropped an unnecessary patch before this and fixed resulting
> 	conflicts in tools/perf/util/pmu.c
> ---
>  tools/perf/pmu-events/jevents.c    | 16 +++++++++++-----
>  tools/perf/pmu-events/jevents.h    |  3 ++-
>  tools/perf/pmu-events/pmu-events.h |  1 +
>  tools/perf/util/pmu.c              | 34 ++++++++++++++++++++++++----------
>  tools/perf/util/pmu.h              |  1 +
>  5 files changed, 39 insertions(+), 16 deletions(-)

please split at least the jevents Topic parsing from the rest
idelay also the alias update and the display change

jirka