diff options
author | Oleg Nesterov <[email protected]> | 2014-06-27 19:01:36 +0200 |
---|---|---|
committer | Steven Rostedt <[email protected]> | 2014-06-30 13:21:58 -0400 |
commit | 48212542067a7ff6cbe829dbae279c2ff7557b44 (patch) | |
tree | cddf954fff810ca3dd1e03b817e263357d2ee686 /net/unix/af_unix.c | |
parent | 4d4c9cc839a308be3289a361ccba4447ee140552 (diff) |
tracing/uprobes: Revert "Support mix of ftrace and perf"
This reverts commit 43fe98913c9f67e3b523615ee3316f9520a623e0.
This patch is very wrong. Firstly, this change leads to unbalanced
uprobe_unregister(). Just for example,
# perf probe -x /lib/libc.so.6 syscall
# echo 1 >> /sys/kernel/debug/tracing/events/probe_libc/enable
# perf record -e probe_libc:syscall whatever
after that uprobe is dead (unregistered) but the user of ftrace/perf
can't know this, and it looks as if nobody hits this probe.
This would be easy to fix, but there are other reasons why it is not
simple to mix ftrace and perf. If nothing else, they can't share the
same ->consumer.filter. This is fixable too, but probably we need to
fix the poorly designed uprobe_register() interface first. At least
"register" and "apply" should be clearly separated.
Link: http://lkml.kernel.org/p/[email protected]
Cc: Tom Zanussi <[email protected]>
Cc: "zhangwei(Jovi)" <[email protected]>
Cc: [email protected] # v3.14
Acked-by: Namhyung Kim <[email protected]>
Acked-by: Srikar Dronamraju <[email protected]>
Reviewed-by: Masami Hiramatsu <[email protected]>
Signed-off-by: Oleg Nesterov <[email protected]>
Signed-off-by: Steven Rostedt <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions