Re: [PATCH 0/8] perf: add ability to sample physical data addresses

Peter Zijlstra
2013-07-30T16:16:44Z

On Tue, Jul 30, 2013 at 06:09:15PM +0200, Stephane Eranian wrote:
> On Tue, Jul 30, 2013 at 5:52 PM, Peter Zijlstra <peterz@infradead.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.

Sure, but any half-way usable profiling will need the existing sideband
data. So I don't think that relying on it for this particular feature is
too much.