Re: [PATCH v4 00/16] perf, persistent: Add persistent events

Borislav Petkov
2014-05-06T18:50:57Z

On Tue, May 06, 2014 at 02:39:07PM +0200, Robert Richter wrote:
> Creation of events basically does (function arguments are a bit pseudo
> code):
>
> attr.type = pmu_type;
> attr.config = event_config;
> event = perf_event_open(attr, cpu, ...);
> id = ioctl(event, PERF_EVENT_IOC_UNCLAIM, 0);

Remind me again pls why we decided for the CLAIM/UNCLAIM part? It is
very unintuitive. What was against we detach from the event (we're
implicitly attached to it in the normal case, when it is not persistent)
and then reattach?

--
Regards/Gruss,
Boris.

Sent from a fat crate under my desk. Formatting is fine.
--