Commit 82ec4cd7 authored by Mikio Hara's avatar Mikio Hara

net: don't crash DNS flood test on darwin

Also renames the test function to TestDNSFlood.

Updates #15659.

Change-Id: Ia562004c43bcc19c2fee9440321c27b591f85da5
Reviewed-on: https://go-review.googlesource.com/23077Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
parent da5ac69b
......@@ -371,12 +371,23 @@ func TestReverseAddress(t *testing.T) {
}
}
func TestLookupIPDeadline(t *testing.T) {
func TestDNSFlood(t *testing.T) {
if !*testDNSFlood {
t.Skip("test disabled; use -dnsflood to enable")
}
const N = 5000
var N = 5000
if runtime.GOOS == "darwin" {
// On Darwin this test consumes kernel threads much
// than other platforms for some reason.
// When we monitor the number of allocated Ms by
// observing on runtime.newm calls, we can see that it
// easily reaches the per process ceiling
// kern.num_threads when CGO_ENABLED=1 and
// GODEBUG=netdns=go.
N = 500
}
const timeout = 3 * time.Second
ctxHalfTimeout, cancel := context.WithTimeout(context.Background(), timeout/2)
defer cancel()
......
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