linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: james.morse@arm.com (James Morse)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 2/2] arm64: Expand the stack trace feature to support IRQ stack
Date: Thu, 15 Oct 2015 17:01:14 +0100	[thread overview]
Message-ID: <561FCDCA.1080804@arm.com> (raw)
In-Reply-To: <179A25A2-C346-46C0-93D1-BABD4B6CB401@gmail.com>

On 15/10/15 15:24, Jungseok Lee wrote:
> On Oct 9, 2015, at 11:24 PM, James Morse wrote:
>> I think unwind_frame() needs to walk the irq stack too. [2] is an example
>> of perf tracing back to userspace, (and there are patches on the list to
>> do/fix this), so we need to walk back to the start of the first stack for
>> the perf accounting to be correct.
> 
> I plan to do re-spin this series without [PATCH 2/2] since 1) Akashi's
> approach looks better than mine and 2) you have the perf patches for [2].

They aren't my patches - the ones I saw on the list were for arm:
https://lkml.org/lkml/2015/10/1/769 - its evidently something perf
supports, so we shouldn't make it worse...


Thanks!

James

  reply	other threads:[~2015-10-15 16:01 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-07 15:28 [PATCH v4 0/2] arm64: Introduce IRQ stack Jungseok Lee
2015-10-07 15:28 ` [PATCH v4 1/2] " Jungseok Lee
2015-10-08 10:25   ` Pratyush Anand
2015-10-08 14:32     ` Jungseok Lee
2015-10-08 16:51       ` Pratyush Anand
2015-10-07 15:28 ` [PATCH v4 2/2] arm64: Expand the stack trace feature to support " Jungseok Lee
2015-10-09 14:24   ` James Morse
2015-10-12 14:53     ` Jungseok Lee
2015-10-12 16:34       ` James Morse
2015-10-12 22:13         ` Jungseok Lee
2015-10-13 11:00           ` James Morse
2015-10-13 15:00             ` Jungseok Lee
2015-10-14 12:12               ` Jungseok Lee
2015-10-15 15:59                 ` James Morse
2015-10-16 13:01                   ` Jungseok Lee
2015-10-16 16:06                     ` Catalin Marinas
2015-10-17 13:38                       ` Jungseok Lee
2015-10-19 16:18                         ` Catalin Marinas
2015-10-20 13:08                           ` Jungseok Lee
2015-10-21 15:14                             ` Jungseok Lee
2015-10-14  7:13     ` AKASHI Takahiro
2015-10-14 12:24       ` Jungseok Lee
2015-10-14 12:55         ` Jungseok Lee
2015-10-15  4:19           ` AKASHI Takahiro
2015-10-15 13:39             ` Jungseok Lee
2015-10-19  6:47               ` AKASHI Takahiro
2015-10-20 13:19                 ` Jungseok Lee
2015-10-15 14:24     ` Jungseok Lee
2015-10-15 16:01       ` James Morse [this message]
2015-10-16 13:02         ` Jungseok Lee

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=561FCDCA.1080804@arm.com \
    --to=james.morse@arm.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).