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

[Arnaldo Carvalho de Melo <acme@ghostprotocols.net>]

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@gmail.com>
> >---
> >  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;
> >