public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Matt Mackall <mpm@selenic.com>
To: Adrian Bunk <bunk@stusta.de>
Cc: linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: Light-weight dynamically extended stacks
Date: Mon, 19 Dec 2005 18:27:59 -0600	[thread overview]
Message-ID: <20051220002759.GE3356@waste.org> (raw)
In-Reply-To: <20051219183604.GT23349@stusta.de>

On Mon, Dec 19, 2005 at 07:36:04PM +0100, Adrian Bunk wrote:
> On Sun, Dec 18, 2005 at 04:12:49PM -0800, Matt Mackall wrote:
> 
> > Perhaps the time for this has come and gone, but it occurred to me
> > that it should be relatively straightforward to make a form of
> > dynamically extended stacks that are appropriate to the kernel.
> > 
> > While we have a good handle on most of the worst stack offenders, we
> > can still run into trouble with pathological cases (say, symlink
> > recursion for XFS on a RAID built from loopback mounts over NFS
> > tunneled over IPSEC through GRE). So there's probably no
> > one-size-fits-all when it comes to stack size.
> 
> My count of bug reports for problems with in-kernel code with 4k stacks 
> after Neil's patch went into -mm is still at 0. That's amazing 
> considering how many people have claimed in this thread how unstable
> 4k stacks were...

I should have said up front that I don't know of any remaining
problems with 4k stacks and support switching to them. Remember, I
dusted off the 4k stack code, cleaned it up, and fixed up some of the
worst offenders in my -tiny tree well before Arjan pushed it to
mainline.

So why am I raising this idea now at all? Because I think Neil's patch
is too clever and too specific to block layer stacking and I'd rather
have a more general solution. Block is by no means the only part of
the system that allows nesting and pathological combinations surely
still exist. And will be introduced in the future.

Also note that my approach might make it reasonable to use one-page
stacks everywhere, not just on x86.

> If more than 3 kB of stack is used on i386 that's a bug.
> And we should fix bugs, not work around them.

One man's fix is another man's work-around.

-- 
Mathematics is the supreme nostalgia of our time.

  reply	other threads:[~2005-12-20  0:28 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-19  0:12 Light-weight dynamically extended stacks Matt Mackall
2005-12-19  8:45 ` Arjan van de Ven
2005-12-19 18:36 ` Adrian Bunk
2005-12-20  0:27   ` Matt Mackall [this message]
2005-12-20 16:43     ` Adrian Bunk
2005-12-20 18:30       ` Matt Mackall
2005-12-20 19:40         ` Patrick McLean
2005-12-21  5:57           ` Valdis.Kletnieks

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=20051220002759.GE3356@waste.org \
    --to=mpm@selenic.com \
    --cc=bunk@stusta.de \
    --cc=linux-kernel@vger.kernel.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