All of lore.kernel.org
 help / color / mirror / Atom feed
From: tim.bird@am.sony.com (Tim Bird)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] ARM 4Kstacks: introduction
Date: Wed, 19 Oct 2011 16:43:21 -0700	[thread overview]
Message-ID: <4E9F6099.20904@am.sony.com> (raw)
In-Reply-To: <1318984301.7569.9.camel@Joe-Laptop>

On 10/18/2011 05:31 PM, Joe Perches wrote:
> On Tue, 2011-10-18 at 17:26 -0700, Tim Bird wrote:
>> On 10/18/2011 05:14 PM, Joe Perches wrote:
>>> On Tue, 2011-10-18 at 16:27 -0700, Tim Bird wrote:
>>>> I'm about to submit a set of patches (really pretty small)
>>>> to add 4K stack support to ARM (defaulted to 'N').
>>> When 4k stacks went away, I thought it safe enough
>>> to submit vsnprintf recursion using %pV.
>> Even inside Sony, usage of 4K stacks is limited
>> to some very special cases, where memory is exceedingly
>> tight (we have one system with 4M of RAM).  And we
>> don't mind lopping off features or coding around
>> problem areas to support our special case.
> 
> I believe CONFIG_PRINTK=n assures no %pV problem.

Indeed. Hehe. :-)  We might already be doing that in
the places we're using this.

Actually, this recursion is interesting.
My stack_size tool to detects stack usage for one instance
of a function.  But it would be easy to add a special
case to do a multiplier for functions known to potentially
recurse.  I'll have to look at that and maybe add something.

Relatedly, it might be interesting to augment the static
analysis with runtime information from the stack tracer.
 -- Tim

=============================
Tim Bird
Architecture Group Chair, CE Workgroup of the Linux Foundation
Senior Staff Engineer, Sony Network Entertainment
=============================

WARNING: multiple messages have this Message-ID (diff)
From: Tim Bird <tim.bird@am.sony.com>
To: Joe Perches <joe@perches.com>
Cc: Russell King <rmk@arm.linux.org.uk>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Arnd Bergmann <arnd@arndb.de>, Andi Kleen <andi@firstfloor.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	linux kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/3] ARM 4Kstacks: introduction
Date: Wed, 19 Oct 2011 16:43:21 -0700	[thread overview]
Message-ID: <4E9F6099.20904@am.sony.com> (raw)
In-Reply-To: <1318984301.7569.9.camel@Joe-Laptop>

On 10/18/2011 05:31 PM, Joe Perches wrote:
> On Tue, 2011-10-18 at 17:26 -0700, Tim Bird wrote:
>> On 10/18/2011 05:14 PM, Joe Perches wrote:
>>> On Tue, 2011-10-18 at 16:27 -0700, Tim Bird wrote:
>>>> I'm about to submit a set of patches (really pretty small)
>>>> to add 4K stack support to ARM (defaulted to 'N').
>>> When 4k stacks went away, I thought it safe enough
>>> to submit vsnprintf recursion using %pV.
>> Even inside Sony, usage of 4K stacks is limited
>> to some very special cases, where memory is exceedingly
>> tight (we have one system with 4M of RAM).  And we
>> don't mind lopping off features or coding around
>> problem areas to support our special case.
> 
> I believe CONFIG_PRINTK=n assures no %pV problem.

Indeed. Hehe. :-)  We might already be doing that in
the places we're using this.

Actually, this recursion is interesting.
My stack_size tool to detects stack usage for one instance
of a function.  But it would be easy to add a special
case to do a multiplier for functions known to potentially
recurse.  I'll have to look at that and maybe add something.

Relatedly, it might be interesting to augment the static
analysis with runtime information from the stack tracer.
 -- Tim

=============================
Tim Bird
Architecture Group Chair, CE Workgroup of the Linux Foundation
Senior Staff Engineer, Sony Network Entertainment
=============================


  reply	other threads:[~2011-10-19 23:43 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-18 23:27 [PATCH 0/3] ARM 4Kstacks: introduction Tim Bird
2011-10-18 23:27 ` Tim Bird
2011-10-18 23:29 ` [PATCH 1/3] ARM 4Kstacks: Add support for 4K kernel stacks to ARM Tim Bird
2011-10-18 23:29   ` Tim Bird
2011-10-18 23:31 ` [PATCH 2/3] ARM: Add static kernel function stack size analyzer, for ARM Tim Bird
2011-10-18 23:31   ` Tim Bird
2011-10-19 10:45   ` Arnd Bergmann
2011-10-19 10:45     ` Arnd Bergmann
2011-10-21 17:03     ` Andi Kleen
2011-10-21 17:03       ` Andi Kleen
2011-10-18 23:34 ` [PATCH 3/3] ARM 4Kstacks: Decrease poll and select stack usage, when using 4K stacks Tim Bird
2011-10-18 23:34   ` Tim Bird
2011-10-20 12:21   ` Arnd Bergmann
2011-10-20 12:21     ` Arnd Bergmann
2011-10-19  0:14 ` [PATCH 0/3] ARM 4Kstacks: introduction Joe Perches
2011-10-19  0:14   ` Joe Perches
2011-10-19  0:26   ` Tim Bird
2011-10-19  0:26     ` Tim Bird
2011-10-19  0:31     ` Joe Perches
2011-10-19  0:31       ` Joe Perches
2011-10-19 23:43       ` Tim Bird [this message]
2011-10-19 23:43         ` Tim Bird
2011-10-19  4:54     ` Dave Chinner
2011-10-19  4:54       ` Dave Chinner
2011-10-19  5:02       ` Andi Kleen
2011-10-19  5:02         ` Andi Kleen
2011-10-19 10:51     ` Arnd Bergmann
2011-10-19 10:51       ` Arnd Bergmann
2011-10-22  8:50       ` Ming Lei
2011-10-22  8:50         ` Ming Lei
2011-10-22 13:13         ` Måns Rullgård
2011-10-22 13:13           ` Måns Rullgård
2011-10-22 14:27           ` Andi Kleen
2011-10-22 14:27             ` Andi Kleen
2011-10-22 13:36         ` Russell King - ARM Linux
2011-10-22 13:36           ` Russell King - ARM Linux
2011-10-23 19:25           ` Tim Bird
2011-10-23 19:25             ` Tim Bird
2011-10-23 20:11             ` Russell King - ARM Linux
2011-10-23 20:11               ` Russell King - ARM Linux
2011-10-24 10:36           ` Ming Lei
2011-10-24 10:36             ` Ming Lei
2011-10-23 14:06       ` Bernd Petrovitsch
2011-10-23 14:06         ` Bernd Petrovitsch
2011-10-19  4:33 ` Dave Chinner
2011-10-19  4:33   ` Dave Chinner
2011-10-19  7:35 ` Russell King - ARM Linux
2011-10-19  7:35   ` Russell King - ARM Linux
2011-10-19 23:36   ` Tim Bird
2011-10-19 23:36     ` Tim Bird
2011-10-20  0:13     ` Måns Rullgård
2011-10-20  0:13       ` Måns Rullgård
2011-10-20  1:08       ` Tim Bird
2011-10-20  1:08         ` Tim Bird
2011-10-20  1:55         ` Måns Rullgård
2011-10-20  1:55           ` Måns Rullgård
2011-10-20 22:21     ` Dave Chinner
2011-10-20 22:21       ` Dave Chinner

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=4E9F6099.20904@am.sony.com \
    --to=tim.bird@am.sony.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 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.