From f82c59b6cf4cc50964c6068a704647db5a73c4fa Mon Sep 17 00:00:00 2001 From: Shenghou Ma Date: Mon, 8 Oct 2012 01:56:27 +0800 Subject: [PATCH] runtime: disable TestParForParallel for now on 32-bit hosts Also add call to GC() to make it easier to re-enable the test. Update #4155. When we have precise GC merged, re-enable this test. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/6622058 --- src/pkg/runtime/parfor_test.go | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/pkg/runtime/parfor_test.go b/src/pkg/runtime/parfor_test.go index 055c134a93..0547db0209 100644 --- a/src/pkg/runtime/parfor_test.go +++ b/src/pkg/runtime/parfor_test.go @@ -92,6 +92,11 @@ func TestParForSetup(t *testing.T) { // 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 @@ -114,4 +119,7 @@ func TestParForParallel(t *testing.T) { t.Fatalf("Wrong element %d: %d", i, data[i]) } } + + data, desc = nil, nil + GC() } -- 2.48.1