Re: [GIT PULL 0/4] perf/core cleanups and fixes

[Ingo Molnar <mingo@kernel.org>]

* Arnaldo Carvalho de Melo <acme@infradead.org> wrote:

> Hi Ingo,
> 
> 	Please consider pulling.
> 
> - Arnaldo
> 
> The following changes since commit 0d09eb7a9a2ca4d7ed32f7b440bea78c0622814a:
> 
>   Merge branch 'perf/urgent' into perf/core (2012-03-22 15:09:08 -0300)
> 
> are available in the git repository at:
> 
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux tags/perf-core-for-mingo
> 
> for you to fetch changes up to 4bf9ce1b5ecffffeb8b9d7e925bac3e6b10109aa:
> 
>   perf diff: Fix to work with new hists design (2012-03-22 15:12:09 -0300)
> 
> ----------------------------------------------------------------
> Cleanups and fixes for perf/core:
> 
> . Short term fix for 'diff' tool breakage related to perf.data files
>   with multiple events. From Jiri Olsa
> 
> . Cleanup for event id tracepoint reading routine, from Borislav Petkov
> 
> . 32-bit compilation fixes from Jiri Olsa
> 
> . Event parsing modifier assignment fixes from Jiri Olsa
> 
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> ----------------------------------------------------------------
> Borislav Petkov (1):
>       perf tools: Simplify event_read_id exit path
> 
> Jiri Olsa (3):
>       perf tools: Fix various casting issues for 32 bits
>       perf tools: Fix modifier to be applied on correct events
>       perf diff: Fix to work with new hists design
> 
>  tools/perf/builtin-diff.c            |   60 +--
>  tools/perf/builtin-test.c            |  116 +++++-
>  tools/perf/util/evsel.c              |    2 +-
>  tools/perf/util/evsel.h              |    2 +
>  tools/perf/util/header.c             |    2 +-
>  tools/perf/util/hist.c               |    8 +-
>  tools/perf/util/parse-events-bison.c |  675 +++++++++++++++++++---------------
>  tools/perf/util/parse-events-bison.h |   15 +-
>  tools/perf/util/parse-events.c       |   29 +-
>  tools/perf/util/parse-events.h       |    7 +-
>  tools/perf/util/parse-events.y       |   40 +-
>  tools/perf/util/session.c            |    1 +
>  tools/perf/util/trace-event-parse.c  |   10 +-
>  13 files changed, 592 insertions(+), 375 deletions(-)

Pulled, thanks Arnaldo!

Btw., the SOB chain in commit 641cc93 is broken:

Author:     Jiri Olsa <jolsa@redhat.com>
Commit:     Arnaldo Carvalho de Melo <acme@redhat.com>
    Acked-by: Peter Zijlstra <peterz@infradead.org>
    Signed-off-by: Peter Zijlstra <peterz@infradead.org>
    Signed-off-by: Jiri Olsa <jolsa@redhat.com>
    Link: http://lkml.kernel.org/n/tip-vhdk5y2hyype9j63prymty36@git.kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

The first SOB by PeterZ is probably bogus.

Please be more careful about this in the future.

Thanks,

	Ingo