Messages in this thread | | | Date | Mon, 3 Feb 2014 19:22:42 +0100 | Subject | Re: [PATCH v5 0/4] perf: AARCH64 arch support | From | Jean Pihet <> |
| |
Hi Will, Arnaldo,
On 24 January 2014 17:35, Will Deacon <will.deacon@arm.com> wrote: > Hi Jean, > > On Wed, Jan 22, 2014 at 02:11:15PM +0000, Jean Pihet wrote: >> Add AARCH64 specific support. This includes the following: >> - AARCH64 perf registers definition and hooks, >> - compat mode registers use, i.e. profiling a 32-bit binary on >> a 64-bit system, >> - unwinding using the dwarf information from the .debug_frame >> section of the ELF binary, >> - unwinding using the frame pointer information; in 64-bit and >> compat modes. >> >> Note: support for unwinding using the dwarf information in compat >> mode requires some changes to the libunwind code. Those changes >> have been submitted on the libunwind ML and are in discussion. >> >> Tested on ARMv7, ARMv8 and x86_64 platforms. The compat mode has been >> tested on ARMv8 using statically built 32-bit binaries. > > This is look alright to me now, but there are still two blockers: > > (1) Getting the libunwind code merged This one is independent of the kernel changes. The libunwind stuff is under disuccsion atm, cf. libunwind ML.
> (2) Splitting this into an arch/arm64 series and a tools/perf series, so > that acme can merge the latter independently (and avoid the mess we had > last time). Split into: - the tools/perf change: '[PATCH] perf: ARM64: wire up perf_regs and unwind support' and - the arch/arm64 changes: '[PATCH v6 0/3] perf: AARCH64 arch support'.
> > Anyway, for the series: > > Acked-by: Will Deacon <will.deacon@arm.com> Thanks! I have added your Acked-By in the series.
> > Will
Jean
| |