Re: [PATCH 02/17] perf tools: Add evlist channel helpers

From: Wangnan (F)
Date: Tue May 17 2016 - 23:31:19 EST




On 2016/5/13 21:05, Arnaldo Carvalho de Melo wrote:
Em Fri, May 13, 2016 at 07:55:59AM +0000, Wang Nan escreveu:
In this commit sereval helpers are introduced to support the principle
several

of channel. Channels hold different groups of evsels which configured
differently. It will be used for overwritable evsels, which allows perf
why not use multiple evlists? An "evlist" is a "list of evsels", why do
we need yet another way of grouping evlists?

- Arnaldo


There's an assumption all over perf that there's only one evlist: in 'struct record'
there's an 'evlist' pointer, in 'struct session' there's also an 'evlist' pointer.
Trying to change them to an array results in 181 errors, so I think fundamentally
moving to multiple evlists is nearly impossible.

Now I'm thinking introducing auxiliary evlists to perf record. We still obey one
evlist assumption, only creates separated evlists for mmap.

Thank you.