Re: [PATCH 1/5] perf tools: Fix assertion failure on dynamic entry

From: Arnaldo Carvalho de Melo
Date: Mon Feb 22 2016 - 10:09:12 EST


Em Sun, Feb 21, 2016 at 06:37:32PM +0100, Jiri Olsa escreveu:
> On Sun, Feb 21, 2016 at 11:22:34PM +0900, Namhyung Kim wrote:
> > The dynamic entry is created for each field in a tracepoint event.
> > Since they have no fixed hpp format index, it should skip when
> > perf_hpp__reset_width() is called.
> >
> > This caused following assertion failure..
> >
> > $ perf record -e sched:sched_switch -a sleep 1
> >
> > $ perf report -s comm,next_pid --stdio
> > perf: ui/hist.c:651: perf_hpp__reset_width:
> > Assertion `!(fmt->idx >= PERF_HPP__MAX_INDEX)' failed.
> >
> > Signed-off-by: Namhyung Kim <namhyung@xxxxxxxxxx>
>
> Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

Thanks, tested and applied the series,

- Arnaldo