]> git.baikalelectronics.ru Git - kernel.git/commit
perf probe: Fix to return error if no probe is added
authorMasami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Tue, 16 Jun 2015 11:50:55 +0000 (20:50 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 16 Jun 2015 14:39:51 +0000 (11:39 -0300)
commit58e085c547f2acd5cbc50a667748f9d6c019dafd
tree4c9e463755ff9705981f4adb72ff2017795aede9
parent0fa3ccc42daaecaa91ab240bbc371fa8d5fb6dde
perf probe: Fix to return error if no probe is added

Fix perf probe to return an error if no probe is added due to the given
probe point being on the blacklist.

To fix this problem, this moves the blacklist checking to right after
finding symbols/probe-points and marks them as skipped.

If all the symbols are skipped, "perf probe"  returns an error as it
fails to find the corresponding probe address.

E.g. currently if a blacklisted probe is given:

  # perf probe do_trap && echo 'succeed'
  Added new event:
  Warning: Skipped probing on blacklisted function: sync_regs
  succeed

No! It must fail! With this patch, it correctly fails:

  # perf probe do_trap && echo 'succeed'
  do_trap is blacklisted function, skip it.
  Probe point 'do_trap' not found.
    Error: Failed to add events.

Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Naohiro Aota <naota@elisp.net>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/20150616115055.19906.31359.stgit@localhost.localdomain
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/probe-event.c