io-event 1.1.7 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0202b38104b1b6254a15fc12f687f9102509ba75592b1584d3dd5d0e44c497fb
-  data.tar.gz: 925de8b198ea2a9fca99b7c02c5262dd691a9864601c7db109a79c332804b944
+  metadata.gz: 02cbd4beff5335467997d91cbdc8e3bbf509947a15e2f36636b3e4e29b7cd91e
+  data.tar.gz: 9ca1138136d115d1fdfd4e41446f6f7b684de878ec6a7834f27a9ea6ba210896
 SHA512:
-  metadata.gz: d0bb87512e17c8be8ca0b04390466b0c3de198c6ee4a4a0a074b31d8e2f3bb0f482ece3378d0d9653b9219d36f06d41882f4c78631b27f6c0fb02e4143688d33
-  data.tar.gz: 4403550628403eccfb92d60e44ccf096ecb2dee1f2e78917fbef65b8c49cc71b15db96b3074e5b16f0ce005f826db928e4a95fd1214973cf33509659195a2e9a
+  metadata.gz: 5fe958c76100f346101702b285b140bebe3a149525288cf8e83a1a1806a710928627e8c2d5ae4d8ec716836c264d6f645daa77b944ce1d59a5aaef6ac572e7aa
+  data.tar.gz: 9ce5953f7c6c3fba1e41e61fcc4ae7e24a294624a004fc532724fd49c707afc14a837359547df45611e0df6ab35e9d954e6f8e09eed748c845ad6ed8a6b61076

data/ext/extconf.rb

@@ -63,6 +63,7 @@ have_func("rb_io_descriptor")
 have_func("&rb_process_status_wait")
 have_func("rb_fiber_current")
 have_func("&rb_fiber_raise")
+have_func("epoll_pwait2")
 
 have_header('ruby/io/buffer.h')
 

data/ext/io/event/selector/epoll.c

@@ -570,38 +570,87 @@ VALUE IO_Event_Selector_EPoll_io_write_compatible(int argc, VALUE *argv, VALUE s
 #endif
 
 static
-int make_timeout(VALUE duration) {
+struct timespec * make_timeout(VALUE duration, struct timespec * storage) {
 	if (duration == Qnil) {
-		return -1;
+		return NULL;
 	}
 	
 	if (FIXNUM_P(duration)) {
-		return NUM2LONG(duration) * 1000L;
+		storage->tv_sec = NUM2TIMET(duration);
+		storage->tv_nsec = 0;
+		
+		return storage;
 	}
 	
 	else if (RB_FLOAT_TYPE_P(duration)) {
 		double value = RFLOAT_VALUE(duration);
+		time_t seconds = value;
 		
-		return value * 1000;
+		storage->tv_sec = seconds;
+		storage->tv_nsec = (value - seconds) * 1000000000L;
+		
+		return storage;
 	}
 	
 	rb_raise(rb_eRuntimeError, "unable to convert timeout");
 }
 
+static
+int timeout_nonblocking(struct timespec * timespec) {
+	return timespec && timespec->tv_sec == 0 && timespec->tv_nsec == 0;
+}
+
 struct select_arguments {
 	struct IO_Event_Selector_EPoll *data;
 	
 	int count;
 	struct epoll_event events[EPOLL_MAX_EVENTS];
-	
-	int timeout;
+
+	struct timespec * timeout;
+	struct timespec storage;
 };
 
+static int make_timeout_ms(struct timespec * timeout) {
+	if (timeout == NULL) {
+		return -1;
+	}
+	
+	if (timeout_nonblocking(timeout)) {
+		return 0;
+	}
+	
+	return (timeout->tv_sec * 1000) + (timeout->tv_nsec / 1000000);
+}
+
+static
+int enosys_error(int result) {
+	if (result == -1) {
+		return errno == ENOSYS;
+	}
+	
+	return 0;
+}
+
 static
 void * select_internal(void *_arguments) {
 	struct select_arguments * arguments = (struct select_arguments *)_arguments;
 	
-	arguments->count = epoll_wait(arguments->data->descriptor, arguments->events, EPOLL_MAX_EVENTS, arguments->timeout);
+#if defined(HAVE_EPOLL_PWAIT2)
+	arguments->count = epoll_pwait2(arguments->data->descriptor, arguments->events, EPOLL_MAX_EVENTS, arguments->timeout, NULL);
+	
+	// Comment out the above line and enable the below lines to test ENOSYS code path.
+	// arguments->count = -1;
+	// errno = ENOSYS;
+	
+	if (!enosys_error(arguments->count)) {
+		return NULL;
+	}
+	else {
+		// Fall through and execute epoll_wait fallback.
+	}
+#endif
+	
+	arguments->count = epoll_wait(arguments->data->descriptor, arguments->events, EPOLL_MAX_EVENTS, make_timeout_ms(arguments->timeout));
 	
 	return NULL;
 }
@@ -642,9 +691,14 @@ VALUE IO_Event_Selector_EPoll_select(VALUE self, VALUE duration) {
 	
 	struct select_arguments arguments = {
 		.data = data,
-		.timeout = 0
+		.storage = {
+			.tv_sec = 0,
+			.tv_nsec = 0
+		},
 	};
-	
+
+	arguments.timeout = &arguments.storage;
+
 	// Process any currently pending events:
 	select_internal_with_gvl(&arguments);
 	
@@ -654,9 +708,9 @@ VALUE IO_Event_Selector_EPoll_select(VALUE self, VALUE duration) {
 	// 3. There are no items in the ready list,
 	// then we can perform a blocking select.
 	if (!ready && !arguments.count && !data->backend.ready) {
-		arguments.timeout = make_timeout(duration);
+		arguments.timeout = make_timeout(duration, &arguments.storage);
 		
-		if (arguments.timeout != 0) {
+		if (!timeout_nonblocking(arguments.timeout)) {
 			// Wait for events to occur
 			select_internal_without_gvl(&arguments);
 		}

data/ext/io/event/selector/selector.h

@@ -43,7 +43,7 @@ enum IO_Event {
 	IO_EVENT_HANGUP = 16
 };
 
-void Init_IO_Event_Selector();
+void Init_IO_Event_Selector(VALUE IO_Event_Selector);
 
 static inline int IO_Event_try_again(int error) {
 	return error == EAGAIN || error == EWOULDBLOCK;

data/lib/io/event/version.rb

@@ -5,6 +5,6 @@
 
 class IO
 	module Event
-		VERSION = "1.1.7"
+		VERSION = "1.2.1"
 	end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: io-event
 version: !ruby/object:Gem::Version
-  version: 1.1.7
+  version: 1.2.1
 platform: ruby
 authors:
 - Samuel Williams
@@ -41,7 +41,7 @@ cert_chain:
   Q2K9NVun/S785AP05vKkXZEFYxqG6EW012U4oLcFl5MySFajYXRYbuUpH6AY+HP8
   voD0MPg1DssDLKwXyt1eKD/+Fq0bFWhwVM/1XiAXL7lyYUyOq24KHgQ2Csg=
   -----END CERTIFICATE-----
-date: 2023-03-11 00:00:00.000000000 Z
+date: 2023-05-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bake
@@ -150,7 +150,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.6
+rubygems_version: 3.4.7
 signing_key:
 specification_version: 4
 summary: An event loop.