Re: [PATCH V2 17/30] perf: changing pmu::setup_aux() parameter to include event

From: Alexander Shishkin
Date: Mon Oct 19 2015 - 09:34:24 EST


Mathieu Poirier <mathieu.poirier@xxxxxxxxxx> writes:

> For some tracers the event carries information to be embedded
> in the private structure returned by setup_aux().

You need to mention here what these tracers are and which bits of
event's information they need in their setup_aux(). Right now I can look
it up in this patchset, but when this code gets merged it will make it
easier to understand why this change was made.

Now, I understand that you're interested in event::attr in your
setup_aux(), I have more comments on that in that other patch. :)

Cheers,
--
Alex
--
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/