Re: [PATCH V6 00/26] Coresight integration with perf

Mathieu Poirier
2015-12-10T21:27:56Z

On 8 December 2015 at 13:34, Arnaldo Carvalho de Melo <acme@kernel.org> wrote:
> Em Tue, Dec 08, 2015 at 01:29:00PM -0700, Mathieu Poirier escreveu:
>> On 3 December 2015 at 14:04, Mathieu Poirier <mathieu.poirier@linaro.org> wrote:
>>
>> Arnaldo, Peter and Ingo,
>>
>> I would like to queue this work for 4.5 but need your help with
>> reviewing patches 20 [1], 23 [2], 24 [3], 25 [4] and 26 [5].
>>
>> A "reviewed-by" would be greatly appreciated.
>
> I looked briefly at it and saw perf_session, IIRc, being passed to more
> functions, which I think can be done otherwise, as, again IIRC, all you
> need is available via perf_evlist, will look again.

I just had a double take here and the only remnant of a "struct
perf_session" I can see is in 26 [1], but that is following the
upstream API [2]. I can send a fix for that but it would be outside
the scope of this patch set.

Thanks,
Mathieu

[1]. https://lkml.org/lkml/2015/12/3/684
[2]. http://lxr.free-electrons.com/source/tools/perf/util/auxtrace.h#L293
>
> - Arnaldo