perf report: Fix sort__sym_cmp to also compare end of symbol
authorYannick Brosseau <scientist@fb.com>
Wed, 17 Jun 2015 23:41:10 +0000 (16:41 -0700)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 19 Jun 2015 21:14:05 +0000 (18:14 -0300)
commitc05676c06232e6459a6106ddf0d4e154ce6cd859
tree999a0847ccfcced28533205a0b8ed7d9dbc760bb
parent3e323dc0a80c3921f30d03ca300426f70dc5a327
perf report: Fix sort__sym_cmp to also compare end of symbol

When using a map file from a JIT, due to memory reuse, we can obtain
multiple symbols with the same start address but a different length.

The symbols__find does check for the end so not doing it in
sort__sym_cmp was causing the hist_entry in the annotate part of a
report to match to the wrong entry, causing a fatal error.

Signed-off-by: Yannick Brosseau <scientist@fb.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: kernel-team@fb.com
Link: http://lkml.kernel.org/r/1434584470-17771-1-git-send-email-scientist@fb.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/sort.c