lkml.org 
[lkml]   [2016]   [May]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[PATCH v3 0/7] Add support for remote unwind
Date
v2 url:
http://thread.gmane.org/gmane.linux.kernel/2218373

Currently, perf script uses host unwind methods(local unwind) to parse
perf.data callchain info regardless of the target architecture. So we
get wrong result and no promotion when do remote unwind on other
platforms/machines.

This patchset adds build tests for the supported platforms for remote
unwinding, and checks whether a dso is 32-bit or 64-bit according to
elf class info for each thread to let perf use the correct remote
unwind methods instead.

Only x86 and aarch64 is added in this patchset to show the work flow,
other platforms can be added easily.

We can see the right result for unwind info on different machines, for
example: perf.data recorded on i686 qemu with '-g' option and parsed
on x86_64 machine.

before this patchset:

hello 1219 [001] 72190.667975: probe:sys_close: (c1169d60)
c1169d61 sys_close ([kernel.kallsyms])
c189c0d7 sysenter_past_esp ([kernel.kallsyms])
b777aba9 [unknown] ([vdso32])

after:
(Add vdso into buildid-cache first by 'perf buildid-cache -a' and
libraries are provided in symfs dir)

hello 1219 [001] 72190.667975: probe:sys_close: (c1169d60)
c1169d61 sys_close ([kernel.kallsyms])
c189c0d7 sysenter_past_esp ([kernel.kallsyms])
b777aba9 __kernel_vsyscall ([vdso32])
b76971cc close (/lib/libc-2.22.so)
804842e fib (/tmp/hello)
804849d main (/tmp/hello)
b75d746e __libc_start_main (/lib/libc-2.22.so)
8048341 _start (/tmp/hello)

v3:

- Remove --vdso option, store vdso buildid in perf.data and let perf
fetch it automatically.
- Use existing dso__type() function to test if dso is 32-bit or
64-bit.

v2:

- Explain the reason why we can omit dwarf judgement when recording
in commit message.
- Elaborate on why we need to add a custom vdso path option, and
change the type name to DSO_BINARY_TYPE__VDSO.
- Hide the build tests status for cross platform unwind.
- Keep generic version of libunwind-debug-frame test.
- Put 32/64-bit test functions into separate patch.
- Extract unwind related functions to unwind-libunwind.c and add new
file for common parts used by both local and remote unwind.
- Eliminate most of the ifdefs in .c file.

Thanks.

He Kuang (7):
perf tools: Set vdso name to vdso[64,32] depending on platform
perf tools: Store vdso buildid unconditionally
perf tools: Remove the logical that skip buildid cache if symfs is
given
perf tools: Promote proper messages for cross-platform unwind
perf callchain: Add support for cross-platform unwind
perf callchain: Support x86 target platform
perf callchain: Support aarch64 cross-platform

.../arch/arm64/include/libunwind/libunwind-arch.h | 18 ++++
tools/perf/arch/arm64/util/unwind-libunwind.c | 5 +-
.../arch/x86/include/libunwind/libunwind-arch.h | 18 ++++
tools/perf/arch/x86/util/unwind-libunwind.c | 42 ++++++++
tools/perf/config/Makefile | 35 ++++++-
tools/perf/util/Build | 13 ++-
tools/perf/util/build-id.c | 2 +-
tools/perf/util/dso.c | 6 +-
tools/perf/util/thread.c | 7 +-
tools/perf/util/thread.h | 14 ++-
tools/perf/util/unwind-libunwind.c | 49 +++++++--
tools/perf/util/unwind-libunwind_common.c | 114 +++++++++++++++++++++
tools/perf/util/unwind.h | 48 +++++++--
tools/perf/util/vdso.h | 7 +-
14 files changed, 348 insertions(+), 30 deletions(-)
create mode 100644 tools/perf/arch/arm64/include/libunwind/libunwind-arch.h
create mode 100644 tools/perf/arch/x86/include/libunwind/libunwind-arch.h
create mode 100644 tools/perf/util/unwind-libunwind_common.c

--
1.8.5.2

\
 
 \ /
  Last update: 2016-05-12 11:01    [W:0.169 / U:3.632 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site