-
Notifications
You must be signed in to change notification settings - Fork 18k
runtime: timeout in TestNeedmDeadlock on darwin-amd64-10_12 builder #44303
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Tentatively milestoning as a release-blocker for Go 1.17 until we better understand the root cause. This may be a deadlock in the runtime, and the failure mode would make it very difficult for a Go user to debug. |
CC @prattmic. |
Friendly ping after 2 weeks since it's a release blocking issue for Go 1.17. |
There seem to only be two failures since the test was added in October, 2020. Both failures are on Darwin 10.12. In both failures the test hits a 3 minute timeout, which is interesting because
It seems that the test program has deadlocked so hard that both of those timeouts fail to stop it. |
This has only happened twice, it's only happened on macOS 10.12, and we are dropping support for macOS 10.12 in 1.17 anyhow. Removing the release-blocker label. |
Change https://golang.org/cl/312692 mentions this issue: |
macOS will not be supported after go1.16. This change limits the macOS 10.12 builders to build previous to go1.17. Updates golang/go#44303 For golang/go#23011 Change-Id: Ib6b4e7939ffb9032b255574d3f0b4fffaad97803 Reviewed-on: https://go-review.googlesource.com/c/build/+/312692 Trust: Carlos Amedee <[email protected]> Run-TryBot: Carlos Amedee <[email protected]> TryBot-Result: Go Bot <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
2021-02-15T01:54:22-33d72fd/darwin-amd64-10_12
2020-10-29T15:13:09-50af50d/darwin-amd64-10_12
It looks like the
testprogcgo
process ends up wedged, butrunBuildTestProg
fails to terminate it so we don't get a useful stack trace.See previously #42207 (the deadlock for which this test was added).
See also #43625 (TestNeedmDeadlock on MIPS).
CC @ianlancetaylor @cherrymui
The text was updated successfully, but these errors were encountered: