net/art: disable the IPv6 100k routes benchmark.

danderson · danderson · commit a5fd51ebdc77 · 2023-04-13T09:04:17.000-07:00
At the current unoptimized memory utilization of the various data structures,
100k IPv6 routes consumes in the ballpark of 3-4GiB, which risks OOMing our
386 test machine.

Until we have the optimizations to (drastically) reduce that consumption,
skip the test that bloats too much for 32-bit machines.

Signed-off-by: David Anderson &lt;danderson@tailscale.com&gt;
diff --git a/net/art/table_test.go b/net/art/table_test.go
@@ -205,7 +205,15 @@ func TestDeleteShuffled(t *testing.T) {
 	}
 }
 
-var benchRouteCount = []int{10, 100, 1000, 10_000, 100_000}
+// 100k routes for IPv6, at the current size of strideTable and strideEntry, is
+// in the ballpark of 4GiB if you assume worst-case prefix distribution. Future
+// optimizations will knock down the memory consumption by over an order of
+// magnitude, so for now just skip the 100k benchmarks to stay well away of
+// OOMs.
+//
+// TODO(go/bug/7781): reenable larger table tests once memory utilization is
+// optimized.
+var benchRouteCount = []int{10, 100, 1000, 10_000} //, 100_000}
 
 // forFamilyAndCount runs the benchmark fn with different sets of
 // routes.
