[PATCH] Fix nptl/tst-cond1{6,7} on 32-bit with many cpus
David Miller
davem@davemloft.net
Tue Mar 27 21:27:00 GMT 2012
From: "Carlos O'Donell" <carlos@systemhalted.org>
Date: Tue, 27 Mar 2012 14:45:27 -0400
> On Mon, Mar 26, 2012 at 5:34 PM, David Miller <davem@davemloft.net> wrote:
>> From: "Carlos O'Donell" <carlos@systemhalted.org>
>> Date: Mon, 26 Mar 2012 10:44:21 -0400
>>> On Sun, Mar 25, 2012 at 9:47 PM, David Miller <davem@davemloft.net> wrote:
>>>> @@ -76,9 +76,15 @@ do_test (void)
>>>> count *= 4;
>>>>
>>>> pthread_t th[count];
>>>> - int i, ret;
>>>> + pthread_attr_t attr;
>>>> + int i, ret, sz;
>>>> + pthread_attr_init (&attr);
>>>> + sz = __getpagesize ();
>>>> + if (sz < 64 * 1024)
>>>> + sz = 64 * 1024;
>>>
>>> Should this be PTHREAD_STACK_MIN instead to allow for per-machine
>>> variations? Is it sufficient to use PTHREAD_STACK_MIN?
>>
>> We still need to take __getpagesize() into account, because some
>> PTHREAD_STACK_MIN definitions are smaller than the largest possible
>> page size on the respective architecture.
>
> This looks good to me, please check this in.
Done, thanks for reviewing.
More information about the Libc-alpha
mailing list