]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/dist: disable misc/cgo/testsanitizers on ppc64le
authorLynn Boger <laboger@linux.vnet.ibm.com>
Fri, 30 Apr 2021 15:21:18 +0000 (10:21 -0500)
committerLynn Boger <laboger@linux.vnet.ibm.com>
Mon, 3 May 2021 12:20:09 +0000 (12:20 +0000)
A while back in this release the sanitizer tests were enabled
for ppc64le, where previously they were never run. This
uncovered some errors in these tests on ppc64le. One linker
fix was made but there are still bugs in how tsan is made to
work within the code, especially in how signals are enabled
with cgo.

Some attempts were made to make this work but intermittent
failures continue to happen with the Trybots so I am just
going to disable this test for ppc64le within cmd/dist.

Updates #45040

Change-Id: I5392368ccecd4079ef568d0c645c9f7c94016d99
Reviewed-on: https://go-review.googlesource.com/c/go/+/315430
Run-TryBot: Lynn Boger <laboger@linux.vnet.ibm.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Trust: Cherry Zhang <cherryyz@google.com>

src/cmd/dist/test.go

index 8b1ec458ed89ac53fadc08a8a4b75b41b089ea0e..33fabd35541cb1e3a4500e7e34d508790632bf41 100644 (file)
@@ -749,8 +749,9 @@ func (t *tester) registerTests() {
                if gohostos == "linux" && goarch == "amd64" {
                        t.registerTest("testasan", "../misc/cgo/testasan", "go", "run", ".")
                }
-               if goos == "linux" {
+               if goos == "linux" && goarch != "ppc64le" {
                        // because syscall.SysProcAttri struct used in misc/cgo/testsanitizers is only built on linux.
+                       // Some inconsistent failures happen on ppc64le so disable for now.
                        t.registerHostTest("testsanitizers", "../misc/cgo/testsanitizers", "misc/cgo/testsanitizers", ".")
                }
                if t.hasBash() && goos != "android" && !t.iOS() && gohostos != "windows" {