On Tue, Jul 15, 2014 at 04:58:53PM +0800, Yan, Zheng wrote: > +void perf_sched_cb_disable(struct pmu *pmu) > +{ > + this_cpu_dec(perf_sched_cb_usages); > +} > + > +void perf_sched_cb_enable(struct pmu *pmu) > +{ > + this_cpu_inc(perf_sched_cb_usages); > +} Still wrong names, they do not enable/disable. Enable/disable is a strictly boolean thing, this is counting.