Re: [PATCH] parisc: increase kernel stack size to 32k

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2013-04-24 at 09:33 +0200, Helge Deller wrote:
> On 04/23/2013 10:22 PM, Helge Deller wrote:
> > commit e4e1e78facf7565cada909a69c7fb6415b6e7b83
> > Author: Helge Deller <deller@xxxxxx>
> > Date:   Tue Apr 23 17:19:37 2013 +0200
> > 
> > parisc: increase kernel stack size to 32k
> > 
> > --- a/arch/parisc/include/asm/thread_info.h
> > +++ b/arch/parisc/include/asm/thread_info.h
> > -#define THREAD_SIZE_ORDER            2
> > +#define THREAD_SIZE_ORDER            3	/* 32k stack */
> 
> I tested again, and it actually needs to be 64k stacks to not crash any longer.
> So, the right temporary fix is:
> 
> > +#define THREAD_SIZE_ORDER            4	/* 64k stack */
> 
> Will send updated patch soon.

This is an indicator of something seriously wrong somewhere.  We've
always had the 16k stack just because of our large frames.  In theory,
the IRQ stack should only be the same size as the kernel stack, so if we
have both in the same place, we should only need at max 32k ... if we're
still seeing problems related to stack overrun, then it might be we have
an IRQ recursion where we shouldn't have.  To be honest, I have a hard
time explaining why our stacks should be over 8k.

James


--
To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux