Re: [RFC/PATCH 03/38] perf tools: Move auxtrace_mmap field to struct perf_evlist

From: Adrian Hunter
Date: Tue Oct 06 2015 - 05:29:29 EST


On 06/10/15 12:03, Namhyung Kim wrote:
> Hi Adrian,
>
> On Mon, Oct 5, 2015 at 8:29 PM, Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote:
>> On 02/10/15 21:45, Arnaldo Carvalho de Melo wrote:
>>> Em Fri, Oct 02, 2015 at 02:18:44PM +0900, Namhyung Kim escreveu:
>>>> Since it's gonna share struct mmap with dummy tracking evsel to track
>>>> meta events only, let's move auxtrace out of struct perf_mmap.
>>> Is this moving around _strictly_ needed?
>>
>> Also, what if you wanted to capture AUX data and tracking together.
>
> Hmm.. I don't know what's the problem. It should be orthogonal and
> support doing that together IMHO. Maybe I'm missing something about
> the aux data processing and Intel PT. I'll take a look at it..
>

It is only orthogonal if you assume we will never want to support parallel
processing with Intel PT.

The only change that needs to be made is not to assume there is only 1
tracking event.

IMHO there could be separate mmap_params also, which would allow for
different mmap sizes for the tracking and main mmaps.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/