From: lauraa@codeaurora.org (Laura Abbott)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] ARM: unwind: enable dumping stacks for SMP && ARM_UNWIND
Date: Thu, 11 Oct 2012 17:52:26 -0700 [thread overview]
Message-ID: <507769CA.6090903@codeaurora.org> (raw)
In-Reply-To: <1346021216-21979-3-git-send-email-ccross@android.com>
Hi,
On 8/26/2012 3:46 PM, Colin Cross wrote:
> Unwinding with CONFIG_ARM_UNWIND is much more complicated than
> unwinding with CONFIG_FRAME_POINTER, but there are only a few points
> that require validation in order to avoid faults or infinite loops.
> Avoiding faults is easy by adding checks to verify that all accesses
> relative to the frame's stack pointer remain inside the stack.
>
> When CONFIG_FRAME_POINTER is not set it is possible for two frames to
> have the same SP, so there is no way to avoid repeated calls to
> unwind_frame continuing forever.
>
> Signed-off-by: Colin Cross <ccross@android.com>
> ---
This is a feature we've wanted for a long time. I ran some test with
repeated catting of stacks with processes dying and the results looked good.
You can add a Tested-by: Laura Abbott <lauraa@codeaurora.org>
Laura
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation
next prev parent reply other threads:[~2012-10-12 0:52 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-26 22:46 [PATCH 0/2] ARM: enable dumping stacks for CONFIG_SMP Colin Cross
2012-08-26 22:46 ` [PATCH 1/2] ARM: stacktrace: enable dumping stacks for SMP && FRAME_POINTER Colin Cross
2012-08-26 22:46 ` [PATCH 2/2] ARM: unwind: enable dumping stacks for SMP && ARM_UNWIND Colin Cross
2012-10-12 0:52 ` Laura Abbott [this message]
2012-10-12 9:08 ` Russell King - ARM Linux
2012-10-12 10:02 ` Dave Martin
2012-10-16 2:15 ` Colin Cross
2012-10-16 10:12 ` Dave Martin
2012-10-16 10:55 ` Russell King - ARM Linux
2012-10-16 12:26 ` Dave Martin
2012-10-16 21:53 ` Colin Cross
2012-10-16 21:30 ` Colin Cross
2012-10-18 6:43 ` Dave Martin
2012-09-23 2:52 ` [PATCH 0/2] ARM: enable dumping stacks for CONFIG_SMP Colin Cross
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=507769CA.6090903@codeaurora.org \
--to=lauraa@codeaurora.org \
--cc=linux-arm-kernel@lists.infradead.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.