]> Cypherpunks.ru repositories - gostls13.git/commit
runtime: eliminate arbitrary timeout in TestCgoLockOSThreadExit
authorBryan C. Mills <bcmills@google.com>
Thu, 2 Feb 2023 15:09:39 +0000 (10:09 -0500)
committerGopher Robot <gobot@golang.org>
Thu, 2 Feb 2023 20:21:33 +0000 (20:21 +0000)
commitfc86770d87360ddf0346bf407a80935fe2dddec8
treecc22d177082dd0e44890aa74e3eb3725967ae34c
parentfcd0e0963f3bdf289216c1083f1439a71dc571a4
runtime: eliminate arbitrary timeout in TestCgoLockOSThreadExit

This test previously failed if running a new pthread took longer than
a hard-coded 100ms. On some slow or heavily-loaded builders, that
scheduling latency is too short.

Since the point of this test is to verify that the background thread
is not reused after it terminates (see #20395), the arbitrary time
limit does not seem helpful: if the background thread fails to
terminate the test will time out on its own, and if the main goroutine
is scheduled on the background thread the test will fail regardless of
how long it takes.

Fixes #58247.

Change-Id: I626af52aac55af7a4c0e7829798573c479750c20
Reviewed-on: https://go-review.googlesource.com/c/go/+/464735
Run-TryBot: Bryan Mills <bcmills@google.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Auto-Submit: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
src/runtime/crash_test.go
src/runtime/testdata/testprogcgo/lockosthread.go