Commit bc471a52 authored by Daniel Martí's avatar Daniel Martí Committed by Ian Lance Taylor

net: reduce TestSplice/big's memory usage

The old code used splice on a 2GB []byte when not in short mode, meaning
that running 'go test net' when one had 4GB or less free memory would
easily result in "out of memory" runtime panics.

Instead, use a much smaller size that is still big enough to not fit
into a single splice(2) syscall. The new size is just 5MB, so the test
uses a fraction of the memory it used to, and there's no longer a need
for a different size on short mode.

This also speeds up the test, which goes from ~1.23s to ~0.01s on my
laptop.

Fixes #26867.

Change-Id: Iae1daa5c0995b549f41992f44339be32ca1ee5e4
Reviewed-on: https://go-review.googlesource.com/128535
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
Reviewed-by: default avatarAndrei Tudor Călin <mail@acln.ro>
Reviewed-by: default avatarIan Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
parent 15c106d9
...@@ -80,10 +80,10 @@ func testSpliceMultipleWrite(t *testing.T) { ...@@ -80,10 +80,10 @@ func testSpliceMultipleWrite(t *testing.T) {
} }
func testSpliceBig(t *testing.T) { func testSpliceBig(t *testing.T) {
size := 1<<31 - 1 // The maximum amount of data that internal/poll.Splice will use in a
if testing.Short() { // splice(2) call is 4 << 20. Use a bigger size here so that we test an
size = 1 << 25 // amount that doesn't fit in a single call.
} size := 5 << 20
srv, err := newSpliceTestServer() srv, err := newSpliceTestServer()
if err != nil { if err != nil {
t.Fatal(err) t.Fatal(err)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment