code review 5753054: test/run: use all available cores on ARM system

test/run: use all available cores on ARM system

[rsc1, 2012-03-06 14:32:46]

http://codereview.appspot.com/5753054/diff/2001/test/run.go
File test/run.go (right):

http://codereview.appspot.com/5753054/diff/2001/test/run.go#newcode33
test/run.go:33: numParallel = flag.Int("n", get_default_parallelism(), "number
of parallel tests to run")
http://golang.org/doc/effective_go.html#Getters
http://golang.org/doc/effective_go.html#mixed-caps

defaultParallel is fine.

[minux1, 2012-03-06 14:37:51]

Hello rsc@golang.org (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/go/

[rsc, 2012-03-06 14:41:32]

Let's just go back to a default of runtime.NumCPU()
and then we don't need the function at all.
I was trying to help uniprocessors overlap some
computation with I/O but that appears to be misguided.

[minux1, 2012-03-06 14:48:09]

PTAL.

On Tue, Mar 6, 2012 at 10:41 PM, Russ Cox <rsc@golang.org> wrote:

> Let's just go back to a default of runtime.NumCPU()
> and then we don't need the function at all.
> I was trying to help uniprocessors overlap some
> computation with I/O but that appears to be misguided.
>
OK.

[rsc, 2012-03-07 02:34:56]

LGTM

[minux1, 2012-03-07 04:43:33]

*** Submitted as http://code.google.com/p/go/source/detail?r=e161795e4095 ***

test/run: use all available cores on ARM system

R=rsc
CC=golang-dev
http://codereview.appspot.com/5753054