perf annotate: Fix handling of goto labels that are valid hex numbers
When parsing the objdump disassembly output we can have goto labels that are valid hex numbers and thus get confused with lines with machine code. Handle the common case of a label that has nothing after it and other cases where there is just source code by validating the resulting "ip". It is still possible that we find goto labels that are in the function address range, but only if they are located before the real address we should be OK. A change in the objdump output to have a clear marker separating addresses from the disassembly would come handy, but we would still have to deal with older versions. Reported-by: Gleb Natapov <gleb@redhat.com> Cc: Frederic Weisbecker <fweisbec@gmail.com> Cc: Gleb Natapov <gleb@redhat.com> Cc: Mike Galbraith <efault@gmx.de> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Stephane Eranian <eranian@google.com> LKML-Reference: <20100722170541.GF17631@ghostprotocols.net> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
This commit is contained in:
Родитель
07fca0e57f
Коммит
70a7cb3b39
|
@ -976,13 +976,17 @@ static int hist_entry__parse_objdump_line(struct hist_entry *self, FILE *file,
|
||||||
* Parse hexa addresses followed by ':'
|
* Parse hexa addresses followed by ':'
|
||||||
*/
|
*/
|
||||||
line_ip = strtoull(tmp, &tmp2, 16);
|
line_ip = strtoull(tmp, &tmp2, 16);
|
||||||
if (*tmp2 != ':' || tmp == tmp2)
|
if (*tmp2 != ':' || tmp == tmp2 || tmp2[1] == '\0')
|
||||||
line_ip = -1;
|
line_ip = -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (line_ip != -1) {
|
if (line_ip != -1) {
|
||||||
u64 start = map__rip_2objdump(self->ms.map, sym->start);
|
u64 start = map__rip_2objdump(self->ms.map, sym->start),
|
||||||
|
end = map__rip_2objdump(self->ms.map, sym->end);
|
||||||
|
|
||||||
offset = line_ip - start;
|
offset = line_ip - start;
|
||||||
|
if (offset < 0 || (u64)line_ip > end)
|
||||||
|
offset = -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
objdump_line = objdump_line__new(offset, line);
|
objdump_line = objdump_line__new(offset, line);
|
||||||
|
|
Загрузка…
Ссылка в новой задаче