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

Jump to bottom

os/exec: test fails with "runtime: netpoll: break fd ready for -2" on OpenBSD 6.2 builders #35566

Closed
bcmills opened this issue Nov 13, 2019 · 2 comments
Closed

os/exec: test fails with "runtime: netpoll: break fd ready for -2" on OpenBSD 6.2 builders #35566

bcmills opened this issue Nov 13, 2019 · 2 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-OpenBSD Soon This needs to be done soon. (regressions, serious bugs, outages) Testing An issue that has been verified to require only test changes, not just a test failure.
Milestone
Go1.14

Comments

@bcmills
Copy link
Contributor

bcmills commented Nov 13, 2019

CL 206939, a test-only change, introduced a regression on the OpenBSD 6.2 builders (https://build.golang.org/log/edea2d1e1ca4fabe71e3cf98868411e29cfb42cb):

fd 3 open at test start
fd 4 open at test start
fd 5 open at test start
runtime: netpoll: break fd ready for -2
fatal error: runtime: netpoll: break fd ready for something unexpected

runtime stack:
runtime.throw(0x83494b2, 0x39)
	/tmp/workdir/go/src/runtime/panic.go:1106 +0x64
runtime.netpoll(0x0, 0x0, 0xac)
	/tmp/workdir/go/src/runtime/netpoll_kqueue.go:140 +0x35d
runtime.sysmon()
	/tmp/workdir/go/src/runtime/proc.go:4526 +0x34e
runtime.mstart1()
	/tmp/workdir/go/src/runtime/proc.go:1133 +0x8f
runtime.mstart()
	/tmp/workdir/go/src/runtime/proc.go:1098 +0x4f

goroutine 1 [runnable]:
testing.(*M).before(0x8855a40)
	/tmp/workdir/go/src/testing/testing.go:1262
testing.(*M).Run(0x8855a40, 0x0)
	/tmp/workdir/go/src/testing/testing.go:1158 +0x77
main.main()
	_testmain.go:108 +0xfe
FAIL	os/exec	0.108s
@bcmills bcmills added Testing An issue that has been verified to require only test changes, not just a test failure. OS-OpenBSD NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. Soon This needs to be done soon. (regressions, serious bugs, outages) labels Nov 13, 2019
@bcmills bcmills added this to the Go1.14 milestone Nov 13, 2019
@bcmills bcmills changed the title os/exec: "fatal error: runtime: netpoll: break fd ready for something unexpected" on OpenBSD 6.2 builders os/exec: "runtime: netpoll: break fd ready for -2" on OpenBSD 6.2 builders Nov 13, 2019
@bcmills bcmills changed the title os/exec: "runtime: netpoll: break fd ready for -2" on OpenBSD 6.2 builders os/exec: test fails with "runtime: netpoll: break fd ready for -2" on OpenBSD 6.2 builders Nov 13, 2019
@ianlancetaylor
Copy link
Contributor

CL 207078

@gopherbot
Copy link

Change https://golang.org/cl/207078 mentions this issue: os/exec: skip poll descriptors when checking for open descriptors

@jbardin jbardin mentioned this issue Mar 7, 2020
Closed
@golang golang locked and limited conversation to collaborators Nov 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-OpenBSD Soon This needs to be done soon. (regressions, serious bugs, outages) Testing An issue that has been verified to require only test changes, not just a test failure.
Projects
None yet
Milestone
Go1.14
Development

No branches or pull requests
3 participants
@ianlancetaylor @bcmills @gopherbot