Re: [PATCH] perf sched replay: fix event lookup

2012-06-27 Thread Arnaldo Carvalho de Melo
Em Wed, Jun 27, 2012 at 01:18:17PM +0400, Dmitry Antipov escreveu: > On 06/26/2012 03:20 AM, Arnaldo Carvalho de Melo wrote: > > > >>Fair enough. The method should be a simple wrapper to libtraceevent APIs > >>like this patch. > > > >The pevent thing is per

[GIT PULL 0/7] perf/core improvements and fixes

2012-06-27 Thread Arnaldo Carvalho de Melo
te symbols, from Pierre-Loup A. Griffais . Fix 'perf test' raw events entries, from Jiri Olsa . Use the events description in the perf.data file, not the sysfs ones. Signed-off-by: Arnaldo Carvalho de Melo -------- Arnaldo

[PATCH 3/7] perf tools: Stop using a global trace events description list

2012-06-27 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo The pevent thing is per perf.data file, so I made it stop being static and become a perf_session member, so tools processing perf.data files use perf_session and _there_ we read the trace events description into session->pevent and then change everywhere to s

Re: [PATCH] perf sched replay: fix event lookup

2012-06-26 Thread Arnaldo Carvalho de Melo
Em Tue, Jun 12, 2012 at 03:01:26PM +0900, Namhyung Kim escreveu: > On Mon, 11 Jun 2012 11:08:52 -0300, Arnaldo Carvalho de Melo wrote: > > So I think we should have a per perf.data (perf_session) method that > > knows that it shouldn't look _at all_ to /sys, but just at

Re: [PATCH] perf sched replay: fix event lookup

2012-06-25 Thread Arnaldo Carvalho de Melo
Em Sat, Jun 09, 2012 at 01:05:58PM +0400, Dmitry Antipov escreveu: > Use new function trace_find_event_by_name to lookup events before > looking through /sys files. This helps 'perf sched replay' to map > event names to IDs correctly when processing perf.data recorded > on another machine. > > Sig

Re: [PATCH] perf sched replay: fix event lookup

2012-06-12 Thread Arnaldo Carvalho de Melo
Em Tue, Jun 12, 2012 at 03:01:26PM +0900, Namhyung Kim escreveu: > Hi, > > On Mon, 11 Jun 2012 11:08:52 -0300, Arnaldo Carvalho de Melo wrote: > > Em Mon, Jun 11, 2012 at 02:46:02PM +0900, Namhyung Kim escreveu: > >> On Sat, 9 Jun 2012 13:05:58 +0400, Dmitry Antip

Re: [PATCH] perf report: fix event name reporting

2012-06-12 Thread Arnaldo Carvalho de Melo
Em Tue, Jun 12, 2012 at 03:34:22PM +0900, Namhyung Kim escreveu: > On Mon, 11 Jun 2012 11:14:16 -0300, Arnaldo Carvalho de Melo wrote: > > Em Fri, Jun 08, 2012 at 04:23:27PM +0400, Dmitry Antipov escreveu: > >> Use trace_find_event to find event name before looking through >

Re: [PATCH] perf report: fix event name reporting

2012-06-11 Thread Arnaldo Carvalho de Melo
Em Fri, Jun 08, 2012 at 04:23:27PM +0400, Dmitry Antipov escreveu: > Use trace_find_event to find event name before looking through > /sys files. This helps 'perf report' to show real event names > instead of 'unknown:unknown' when processing perf.data recorded > on another machine. We have to som

Re: [PATCH] perf sched replay: fix event lookup

2012-06-11 Thread Arnaldo Carvalho de Melo
Em Mon, Jun 11, 2012 at 02:46:02PM +0900, Namhyung Kim escreveu: > On Sat, 9 Jun 2012 13:05:58 +0400, Dmitry Antipov wrote: > > Use new function trace_find_event_by_name to lookup events before > > looking through /sys files. This helps 'perf sched replay' to map > > event names to IDs correctly w

Re: Perf record format portability

2012-05-29 Thread Arnaldo Carvalho de Melo
Em Fri, May 18, 2012 at 09:48:26AM +0400, Dmitry Antipov escreveu: > On 05/17/2012 03:48 PM, Steven Rostedt wrote: > > >trace-cmd copies the entire /sys/kernel/debug/tracing/events directory > >into the data file (well it copies only the events you specify). > >I thought perf did the same. It shou

Re: Perf record format portability

2012-05-17 Thread Arnaldo Carvalho de Melo
Em Wed, May 16, 2012 at 12:58:23PM -0400, Steven Rostedt escreveu: > On Wed, 2012-05-16 at 11:59 -0300, Arnaldo Carvalho de Melo wrote: > > Was the kernel trace events infrastructure designed with that in > > mind? I.e. cross analysis? I must be missing something here, stil

Re: Perf record format portability

2012-05-16 Thread Arnaldo Carvalho de Melo
Em Wed, May 16, 2012 at 05:16:55PM +0200, Jiri Olsa escreveu: > On Wed, May 16, 2012 at 11:59:27AM -0300, Arnaldo Carvalho de Melo wrote: > > Adding Jiri and Steven to the CC list. > > > > Em Wed, May 16, 2012 at 02:50:31PM +0400, Dmitry Antipov escreveu: > > > O

Re: Perf record format portability

2012-05-16 Thread Arnaldo Carvalho de Melo
Adding Jiri and Steven to the CC list. Em Wed, May 16, 2012 at 02:50:31PM +0400, Dmitry Antipov escreveu: > On 05/15/2012 07:51 PM, Arnaldo Carvalho de Melo wrote: > >Em Tue, May 15, 2012 at 07:27:39PM +0400, Dmitry Antipov escreveu: > >>are there any thoughts on how much o

Re: Perf record format portability

2012-05-15 Thread Arnaldo Carvalho de Melo
Em Tue, May 15, 2012 at 07:27:39PM +0400, Dmitry Antipov escreveu: > Hello, > > are there any thoughts on how much of the perf.data is portable and how much > it should be? > I'm interesting in recording scheduler activity on one machine and then > replaying on > another. As I can see, replaying

Re: [PATCH 0/2] perf: symbol offset breakage with separated debug

2010-08-03 Thread Arnaldo Carvalho de Melo
Em Tue, Aug 03, 2010 at 12:48:34PM +0100, Dave Martin escreveu: > Hi all, > > I've hit some problems in trying to get perf record/report to > work correctly on systems with separated debug images (Ubuntu, > in particular). I worked on some patches to ensure that > separate debug images do actuall

Re: [PATCH 1/2] perf events: Fix mmap offset determination

2010-08-03 Thread Arnaldo Carvalho de Melo
Em Tue, Aug 03, 2010 at 12:48:35PM +0100, Dave Martin escreveu: > Fix buggy-looking code which unnecessarily adjusts the file offset > fields read from /proc/*/maps. > > This may have gone unnoticed since the offset is usually 0 (and the > logic in util/symbol.c may work incorrectly for other offs