On Tue, Dec 20, 2005 at 05:43:17PM +0100, Adrian Bunk wrote:
> On Mon, Dec 19, 2005 at 06:27:59PM -0600, Matt Mackall wrote:
> >...
> > 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.
> >...
>
> I'm really looking forward to seeing your patch.
I might get to it after the New Year.
> It will e.g. be interesting to measure whether there'll be any
> performance impact.
>
> And since after this patch driver authors might become more sloppy with
> stack usage since there's no longer a hard limit, it will be especially
> interesting to see how you'll implement ensuring that there are no
> additional stack usages > 1 kB between two invocations of you check
> function, because otherwise your patch won't work reliable.
I can give you the answer to that now: use the existing stack overflow
detection.
This is not intended to be an automatic scheme. To use it, you must
actually insert code into the troublesome codepaths, which will of
course serve as a red flag for code review.
It's entirely possible that this idea won't be needed on x86 because
we can manage to squeeze everything into 4k. But 32-bit x86's days as
the majority platform are numbered.
--
Mathematics is the supreme nostalgia of our time.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]