On Thu, Nov 30, 2017 at 03:50:20PM -0800, Song Liu wrote: > +static struct pmu perf_kprobe = {
> +}; > +static inline bool perf_event_is_tracing(struct perf_event *event) > +{ > + return event->attr.type == PERF_TYPE_TRACEPOINT || > + strncmp(event->pmu->name, "kprobe", 6) == 0; Why a strcmp here? Wouldn't something like: event->pmu == &perf_kprobe work much simpler? > +}