[PATCH] Fix manythreads test case on s390

[PATCH] Fix manythreads test case on s390

[Ulrich Weigand]

Hello,

the new manythreads.c test case fails on s390 when using NPTL,
because it tries to create 256 threads with default thread stack
size (which happens to be 8MB for NPTL).  This is hard to do
when the total address space size is 2 GB...

This can be fixed by using pthread_addr_setstacksize to reduce
the thread stack size.

Bye,
Ulrich

ChangeLog:

	* gdb.threads/manythreads.c: Reduce thread stack size.

Index: gdb/testsuite/gdb.threads/manythreads.c
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.threads/manythreads.c,v
retrieving revision 1.1
diff -c -p -r1.1 manythreads.c
*** gdb/testsuite/gdb.threads/manythreads.c	22 Apr 2004 22:19:40 -0000	1.1
--- gdb/testsuite/gdb.threads/manythreads.c	23 Apr 2004 16:15:20 -0000
*************** main (int argc, char **argv)
*** 19,24 ****
--- 19,25 ----
    int i, j;
  
    pthread_attr_init (&attr);
+   pthread_attr_setstacksize (&attr, PTHREAD_STACK_MIN);
  
    /* Create a ton of quick-executing threads, then wait for them to
       complete.  */
-- 
  Dr. Ulrich Weigand
  weigand@informatik.uni-erlangen.de

Re: [PATCH] Fix manythreads test case on s390

[Daniel Jacobowitz]

On Fri, Apr 23, 2004 at 06:33:48PM +0200, Ulrich Weigand wrote:
> Hello,
> 
> the new manythreads.c test case fails on s390 when using NPTL,
> because it tries to create 256 threads with default thread stack
> size (which happens to be 8MB for NPTL).  This is hard to do
> when the total address space size is 2 GB...
> 
> This can be fixed by using pthread_addr_setstacksize to reduce
> the thread stack size.

None of the threads run to completion in time to reuse their stack
space?  I'm surprised by that (though it's certainly a valid outcome,
it's unlike how NPTL behaves elsewhere).

Anyway, I've no objection to the patch.

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer

Re: [PATCH] Fix manythreads test case on s390

[Jeff Johnston]

Andrew Cagney wrote:
>> Hello,
>>
>> the new manythreads.c test case fails on s390 when using NPTL,
>> because it tries to create 256 threads with default thread stack
>> size (which happens to be 8MB for NPTL).  This is hard to do
>> when the total address space size is 2 GB...
>>
>> This can be fixed by using pthread_addr_setstacksize to reduce
>> the thread stack size.
> 
> 
> Outch, the fix is "obvious".  Jeff, what to check this in along with the 
> (C) fix?
> 

Both changes checked in.

-- Jeff J.

Re: [PATCH] Fix manythreads test case on s390

[Ulrich Weigand]

Daniel Jacobowitz wrote:

> None of the threads run to completion in time to reuse their stack
> space?  I'm surprised by that (though it's certainly a valid outcome,
> it's unlike how NPTL behaves elsewhere).

On my system the 127th pthread_create reproducibly fails, which
is consistent with no thread stack being reused early.

Maybe it's an SMP effect (one CPU is quickly running all the 
pthread_create's, while the others are executing the thread
functions, which may be slow due to printf ...)?

On the other hand, it may also be the case that running in a
virtualized environment exaggerates race windows; we've seen
plenty of examples for this.

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  weigand@informatik.uni-erlangen.de

Re: [PATCH] Fix manythreads test case on s390

[Andrew Cagney]

> Hello,
> 
> the new manythreads.c test case fails on s390 when using NPTL,
> because it tries to create 256 threads with default thread stack
> size (which happens to be 8MB for NPTL).  This is hard to do
> when the total address space size is 2 GB...
> 
> This can be fixed by using pthread_addr_setstacksize to reduce
> the thread stack size.

Outch, the fix is "obvious".  Jeff, what to check this in along with the 
(C) fix?

Andrew

> Bye,
> Ulrich
> 
> ChangeLog: