summaryrefslogtreecommitdiffstats
path: root/tools/perf/scripts
diff options
context:
space:
mode:
authorJiri Olsa2017-02-17 15:00:56 +0100
committerArnaldo Carvalho de Melo2017-02-17 21:28:22 +0100
commit99e7138eb7897aa0ccc6661173ae2d7e79721e05 (patch)
tree6d603e00226f5a25356c0315827701ea6bc96ff8 /tools/perf/scripts
parentperf tools: Move new_term arguments into struct parse_events_term template (diff)
downloadkernel-qcow2-linux-99e7138eb7897aa0ccc6661173ae2d7e79721e05.tar.gz
kernel-qcow2-linux-99e7138eb7897aa0ccc6661173ae2d7e79721e05.tar.xz
kernel-qcow2-linux-99e7138eb7897aa0ccc6661173ae2d7e79721e05.zip
perf tools: Fail on using multiple bits long terms without value
Currently we allow not to specify value for numeric terms and we set them to value 1. This was originaly meant just for single bit terms to allow user to type: $ perf record -e 'cpu/cpu-cycles,any' instead of: $ perf record -e 'cpu/cpu-cycles,any=1' However it works also for multi bits terms like: $ perf record -e 'cpu/event/' ls ... $ perf evlist -v ..., config: 0x1, ... After discussion with Peter we decided making such term usage to fail, like: $ perf record -e 'cpu/event/' ls event syntax error: 'cpu/event/' \___ no value assigned for term ... Signed-off-by: Jiri Olsa <jolsa@kernel.org> Cc: David Ahern <dsahern@gmail.com> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Link: http://lkml.kernel.org/r/1487340058-10496-4-git-send-email-jolsa@kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/scripts')
0 files changed, 0 insertions, 0 deletions