Define eventfd on NetBSD #4830
Define eventfd on NetBSD #4830
Conversation
krobelus
force-pushed
the
netbsd-eventfd
branch
from
f0917cc to
03be810
Compare
| pub fn eventfd(init: c_uint, flags: c_int) -> c_int; | ||
| pub fn eventfd_read(fd: c_int, value: *mut eventfd_t) -> c_int; | ||
| pub fn eventfd_write(fd: c_int, value: eventfd_t) -> c_int; |
There was a problem hiding this comment.
Any reason that these fn args names don't match with man page?
krobelus
force-pushed
the
netbsd-eventfd
branch
from
03be810 to
2bf5e32
Compare
|
+ pub fn eventfd_write(fd: c_int, value: eventfd_t) -> c_int;
Any reason that these fn args names don't match with man page?
I might have copied it from the FreeBSD ones. Fixed
|
Like FreeBSD, NetBSD supports eventfd, see https://man.netbsd.org/eventfd.2. OpenBSD does not AFAICT. While at, it make the already define eventfd* argument names match the respective OSs man pages (or header files).
JohnTitor
force-pushed
the
netbsd-eventfd
branch
from
2bf5e32 to
916a8cf
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
Please include permalinks to the headers too, manpages don't show the values. In this case: https://github.com/NetBSD/src/blob/d04b0c735abc997743bb3faa74464524cbe7becd/sys/sys/eventfd.h. To fix the CI failure you'll need to add the relevant header to Why is the arg name changed in all the other files? That should be dropped, or at least moved to a separate commit if there is a reason. |
|
Reminder, once the PR becomes ready for a review, use |
I forgot to copy the commit message to the PR description - this is to match the OS manpages (or headers where I didn't find manpages).
sure |
4 participants
Like FreeBSD, NetBSD supports eventfd, see
https://man.netbsd.org/eventfd.2.
OpenBSD does not AFAICT.
(for some reason my local rustfmt wants to reformat imports)