diff options
author | Oleg Nesterov <oleg@redhat.com> | 2013-03-28 18:58:11 +0100 |
---|---|---|
committer | Oleg Nesterov <oleg@redhat.com> | 2013-04-13 15:31:59 +0200 |
commit | 456fdbcb8648a20f56977efbc6f13e28936fcf0b (patch) | |
tree | e6e868c2830ae0212a1329edf7ecb0a901534052 /kernel | |
parent | 07720b63a964851928fa5d8b00ee5270d66b94f7 (diff) | |
download | op-kernel-dev-456fdbcb8648a20f56977efbc6f13e28936fcf0b.zip op-kernel-dev-456fdbcb8648a20f56977efbc6f13e28936fcf0b.tar.gz |
uprobes/tracing: Kill the pointless seq_print_ip_sym() call
seq_print_ip_sym(ip) in print_uprobe_event() is pointless,
kallsyms_lookup(ip) can not resolve a user-space address.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Acked-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Tested-by: Anton Arapov <anton@redhat.com>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/trace/trace_uprobe.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/kernel/trace/trace_uprobe.c b/kernel/trace/trace_uprobe.c index af5b773..8e00901 100644 --- a/kernel/trace/trace_uprobe.c +++ b/kernel/trace/trace_uprobe.c @@ -531,13 +531,7 @@ print_uprobe_event(struct trace_iterator *iter, int flags, struct trace_event *e field = (struct uprobe_trace_entry_head *)iter->ent; tu = container_of(event, struct trace_uprobe, call.event); - if (!trace_seq_printf(s, "%s: (", tu->call.name)) - goto partial; - - if (!seq_print_ip_sym(s, field->ip, flags | TRACE_ITER_SYM_OFFSET)) - goto partial; - - if (!trace_seq_puts(s, ")")) + if (!trace_seq_printf(s, "%s: (0x%lx)", tu->call.name, field->ip)) goto partial; data = (u8 *)&field[1]; |