| To: | Hemant Kumar <hemant@xxxxxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH v2] perfevent_pmda: Add extension for perf derived events |
| From: | Nathan Scott <nathans@xxxxxxxxxx> |
| Date: | Mon, 8 Feb 2016 21:35:35 -0500 (EST) |
| Cc: | pcp@xxxxxxxxxxx, jpwhite4@xxxxxxxxxxx |
| Delivered-to: | pcp@xxxxxxxxxxx |
| In-reply-to: | <1454385589-19166-1-git-send-email-hemant@xxxxxxxxxxxxxxxxxx> |
| References: | <1454385589-19166-1-git-send-email-hemant@xxxxxxxxxxxxxxxxxx> |
| Reply-to: | Nathan Scott <nathans@xxxxxxxxxx> |
| Thread-index: | oLAuFKVbzRzdjJIDzOUGiVzHh8epqg== |
| Thread-topic: | perfevent_pmda: Add extension for perf derived events |
Hi Hemant, ----- Original Message ----- > This patch gives an ability to perfevent agent to monitor derived > events. Derived events are based on the hardware/perf counters. > [...] > Looking forward for your suggestion/comments. > Looks pretty good to me. There are a handful of small things... 1. Needs a man page update to man/man5/perfevent.conf.5 2. Needs updates to qa/perfevent/ - currently this fails to build, then also needs some unit test cases for this new code (see the qa/756 script for how to invoke this) 3. The one-line ./Install script change ... seems unnecessary? ( > +pmns_dupok=true ) If you could send through a v3 patch with those things, I think it's ready to be merged. cheers. -- Nathan |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | pcp updates: qa, dbpmda, odds and sods, Ken McDonell |
|---|---|
| Next by Date: | Derived metric issues, Marko Myllynen |
| Previous by Thread: | [PATCH v2] perfevent_pmda: Add extension for perf derived events, Hemant Kumar |
| Next by Thread: | Code Quality Improvements for parfait, george . kankava |
| Indexes: | [Date] [Thread] [Top] [All Lists] |