]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: skip TestMemmoveOverflow with asan
authorMichael Anthony Knyszek <mknyszek@google.com>
Tue, 29 Oct 2024 21:13:41 +0000 (21:13 +0000)
committerGopher Robot <gobot@golang.org>
Tue, 29 Oct 2024 22:12:53 +0000 (22:12 +0000)
On a whim I decided to investigate the possibility of whether the
flakiness on the asan builder was due to a concurrently executing test.
Of the most recent failures there were a few candidates, and this test
was one of them. After disabling each candidate one by one, we had a
winner: this test causes other concurrently executing tests, running
pure Go code, to spuriously fail.

I do not know why yet, but this test doesn't seem like it would have
incredibly high value for ASAN, and does funky things like MAP_FIXED in
recently unmapped regions, so I think it's fine.

For #70054.
For #64257.

Change-Id: Ib9a84d9b69812e76c390d99b00698710ee1ece1a
Cq-Include-Trybots: luci.golang.try:gotip-linux-amd64-asan-clang15
Reviewed-on: https://go-review.googlesource.com/c/go/+/623336
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
src/runtime/memmove_linux_amd64_test.go

index 5f900623bee83d12432fc995c9d0a3046f9c040c..c558811599cc2bffad1cd8d37cc5ad493ec44541 100644 (file)
@@ -5,6 +5,7 @@
 package runtime_test
 
 import (
+       "internal/asan"
        "os"
        "syscall"
        "testing"
@@ -14,6 +15,10 @@ import (
 // TestMemmoveOverflow maps 3GB of memory and calls memmove on
 // the corresponding slice.
 func TestMemmoveOverflow(t *testing.T) {
+       if asan.Enabled {
+               t.Skip("appears to break asan and causes spurious failures")
+       }
+
        t.Parallel()
        // Create a temporary file.
        tmp, err := os.CreateTemp("", "go-memmovetest")