[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