lkml.org 
[lkml]   [2007]   [Jul]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH][RFC] 4K stacks default, not a debug thing any more...?
Date
Hi,

I'm wondering if it's time to make 4K stacks the default and to start
considering removing the 8K stack option alltogether soon?

One of the big problem spots was XFS, but that got some stack usage
fixes recently, and the 4K stack option has been around for quite a
while now, so people really should have gotten around to fixing any
code that can't handle it. Are there still any big problem areas
remaining?

As a starting point, something like the patch below that makes 4K
stacks default and make them no longer depend on DEBUG_KERNEL
could be the way to go. Then after a while of this being in effect we
could remove the option of a 8K stack alltogether.

What say you?


Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
---
arch/i386/Kconfig.debug | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/i386/Kconfig.debug b/arch/i386/Kconfig.debug
index b31c080..9b23bff 100644
--- a/arch/i386/Kconfig.debug
+++ b/arch/i386/Kconfig.debug
@@ -58,7 +58,7 @@ config DEBUG_RODATA

config 4KSTACKS
bool "Use 4Kb for kernel stacks instead of 8Kb"
- depends on DEBUG_KERNEL
+ default y
help
If you say Y here the kernel will use a 4Kb stacksize for the
kernel stack attached to each process/thread. This facilitates



--
Jesper Juhl <jesper.juhl@gmail.com>
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please http://www.expita.com/nomime.html


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-07-11 19:21    [W:0.122 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site