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

From: Borislav Petkov
Date: Tue May 06 2014 - 14:51:00 EST


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.
--
--
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/