Messages in this thread |  | | | Date | Mon, 14 May 2012 19:42:30 -0600 | | From | David Ahern <> | | Subject | Re: [PATCH] perf-record: Create events initially disabled -- again |
| |
On 5/14/12 7:07 PM, Namhyung Kim wrote: >> Something else is wrong then. I tested that command (saw your patch in >> the history) and it worked for me. Also, this code path does not >> affect perf-stat -- it touches perf-record and perf-test only. >> > > Ah, right. But still wouldn't it be better changing the conditional > rather than disabling it unconditionally? > > Thanks, > Namhyung
I think it would be best to disable all events initially and then enable them when ready. It works for perf-record and perf-test just fine and limits the samples to what you care about.
David
|  |