Re: [PATCH v4 0/4] metag,parisc: Stack limit patches

[James Hogan <james.hogan-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 2728 bytes --]

Hi Helge,

On 14/05/14 20:22, Helge Deller wrote:
> On 05/14/2014 02:52 PM, James Hogan wrote:
>> Patch 1 is a metag bug fix for v3.15 & stable.
>>
>> Patch 2 depends on patch 1, and makes the stack limit configurable.
>> Helge: I don't object if you want to take this patch yourself since it
>> was originally yours, so long as it goes after patch 1 (which I'll try
>> and get into v3.15-rc6).
> 
> Thanks for cleaning it up!
> I'd really prefer if you could push the whole series through your tree.
> That way you can easily ensure the correct sequence.

Okay, no problem, and I see the parisc patch is already in mainline.

> IMHO, it would be nice if all four patches would still go into 3.15, esp.
> since it affects our platforms (parisc & metag) only.

Right. That doesn't sound unreasonable since patch 2 restores default
80MB stacks for parisc, patch 3 may prevent confusing userland, and
patch 4 is just an obvious cleanup as a result of earlier patches.

Cheers
James

> 
> Helge
> 
>>
>> Patch 3 depends on patch 1, and removes the RLIMIT_STACK default value
>> for metag. I'll take this through metag tree for v3.16.
>>
>> Patch 4 depends on patch 3 and the corresponding parisc patch and
>> removes the ability to override the default hard RLIMIT_STACK. Depending
>> on when the parisc patch reaches mainline I'll try and get this in for
>> v3.16.
>>
>> Helge Deller (1):
>>   parisc,metag: Do not hardcode maximum userspace stack size
>>
>> James Hogan (3):
>>   metag: Reduce maximum stack size to 256MB
>>   metag: Remove _STK_LIM_MAX override
>>   asm-generic: remove _STK_LIM_MAX
>>
>>  arch/metag/include/asm/processor.h     |  2 ++
>>  arch/metag/include/uapi/asm/Kbuild     |  2 +-
>>  arch/metag/include/uapi/asm/resource.h |  7 -------
>>  arch/parisc/include/asm/processor.h    |  5 +++++
>>  arch/parisc/kernel/sys_parisc.c        |  6 +++---
>>  fs/exec.c                              |  6 +++---
>>  include/asm-generic/resource.h         |  2 +-
>>  include/uapi/asm-generic/resource.h    |  7 -------
>>  mm/Kconfig                             | 15 +++++++++++++++
>>  9 files changed, 30 insertions(+), 22 deletions(-)
>>  delete mode 100644 arch/metag/include/uapi/asm/resource.h
>>
>> Cc: Helge Deller <deller-Mmb7MZpHnFY@public.gmane.org>
>> Cc: "James E.J. Bottomley" <jejb-6jwH94ZQLHl74goWV3ctuw@public.gmane.org>
>> Cc: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
>> Cc: John David Anglin <dave.anglin-CzeTG9NwML0@public.gmane.org>
>> Cc: linux-arch-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>> Cc: linux-metag-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>> Cc: linux-parisc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>>
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]