From: Will Deacon <will.deacon@arm.com>
To: Jean Pihet <jean.pihet@linaro.org>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linaro-kernel@lists.linaro.org" <linaro-kernel@lists.linaro.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Arnaldo <acme@ghostprotocols.net>, Ingo Molnar <mingo@kernel.org>,
Jiri Olsa <jolsa@redhat.com>,
"patches@linaro.org" <patches@linaro.org>
Subject: Re: [PATCH v5 0/4] perf: AARCH64 arch support
Date: Fri, 24 Jan 2014 16:35:49 +0000 [thread overview]
Message-ID: <20140124163548.GH31040@mudshark.cambridge.arm.com> (raw)
In-Reply-To: <1390399879-5109-1-git-send-email-jean.pihet@linaro.org>
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
(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).
Anyway, for the series:
Acked-by: Will Deacon <will.deacon@arm.com>
Will
next prev parent reply other threads:[~2014-01-24 16:36 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-22 14:11 Jean Pihet
2014-01-22 14:11 ` [PATCH 1/4] ARM64: perf: add support for perf registers API Jean Pihet
2014-01-22 14:11 ` [PATCH 2/4] ARM64: perf: wire up perf_regs and unwind support Jean Pihet
2014-01-22 14:11 ` [PATCH 3/4] ARM64: perf: add support for frame pointer unwinding in compat mode Jean Pihet
2014-01-22 14:11 ` [PATCH 4/4] ARM64: perf: support dwarf " Jean Pihet
2014-01-24 16:35 ` Will Deacon [this message]
2014-02-03 18:22 ` [PATCH v5 0/4] perf: AARCH64 arch support Jean Pihet
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140124163548.GH31040@mudshark.cambridge.arm.com \
--to=will.deacon@arm.com \
--cc=acme@ghostprotocols.net \
--cc=jean.pihet@linaro.org \
--cc=jolsa@redhat.com \
--cc=linaro-kernel@lists.linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=patches@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®