On Wed, Aug 05, 2015 at 03:53:17PM +0200, Peter Zijlstra wrote: > +/* > + * NMI-safe method to read a local event, that is an event that > + * is: > + * - either for the current task, or for this CPU > + * - does not have inherit set, for inherited task events > + * will not be local and we cannot read them atomically > + * - must not have a pmu::count method > + */ > +u64 perf_event_read_local(struct perf_event *event) > +{ > + unsigned long flags; > + u64 val; > + > + /* > + * Disabling interrupts avoids all counter scheduling (context > + * switches, timer based rotation and IPIs). > + */ > + local_irq_safe(flags);
Hmm, I think local_irq_save(flags) will compile much better. Obviously this patch hasn't seen a compiler up close. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html