This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v2] Fix stacksize returned for main process thread whenRLIMIT_STACK is relevant
- From: "Carlos O'Donell" <carlos at systemhalted dot org>
- To: Siddhesh Poyarekar <siddhesh at redhat dot com>
- Cc: libc-alpha at sourceware dot org, KOSAKI Motohiro <kosaki dot motohiro at gmail dot com>, Roland McGrath <roland at hack dot frob dot com>
- Date: Mon, 18 Jun 2012 23:47:55 -0400
- Subject: Re: [PATCH v2] Fix stacksize returned for main process thread whenRLIMIT_STACK is relevant
- References: <20120615005913.4f517e02@spoyarek><20120619091017.14a25471@spoyarek>
On Mon, Jun 18, 2012 at 11:40 PM, Siddhesh Poyarekar
<siddhesh@redhat.com> wrote:
> Hi,
>
> This is version 2 of the patch. I have removed the conditional that
> Kosaki pointed out as unnecessary; everything else remains the same.
>
> nptl/ChangeLog:
>
> 2012-06-15 ?Siddhesh Poyarekar ?<siddhesh@redhat.com>
>
> ? ? ? ?* Makefile (tests): Add test case.
> ? ? ? ?* pthread_getattr_np.c (pthread_getattr_np): Deduct pages below
> ? ? ? ?the __libc_stack_end page from stacksize. ?Truncate stacksize
> ? ? ? ?to make it page aligned when it is computed from RLIMIT_STACK.
> ? ? ? ?* tst-pthread-getattr.c: New test case. Verify that stackaddr
> ? ? ? ?is accessible.
Please add a license header to tst-pthread-getattr.c.
This patch looks good for 2.17 once we are out of the code freeze.
I'd suggest filing a bug to keep track of this, but I know you won't forget :-)
Cheers,
Carlos.