Re: [PATCH 01/21] perf c2c: Shared data analyser

[Don Zickus <dzickus@redhat.com>]

On Tue, Feb 11, 2014 at 08:31:27AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Mon, Feb 10, 2014 at 02:10:04PM -0800, Davidlohr Bueso escreveu:
> > On Mon, 2014-02-10 at 14:18 -0500, Don Zickus wrote:
> > > From: Arnaldo Carvalho de Melo <acme@redhat.com>
> > > 
> > > This is the start of a new perf tool that will collect information about
> > > memory accesses and analyse it to find things like hot cachelines, etc.
> > > 
> > > This is basically trying to get a prototype written by Richard Fowles
> > > written using the tools/perf coding style and libraries.
> > > 
> > > Start it from 'perf sched', this patch starts the process by adding the
> > > 'record' subcommand to collect the needed mem loads and stores samples.
> > > 
> > > It also have the basic 'report' skeleton, resolving the sample address
> > > and hooking the events found in a perf.data file with methods to handle
> > > them, right now just printing the resolved perf_sample data structure
> > > after each event name.
> > 
> > What tree/branch is this developed against? I'm getting the following
> > with Linus' latest and tip tree:
> 
> I'll try refreshing it on top of my perf/core branch today

Sorry everyone,  I managed to rebase it on top of Ingo's master branch
f58a0b1790e3973b23548e297db60c18b29b0818

Let me find your perf/core branch.

Cheers,
Don

>  
> > builtin-c2c.c: In function ‘perf_c2c__process_sample’:
> > builtin-c2c.c:68:20: error: request for member ‘func’ in something not a structure or union
> > builtin-c2c.c:69:36: error: request for member ‘func’ in something not a structure or union
> > builtin-c2c.c: In function ‘perf_c2c__read_events’:
> > builtin-c2c.c:81:2: error: passing argument 1 of ‘perf_session__new’ from incompatible pointer type [-Werror]
> > In file included from builtin-c2c.c:6:0:
> > util/session.h:52:22: note: expected ‘struct perf_data_file *’ but argument is of type ‘const char *’
> > builtin-c2c.c:81:2: error: too many arguments to function ‘perf_session__new’
> > In file included from builtin-c2c.c:6:0:
> > util/session.h:52:22: note: declared here
> >