[PATCH] Decrease number of threads used by goroutines.go test

[Jakub Jelinek]

On Tue, Mar 08, 2011 at 04:27:04PM +0100, Rainer Orth wrote:
> >> 2011-03-08  Jakub Jelinek  <jakub@redhat.com>
> >>
> >> 	* go.test/go-test.exp: For goroutines.go test pass
> >> 	max($[`ulimit -u`/4], 10000) as first argument, or 64 as a safe
> >> 	default.
> >
> > How about if we do this unless the environment variable
> > GCCGO_RUN_ALL_TESTS is set, so that people have a way to run the full
> > testsuite.  I can also change the libgo testsuite to only run the
> > networking tests if that environment variable is set.  This patch is OK
> > with that change.  Thanks for doing it.
> 
> Alternatively, one might use GCC_TEST_RUN_EXPENSIVE, which is already
> used to control dg-require-effective-target run_expensive_tests.  This
> avoids separate mechanisms per testsuite/language.

I guess [getenv GCC_TEST_RUN_EXPENSIVE] != "" could be a usable test here,
if false it could always use 64 threads or something like that, if true
it should IMNSHO still bound it to at most max($[`ulimit -u`/2], 10000)
when ulimit -u is available, because running the thread when it is known
to break other things is a bad idea.
But of course if Ian wants to guard networking libgo tests with some env
var, GCC_TEST_RUN_EXPENSIVE probably isn't an env var to use...

	Jakub