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: Tue, 18 Oct 2011 17:26:44 -0700	[thread overview]
Message-ID: <4E9E1944.80601@am.sony.com> (raw)
In-Reply-To: <1318983248.7569.5.camel@Joe-Laptop>

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.
> 
> I believe the current vsnprintf max recursion is 3.
> Each recursion uses at least 300 bytes of stack.
> 
> That might be some additional issue for 4k stacks.

Interesting.  Thanks very much for the heads up.
I don't know how prevalent %pV is, but I'll take a
look tomorrow and see if I think this would be an
issue.  One option would be to turn off recursion
in the 4K stacks case (but I don't know how badly
this would mangle the code).

I'm not sure I'm willing to advocate eliminating
all occurrences of high stack usage in the kernel.
It seems like this would generate a lot of friction
for getting this mainlined.  As long as I can document
the trouble that people might get in when they
turn this on, I think that would be very good.

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.

Thanks again for the notice.
 -- 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: Tue, 18 Oct 2011 17:26:44 -0700	[thread overview]
Message-ID: <4E9E1944.80601@am.sony.com> (raw)
In-Reply-To: <1318983248.7569.5.camel@Joe-Laptop>

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.
> 
> I believe the current vsnprintf max recursion is 3.
> Each recursion uses at least 300 bytes of stack.
> 
> That might be some additional issue for 4k stacks.

Interesting.  Thanks very much for the heads up.
I don't know how prevalent %pV is, but I'll take a
look tomorrow and see if I think this would be an
issue.  One option would be to turn off recursion
in the 4K stacks case (but I don't know how badly
this would mangle the code).

I'm not sure I'm willing to advocate eliminating
all occurrences of high stack usage in the kernel.
It seems like this would generate a lot of friction
for getting this mainlined.  As long as I can document
the trouble that people might get in when they
turn this on, I think that would be very good.

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.

Thanks again for the notice.
 -- Tim


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


  reply	other threads:[~2011-10-19  0:26 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 [this message]
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
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=4E9E1944.80601@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.