Messages in this thread | | | From | Victor Kamensky <> | Subject | [PATCH V3 0/2] perf symbols: debuglink and arm mapping symbols changes | Date | Mon, 26 Jan 2015 22:34:00 -0800 |
| |
Hi Folks,
Here is third version of mini-series that addresses couple issues in perf symbols handling.
Changes since V2:
o Since V2 caused some controversy wrt aarch64/arm fixes [3], replace 'perf symbols: Ignore mapping symbols on aarch64' and 'perf symbols: improve abi compliance in arm mapping symbols handling with another version of almost equivalent single patch that was discussed on [1]. In new patch superset of aarch64 and arm mapping symbols handled by single code snippet, that is more aligned with other places where aarch64 and arm mapping symbols are handled.
Changes since V1:
o 'perf symbols: Ignore mapping symbols on aarch64': based on feedback from Will Deacon and Russell King [1] added code that handles ARM/Aarch64 mapping symbols like "$d.<any>". Code uses similar snippet to handle ARM mapping symbols as in another places in kernel, but it has its own version of the code. Added proper Acked-by.
o 'perf symbols: debuglink should take symfs option into account': based on feedback from David Ahern [2] picked more simple version that does not make malloced copy of 'filename', that makes patch simpler. Added proper 'Acked-and-tested-by'.
[1] https://lkml.org/lkml/2015/1/13/507
[2] https://lkml.org/lkml/2015/1/13/508
[3] https://lkml.org/lkml/2015/1/22/382
Victor Kamensky (2): perf symbols: Ignore mapping symbols on aarch64 perf symbols: debuglink should take symfs option into account
tools/perf/util/dso.c | 6 +++--- tools/perf/util/symbol-elf.c | 7 +++---- 2 files changed, 6 insertions(+), 7 deletions(-)
-- 1.9.3
| |