diff options
author | Oleg Nesterov <oleg@redhat.com> | 2013-03-28 13:58:11 -0400 |
---|---|---|
committer | Oleg Nesterov <oleg@redhat.com> | 2013-04-13 09:31:59 -0400 |
commit | 456fdbcb8648a20f56977efbc6f13e28936fcf0b (patch) | |
tree | e6e868c2830ae0212a1329edf7ecb0a901534052 /kernel/trace | |
parent | 07720b63a964851928fa5d8b00ee5270d66b94f7 (diff) |
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/trace')
-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 af5b7735cebf..8e009016e4ba 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 | |||
531 | field = (struct uprobe_trace_entry_head *)iter->ent; | 531 | field = (struct uprobe_trace_entry_head *)iter->ent; |
532 | tu = container_of(event, struct trace_uprobe, call.event); | 532 | tu = container_of(event, struct trace_uprobe, call.event); |
533 | 533 | ||
534 | if (!trace_seq_printf(s, "%s: (", tu->call.name)) | 534 | if (!trace_seq_printf(s, "%s: (0x%lx)", tu->call.name, field->ip)) |
535 | goto partial; | ||
536 | |||
537 | if (!seq_print_ip_sym(s, field->ip, flags | TRACE_ITER_SYM_OFFSET)) | ||
538 | goto partial; | ||
539 | |||
540 | if (!trace_seq_puts(s, ")")) | ||
541 | goto partial; | 535 | goto partial; |
542 | 536 | ||
543 | data = (u8 *)&field[1]; | 537 | data = (u8 *)&field[1]; |