Re: [PATCH 3/5] perf trace: Add mmap2 handler

From: Arnaldo Carvalho de Melo
Date: Tue Sep 24 2013 - 13:16:36 EST


Em Mon, Sep 23, 2013 at 06:22:37PM -0600, David Ahern escreveu:
> Arnaldo:
>
> I see this is you perf core tree; the problem exists in Linus' tree
> so it is needed in your urgent tree as well.

Sure, reproduced the segfault, cherry picked this patch from perf/core,
segfault fixes, thanks.

- Arnaldo

> David
>
> On 9/22/13 7:44 PM, David Ahern wrote:
> >5c5e854b changed perf_event__synthesize_mmap_events to generate MMAP2
> >events. Since perf-trace does not have a handler for it it dies with a
> >segfault when trying to process files:
> >
> >perf trace -i /tmp/perf.data
> >Segmentation fault
> >
> >Signed-off-by: David Ahern <dsahern@xxxxxxxxx>
> >---
> > tools/perf/builtin-trace.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> >diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
> >index 9c7f017..f61c968 100644
> >--- a/tools/perf/builtin-trace.c
> >+++ b/tools/perf/builtin-trace.c
> >@@ -1383,6 +1383,7 @@ static int trace__replay(struct trace *trace)
> >
> > trace->tool.sample = trace__process_sample;
> > trace->tool.mmap = perf_event__process_mmap;
> >+ trace->tool.mmap2 = perf_event__process_mmap2;
> > trace->tool.comm = perf_event__process_comm;
> > trace->tool.exit = perf_event__process_exit;
> > trace->tool.fork = perf_event__process_fork;
> >
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/