>-----Original Message----- >From: Jiri Olsa [mailto:jo...@redhat.com] >Sent: Saturday, November 4, 2017 6:26 AM >To: Megha Dey <megha....@linux.intel.com> >Cc: x...@kernel.org; linux-ker...@vger.kernel.org; linux- >d...@vger.kernel.org; t...@linutronix.de; mi...@redhat.com; >h...@zytor.com; andriy.shevche...@linux.intel.com; >kstew...@linuxfoundation.org; Yu, Yu-cheng <yu-cheng...@intel.com>; >Brown, Len <len.br...@intel.com>; gre...@linuxfoundation.org; >pet...@infradead.org; a...@kernel.org; >alexander.shish...@linux.intel.com; namhy...@kernel.org; >vikas.shiva...@linux.intel.com; pombreda...@nexb.com; >m...@kylehuey.com; b...@suse.de; Andrejczuk, Grzegorz ><grzegorz.andrejc...@intel.com>; Luck, Tony <tony.l...@intel.com>; >cor...@lwn.net; Shankar, Ravi V <ravi.v.shan...@intel.com>; Dey, Megha ><megha....@intel.com> >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 > >> +static unsigned int bm_threshold = BM_MAX_THRESHOLD; static >unsigned >> +int bm_mispred_evt_cnt; >> + >> +/* Branch monitoring counter owners */ static struct perf_event >> +*bm_counter_owner[2]; > >SNIP > >> + * Find a hardware counter for the target task >> + */ >> + for (i = 0; i < bm_num_counters; i++) { >> + if ((bm_counter_owner[i] == NULL) || >> + (bm_counter_owner[i]->state == >PERF_EVENT_STATE_DEAD)) { >> + counter_to_use = i; >> + bm_counter_owner[i] = event; >> + break; >> + } >> + } >> + >> + if (counter_to_use == -1) >> + return -EBUSY; > >not sure I understand, your docs says: "There are 2 8-bit counters that each.. >" > >so there are 2 counters per CPU? if that's corrent, isn't this check too strict >then? you could have more events configured running on other CPUs for >another tasks > >given that we do task only events here, should bm_counter_owner be part >of task, together with the limit..? I'm probably missing something..
Yes you are right. Initially, we had support for 2 events(from one or 2 tasks) to be monitored for the entire system. This indeed seems very limiting. In the next patchset, I will add support for 2 events per task (This is what the hardware can support). > >thanks, >jirka -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
