Messages in this thread | | | Date | Tue, 21 Jul 2015 16:56:15 +0200 | From | Jiri Olsa <> | Subject | Re: [PATCH RFC V5 2/4] perf,tool: per-event time support |
| |
On Tue, Jul 21, 2015 at 11:45:01PM +0900, Namhyung Kim wrote:
SNIP
> > > > > > [jolsa@krava perf]$ ./perf record -e 'cpu/instructions,period=20000/' -c > > > 1000 sleep 1 [ perf record: Woken up 1 times to write data ] /proc/kcore > > > requires CAP_SYS_RAWIO capability to access. > > > [ perf record: Captured and wrote 0.015 MB perf.data (35 samples) ] > > > [jolsa@krava perf]$ ./perf evlist -vv > > > cpu/instructions,period=20000/: type: 4, size: 112, config: 0xc0, > > > { sample_period, sample_freq }: 20000 > > > > > > > So you are going to change current behavior? > > The current behavior is "global opts setting" > "per_event settring" > default > > You are going to change it to "per_event settring" > "global opts setting" > > > Default. Right? > > Hmm.. I agree that changing current behavior is not good. But I think > it makes more sense to prefer per-event settings over global settings > in general.
hum right.. but it's just period ATM that will be affected.. I'd risk it, since I dont think maybe people actualy used it ;-)
together with time term, which wasn't possible to change before anyway, so there should be no problem
jirka
| |