code review 13368045: runtime: re-enable TestParForParallel

runtime: re-enable TestParForParallel

See how it flies. We'll disable it again if the underlying issue is not resolved.
See issue 4155 for details.

Fixes issue 4155.

[r, 2013-08-29 01:31:51]

Hello golang-dev@googlegroups.com,

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

[bradfitz, 2013-08-29 01:32:23]

LGTM
 On Aug 28, 2013 6:31 PM, <r@golang.org> wrote:

> Reviewers: golang-dev1,
>
> Message:
> Hello golang-dev@googlegroups.com,
>
> I'd like you to review this change to
> https://code.google.com/p/go/
>
>
> Description:
> runtime: re-enable TestParForParallel
>
> See how it flies. We'll disable it again if the underlying issue is not
> resolved.
> See issue 4155 for details.
>
> Fixes issue 4155.
>
> Please review this at
https://codereview.appspot.**com/13368045/<https://codereview.appspot.com/133...
>
> Affected files:
>   M src/pkg/runtime/parfor_test.go
>
>
> Index: src/pkg/runtime/parfor_test.go
> ==============================**==============================**=======
> --- a/src/pkg/runtime/parfor_test.**go
> +++ b/src/pkg/runtime/parfor_test.**go
> @@ -102,11 +102,6 @@
>
>  // Test parallel parallelfor.
>  func TestParForParallel(t *testing.T) {
> -       if GOARCH != "amd64" {
> -               t.Log("temporarily disabled, see
> http://golang.org/issue/4155")
> -               return
> -       }
> -
>         N := uint64(1e7)
>         if testing.Short() {
>                 N /= 10
>
>
> --
>
> ---You received this message because you are subscribed to the Google
> Groups "golang-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to
golang-dev+unsubscribe@**googlegroups.com<golang-dev%2Bunsubscribe@googlegrou...
> .
> For more options, visit
https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/o...
> .
>

[r, 2013-08-29 01:42:25]

*** Submitted as https://code.google.com/p/go/source/detail?r=7f8fb6d64c75 ***

runtime: re-enable TestParForParallel

See how it flies. We'll disable it again if the underlying issue is not
resolved.
See issue 4155 for details.

Fixes issue 4155.

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/13368045