Re: [PATCH 1/2] perf: Add munmap callback

From: Andi Kleen
Date: Wed Oct 24 2018 - 14:28:57 EST


> > void perf_event_mmap(struct vm_area_struct *vma)
> > {
> > struct perf_mmap_event mmap_event;
> >
> > if (!atomic_read(&nr_mmap_events))
> > return;
> > <SNIP>
> > }
> >
>
> Thanks. I'll add the nr_mmap_events check in V2.

No, that's the wrong check here. The PEBS flush is independent of mmap
events being requested.

If anything would need to check for any PEBS events active, which
would need a new counter. I think the easiest is to just check if
this_cpu_ptr(&sched_cb_list)
is empty, which should be good enough.

-Andi