Bug #3268
closedparent process does not receive POLLHUP events
Description
The attached `main.c` code is a reduction which takes as args, a command (absolute path) to fork/exec and poll stdout/stderr. POLLHUP events should be issued for both pipes when the child is finished.
For the Zig project we have this implemented in the zig language but I've provided a similar .c implementation since it was already done during my triage. For both .zig and .c code, linux, macos, freebsd, netbsd, openbsd work as expected and dragonfly manifests this unexpected behaviour of not receiving POLLHUP.
In the provided reduction, the bug manifests as an infinite loop after reading the bytes from child stdout/stderr. The parent keeps getting POLLIN events, and the read keeps returning 0 nbytes read which indicate the pipe has been closed. In the interim I will likely workaround this bug with conditional code for dragonfly to key off nbytes=0.
I have reproduced the bug on dragonfly 5.8, 6.0 and 5.0.0 (the oldest .iso I could find).
Files
Updated by dillon over 3 years ago
- File pollhup01.patch pollhup01.patch added
- Status changed from New to In Progress
- Assignee set to dillon
- Priority changed from Normal to High
Please try the included patch. I did a quick test and it appears to work.
Currently only the socket code allows POLLHUP to be set. This patch will also enable it for the pipe code. And if it looks good I should probably also implement the same thing for the fifo code.
-Matt
Updated by dillon over 3 years ago
- File pollhup02.patch pollhup02.patch added
pollhup02.patch is a more complete patch (also removes unrelated cruft that was in my tree). It patches the read and write side for pipes and for filesystem fifos.
-Matt
Updated by mikdusan over 3 years ago
confirmed : pollhup02.patch fixes issue for me:
- applied against
DragonFly_RELEASE_6_0
- main.c now works
- zig
std.ChildProcess.exec()
now works
Thank you much Matt!
Updated by tuxillo over 3 years ago
- Target version changed from 6.2 to 6.0
- % Done changed from 0 to 100
Updated by dillon over 3 years ago
- Status changed from In Progress to Resolved
Ok, excellent. I have pushed the patch to 6.0-release and master. Going to close the bug. Thanks!
-Matt