[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CABPqkBQFpJjHLYuErcMdxTNKe4-JW_JZfYkEWrX5JHTZmWOqDg@mail.gmail.com>
Date: Tue, 30 Jul 2013 18:09:15 +0200
From: Stephane Eranian <eranian@...gle.com>
To: Peter Zijlstra <peterz@...radead.org>
Cc: Ingo Molnar <mingo@...nel.org>,
LKML <linux-kernel@...r.kernel.org>,
"mingo@...e.hu" <mingo@...e.hu>,
"ak@...ux.intel.com" <ak@...ux.intel.com>,
Arnaldo Carvalho de Melo <acme@...hat.com>,
Jiri Olsa <jolsa@...hat.com>,
Namhyung Kim <namhyung.kim@....com>
Subject: Re: [PATCH 0/8] perf: add ability to sample physical data addresses
On Tue, Jul 30, 2013 at 5:52 PM, Peter Zijlstra <peterz@...radead.org> wrote:
> On Tue, Jul 30, 2013 at 04:21:41PM +0200, Stephane Eranian wrote:
>> Peter,
>>
>> One thing that bothers me with the MMAP2 approach is that
>> it forces integration into perf.
>
> This is a good (TM) thing, yes? ;-)
>
Well, that's one way to look at it. But I'd like the API to
be able to support more than one tool easily. If the barrier
for any advanced analysis is too high from the raw kernel
API then we will only have one tool. I don't think that sane.
>> Now, you will need to analyze
>> the MMAP2 records. With my sample_type approach, you
>> simply needed a cmdline option on perf record, and then
>> you could dump the sample using perf report -D and feed
>> them into a post-processing script. But now, the analysis
>> needs to be integrated into perf or the tool needs to parse
>> the full perf.data file.
>
> So the disadvantage of the sample_type approach is that it generates
> more data and bloats the fast path.
>
> If its useful it shouldn't live in a script anyway ;-) Also if the
> script muck can't deal with the side-band information its a worse broken
> piece of crap than I thought it was.
I never use the scripting support. Don't know its current shape.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists