Skip to content
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

os: CL 156379 causes deadlocks on FreeBSD 10.3 builders #29633

Closed
bcmills opened this issue Jan 9, 2019 · 4 comments
Closed

os: CL 156379 causes deadlocks on FreeBSD 10.3 builders #29633

bcmills opened this issue Jan 9, 2019 · 4 comments

Comments

@bcmills
Copy link
Member

@bcmills bcmills commented Jan 9, 2019

https://golang.org/cl/156379 (which updates #19093) seems to be causing consistent deadlocks in the FreeBSD 10.3 builders:
https://build.golang.org/log/6e053738a50b6d2783d82146712ea469501a63a3
https://build.golang.org/log/dd23083ba54882a43e2d70fcf984722df0191e61
https://build.golang.org/log/6117379f9204b491204d9412db024eb95847248f
https://build.golang.org/log/6087ce300fadc2b1461dffbbfb5171d70fad965f

It is not obvious to me whether this should block the 1.12 release: it seems the 10.3 builders are flaky in general (see #25289). If not, please, remove the release-blocker label.

CC @ianlancetaylor @paulzhol @tklauser

@ianlancetaylor

This comment has been minimized.

Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Jan 9, 2019

According to the release notes Go 1.12 should still support FreeBSD 10, so perhaps https://golang.org/cl/156379 was overly optimistic.

@gopherbot

This comment has been minimized.

Copy link

@gopherbot gopherbot commented Jan 9, 2019

Change https://golang.org/cl/157099 mentions this issue: os: always treat files as unpollable on FreeBSD

@ianlancetaylor

This comment has been minimized.

Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Jan 9, 2019

CL link doesn't work due to #28836, see https://go-review.googlesource.com/c/go/+/157099.

@gopherbot

This comment has been minimized.

Copy link

@gopherbot gopherbot commented Mar 7, 2019

Change https://golang.org/cl/165801 mentions this issue: os: drop special case for FreeBSD 10.4 in newFile

gopherbot pushed a commit that referenced this issue Mar 12, 2019
Support for FreeBSD 10 will be dropped with Go 1.13, so revert the
workaround introduced in CL 157099.

Updates #29633
Updates #27619

Change-Id: I1a2e50d3f807a411389f3db07c0f4535a590da02
Reviewed-on: https://go-review.googlesource.com/c/go/+/165801
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
@golang golang locked and limited conversation to collaborators Mar 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.