]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: relax TestMemoryLimit on darwin a bit more
authorKeith Randall <khr@golang.org>
Thu, 2 Oct 2025 20:09:03 +0000 (13:09 -0700)
committerKeith Randall <khr@google.com>
Mon, 17 Nov 2025 21:45:01 +0000 (13:45 -0800)
Add 8MB more. Covers most of the failures watchflakes has seen.

Fixes #73136

Change-Id: I593c599a9519b8b31ed0f401d4157d27ac692587
Reviewed-on: https://go-review.googlesource.com/c/go/+/708617
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Keith Randall <khr@google.com>
src/runtime/testdata/testprog/gc.go

index bbe1453401325f8efcca14951c11ad5eb3bbf367..32e2c5e1b49521de27277de7c938c949f00143e0 100644 (file)
@@ -396,7 +396,7 @@ func gcMemoryLimit(gcPercent int) {
                // should do considerably better than this bound.
                bound := int64(myLimit + 16<<20)
                if runtime.GOOS == "darwin" {
-                       bound += 16 << 20 // Be more lax on Darwin, see issue 73136.
+                       bound += 24 << 20 // Be more lax on Darwin, see issue 73136.
                }
                start := time.Now()
                for time.Since(start) < 200*time.Millisecond {