>-----Original Message----- >From: Jiri Olsa [mailto:[email protected]] >Sent: Saturday, November 4, 2017 6:25 AM >To: Megha Dey <[email protected]> >Cc: [email protected]; [email protected]; linux- >[email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; >[email protected]; Yu, Yu-cheng <[email protected]>; >Brown, Len <[email protected]>; [email protected]; >[email protected]; [email protected]; >[email protected]; [email protected]; >[email protected]; [email protected]; >[email protected]; [email protected]; Andrejczuk, Grzegorz ><[email protected]>; Luck, Tony <[email protected]>; >[email protected]; Shankar, Ravi V <[email protected]>; Dey, Megha ><[email protected]> >Subject: Re: [PATCH V0 2/3] perf/x86/intel/bm.c: Add Intel Branch >Monitoring support > >On Fri, Nov 03, 2017 at 11:00:05AM -0700, Megha Dey wrote: > >SNIP > >> + event->event_caps |= PERF_EV_CAP_BM; >> + /* >> + * cfg contains one of the 6 possible Branch Monitoring events >> + */ >> + cfg = event->attr.config; >> + if (cfg < 0 || cfg > (BM_MAX_EVENTS - 1)) >> + return -EINVAL; >> + >> + if (event->attr.sample_period) /* no sampling */ >> + return -EINVAL; > >you can use the 'is_sampling_event' function
Will make the change. > >jirka
