diff options
author | Shenghou Ma <minux@golang.org> | 2015-04-29 02:42:43 -0400 |
---|---|---|
committer | Minux Ma <minux@golang.org> | 2015-04-29 19:18:07 +0000 |
commit | 4d1ab2d8d10283925e486ad75237ae694f3b6cce (patch) | |
tree | 7eea7bc186636ddc14520411cb6db1f826dbeeb4 /src/runtime/norace_linux_test.go | |
parent | 06946aad1973c42e26ddb0890c57ef37fecbb48a (diff) | |
download | go-4d1ab2d8d10283925e486ad75237ae694f3b6cce.tar.gz go-4d1ab2d8d10283925e486ad75237ae694f3b6cce.zip |
runtime: re-enable TestNewProc0 on android/arm and fix heap corruption
The problem is not actually specific to android/arm. Linux/ARM's
runtime.clone set the stack pointer to child_stk-4 before calling
the fn. And then when fn returns, it tries to write to 4(R13) to
provide argument for runtime.exit, which is just beyond the allocated
child stack, and thus it will corrupt the heap randomly or trigger
segfault if that memory happens to be unmapped.
While we're at here, shorten the test polling interval to 0.1s to
speed up the test (it was only checking at 1s interval, which means
the test takes at least 1s).
Fixes #10548.
Change-Id: I57cd63232022b113b6cd61e987b0684ebcce930a
Reviewed-on: https://go-review.googlesource.com/9457
Reviewed-by: David Crawshaw <crawshaw@golang.org>
Diffstat (limited to 'src/runtime/norace_linux_test.go')
-rw-r--r-- | src/runtime/norace_linux_test.go | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/runtime/norace_linux_test.go b/src/runtime/norace_linux_test.go index c355ccc0f8..bbf9d0b413 100644 --- a/src/runtime/norace_linux_test.go +++ b/src/runtime/norace_linux_test.go @@ -24,13 +24,8 @@ func newOSProcCreated() { // Can't be run with -race because it inserts calls into newOSProcCreated() // that require a valid G/M. func TestNewOSProc0(t *testing.T) { - if runtime.GOOS == "android" && runtime.GOARCH == "arm" { - // newosproc0 does not work for android/arm. - // See issue 10548. - t.Skipf("skipping on %v", runtime.GOOS) - } runtime.NewOSProc0(0x800000, unsafe.Pointer(runtime.FuncPC(newOSProcCreated))) - check := time.NewTicker(1 * time.Second) + check := time.NewTicker(100 * time.Millisecond) defer check.Stop() end := time.After(5 * time.Second) for { |