* [PATCH] kqueue: ensure close-on-exec flag is set
@ 2017-04-14 1:54 7% Eric Wong
0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2017-04-14 1:54 UTC (permalink / raw)
To: sleepy-penguin
Calling IO::new apparently does not set the close-on-exec flag
in Ruby 2.x (tested with 2.2.6 on FreeBSD), so we must set it
ourselves.
It's not a huge deal to be missing close-on-exec for kqueue,
since kqueue already has close-on-fork behavior(!) and it's rare
for a process using kqueue to exec without forking, first.
---
ext/sleepy_penguin/kqueue.c | 6 +++++-
test/test_kqueue.rb | 1 +
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/ext/sleepy_penguin/kqueue.c b/ext/sleepy_penguin/kqueue.c
index 430bc88..d7e8d8e 100644
--- a/ext/sleepy_penguin/kqueue.c
+++ b/ext/sleepy_penguin/kqueue.c
@@ -110,6 +110,7 @@ static VALUE s_new(VALUE klass)
{
VALUE rv;
int fd = kqueue();
+ int flags;
if (fd < 0) {
/*
@@ -122,9 +123,12 @@ static VALUE s_new(VALUE klass)
rb_sys_fail("kqueue");
}
+ flags = fcntl(fd, F_GETFD);
+ if (flags != -1)
+ fcntl(fd, F_SETFD, flags | FD_CLOEXEC);
+
rv = INT2FIX(fd);
- /* This will set FD_CLOEXEC on Ruby 2.0.0+: */
return rb_call_super(1, &rv);
}
diff --git a/test/test_kqueue.rb b/test/test_kqueue.rb
index ae3203d..9d853c1 100644
--- a/test/test_kqueue.rb
+++ b/test/test_kqueue.rb
@@ -6,6 +6,7 @@ class TestKqueue < Test::Unit::TestCase
def test_kqueue
kq = Kqueue.new
assert_kind_of IO, kq.to_io
+ assert_predicate kq.to_io, :close_on_exec?
rd, wr = IO.pipe
ev = Kevent[rd.fileno, EvFilt::READ, Ev::ADD|Ev::ONESHOT, 0, 0, rd]
thr = Thread.new do
--
EW
^ permalink raw reply related [relevance 7%]
* [ANN] sleepy_penguin 3.5.1 - Linux I/O events (and more) for Ruby
@ 2018-12-15 4:46 4% ` Eric Wong
0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2018-12-15 4:46 UTC (permalink / raw)
To: ruby-talk, sleepy-penguin
sleepy_penguin provides access to newer, Linux-only system calls to wait
on events from traditionally non-I/O sources. Bindings to the eventfd,
timerfd, inotify, and epoll interfaces are provided. Experimental support
for kqueue on FreeBSD (and likely OpenBSD/NetBSD) is also provided.
* homepage: https://bogomips.org/sleepy_penguin/
* git clone https://bogomips.org/sleepy_penguin.git
* Atom feed: https://bogomips.org/sleepy_penguin/NEWS.atom.xml
* mailing list: sleepy-penguin@bogomips.org
* mail archives: https://bogomips.org/sleepy-penguin/
nntp://news.public-inbox.org/inbox.comp.lang.ruby.sleepy-penguin
https://bogomips.org/sleepy-penguin/new.atom
Changes:
sleepy_penguin 3.5.1
This release fixes some minor things for kqueue and clang
users. Not much going on... *yawn*
6 changes since v3.5.0 (2017-03-22):
kqueue: ensure close-on-exec flag is set
kqueue: modernize Struct usage for Ruby 2.5.0dev
fix 64-to-32-bit truncation warning
kqueue: get rid of unused variable warnings
epoll: avoid reduce block capture and ivar requests
doc: use HTTPS URLS when possible
^ permalink raw reply [relevance 4%]
Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2017-03-22 7:27 [ANN] sleepy_penguin 3.5.0 - Linux I/O events (and more) for Ruby Eric Wong
2018-12-15 4:46 4% ` [ANN] sleepy_penguin 3.5.1 " Eric Wong
2017-04-14 1:54 7% [PATCH] kqueue: ensure close-on-exec flag is set Eric Wong
Code repositories for project(s) associated with this public inbox
https://yhbt.net/sleepy_penguin.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).