On Mon, 30 Apr 2007 20:52:42 +0200 Andi Kleen <[email protected]> wrote: > On Mon, Apr 30, 2007 at 10:45:10AM -0700, William Lee Irwin III wrote: > > On Mon, Apr 30, 2007 at 10:38:19AM -0700, William Lee Irwin III wrote: > > > Here's what I did for i386 for someone concerned about blowing the stack. > > > > Add checks to __pa() so it goes BUG() on vmallocspace addresses. > > Sorry I think that's too costly to do. __pa is pretty common But not too costly to do if it is done solely with vmalloc the stack for debug purposes. The bigger problem with the vmalloc approach is there are still offenders who DMA off the kernel stack on i386 although I'd hope they are all ancient... we'll find out with this anyway - 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/
- Follow-Ups:
- References:
- [-mm patch] i386: enable 4k stacks by default
- From: Adrian Bunk <[email protected]>
- Re: [-mm patch] i386: enable 4k stacks by default
- From: Zan Lynx <[email protected]>
- Re: [-mm patch] i386: enable 4k stacks by default
- From: David Chinner <[email protected]>
- Re: [-mm patch] i386: enable 4k stacks by default
- From: Alan Cox <[email protected]>
- Re: [-mm patch] i386: enable 4k stacks by default
- From: Andi Kleen <[email protected]>
- Re: [-mm patch] i386: enable 4k stacks by default
- From: Christoph Hellwig <[email protected]>
- Re: [-mm patch] i386: enable 4k stacks by default
- From: Andi Kleen <[email protected]>
- Re: [-mm patch] i386: enable 4k stacks by default
- From: William Lee Irwin III <[email protected]>
- [4/6] go BUG on vmallocspace in __pa() (was: Re: [-mm patch] i386: enable 4k stacks by default)
- From: William Lee Irwin III <[email protected]>
- Re: [4/6] go BUG on vmallocspace in __pa() (was: Re: [-mm patch] i386: enable 4k stacks by default)
- From: Andi Kleen <[email protected]>
- [-mm patch] i386: enable 4k stacks by default
- Prev by Date: Re: [patch] pm: include EIO from errno-base.h
- Next by Date: Re: [PATCH -mm] Allow selective freezing of the system for different events
- Previous by thread: Re: [4/6] go BUG on vmallocspace in __pa() (was: Re: [-mm patch] i386: enable 4k stacks by default)
- Next by thread: Re: [4/6] go BUG on vmallocspace in __pa() (was: Re: [-mm patch] i386: enable 4k stacks by default)
- Index(es):
 |