From: Michael Anthony Knyszek Date: Wed, 3 Dec 2025 23:13:36 +0000 (+0000) Subject: runtime: relax threadsSlack in TestReadMetricsSched X-Git-Tag: go1.26rc1~2^2~5 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=fe4952f1165c43dcd195ab8b024f79ac5beced8a;p=gostls13.git runtime: relax threadsSlack in TestReadMetricsSched runtime.GC is called in the test and may spin up GOMAXPROCS (proportional to the initial count) new threads. We need to be robust to this, and it happens relatively frequently on some platforms. We didn't notice this earlier since the heap is so miniscule that runtime.GC essentially finished instantly without all the threads getting spun up. Fixes #76613. Change-Id: I2af02cb090d5c1c952e4db53ad35895b6f23f638 Reviewed-on: https://go-review.googlesource.com/c/go/+/726642 LUCI-TryBot-Result: Go LUCI Reviewed-by: Michael Pratt --- diff --git a/src/runtime/testdata/testprog/schedmetrics.go b/src/runtime/testdata/testprog/schedmetrics.go index 8e8abc4484..7fad95a976 100644 --- a/src/runtime/testdata/testprog/schedmetrics.go +++ b/src/runtime/testdata/testprog/schedmetrics.go @@ -91,8 +91,10 @@ func SchedMetrics() { // threads through frequent scheduling, like mayMoreStackPreempt. // A slack of 5 is arbitrary but appears to be enough to cover // the leftovers plus any inflation from scheduling-heavy build - // modes. - const threadsSlack = 5 + // modes. We then also add initialGMP to this slack, since we're + // about to call runtime.GC, and in the worst case this will + // spin up GOMAXPROCS new threads to run those workers. + threadsSlack := 5 + uint64(initialGMP) // Make sure GC isn't running, since GC workers interfere with // expected counts.