io-event 0.3.2 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 66b02920cb810ad1ae4c312add97954e801c4f774a22cbaa278572a2985fdf2d
-  data.tar.gz: daa403e0c8fdd4d5849b6d984ba65613d9d621724abc41a1201598cfb94004b1
+  metadata.gz: 19b363c469b81cd49232e4e91a07c26cd0883984b8b54aa3fb34c5d861d5c577
+  data.tar.gz: 050f76cc397e3ec9e44b861c6469ba37d845d6a11fbc05f390e32a12a24061b5
 SHA512:
-  metadata.gz: d5ad6dc3b027d03c2cf97d96e89c37b01f80e7a8f2e2437c4add9958076a877f85ba01dd188829dec3f798b4608cc40ac8d9f755465a799bdbfe4d130e09a0eb
-  data.tar.gz: 2a312b4ea5fc8d785a1233c087010608ca82e3834b198fbd08bb66a0ef88b3746d2da04b457841a4a848a35eeee5dc31c274a7a8b2da11f1f87ebe24ce694f0b
+  metadata.gz: 704987de2d3aa4107621717ef84e9aad8d06c0c0ba062e4f63b0f0ef40f11e86e2971fec26e2e95f45494d2c60045d1991cdaf26c92bb7580d98e4eea0a94be8
+  data.tar.gz: 1d6da8310ee63c3e6d821d2dbc5c23affe02015ba44daa1e999a71de6a929d7c274379fab981495e47ea82157bda22e1dfe1e5ffc36ba380a36cd8ac9c65515f

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

@@ -365,7 +365,7 @@ VALUE io_read_loop(VALUE _arguments) {
 	
 	void *base;
 	size_t size;
-	rb_io_buffer_get_mutable(arguments->buffer, &base, &size);
+	rb_io_buffer_get_bytes_for_writing(arguments->buffer, &base, &size);
 	
 	size_t offset = 0;
 	size_t length = arguments->length;
@@ -437,7 +437,7 @@ VALUE io_write_loop(VALUE _arguments) {
 	
 	const void *base;
 	size_t size;
-	rb_io_buffer_get_immutable(arguments->buffer, &base, &size);
+	rb_io_buffer_get_bytes_for_reading(arguments->buffer, &base, &size);
 	
 	size_t offset = 0;
 	size_t length = arguments->length;
@@ -540,7 +540,11 @@ void select_internal_without_gvl(struct select_arguments *arguments) {
 	arguments->data->blocked = 0;
 	
 	if (arguments->count == -1) {
-		rb_sys_fail("select_internal_without_gvl:epoll_wait");
+		if (errno != EINTR) {
+			rb_sys_fail("select_internal_without_gvl:epoll_wait");
+		} else {
+			arguments->count = 0;
+		}
 	}
 }
 
@@ -549,7 +553,11 @@ void select_internal_with_gvl(struct select_arguments *arguments) {
 	select_internal((void *)arguments);
 	
 	if (arguments->count == -1) {
-		rb_sys_fail("select_internal_with_gvl:epoll_wait");
+		if (errno != EINTR) {
+			rb_sys_fail("select_internal_with_gvl:epoll_wait");
+		} else {
+			arguments->count = 0;
+		}
 	}
 }
 

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

@@ -396,7 +396,7 @@ VALUE io_read_loop(VALUE _arguments) {
 	
 	void *base;
 	size_t size;
-	rb_io_buffer_get_mutable(arguments->buffer, &base, &size);
+	rb_io_buffer_get_bytes_for_writing(arguments->buffer, &base, &size);
 	
 	size_t offset = 0;
 	size_t length = arguments->length;
@@ -478,7 +478,7 @@ VALUE io_write_loop(VALUE _arguments) {
 	
 	const void *base;
 	size_t size;
-	rb_io_buffer_get_immutable(arguments->buffer, &base, &size);
+	rb_io_buffer_get_bytes_for_reading(arguments->buffer, &base, &size);
 	
 	size_t offset = 0;
 	size_t length = arguments->length;
@@ -605,7 +605,11 @@ void select_internal_without_gvl(struct select_arguments *arguments) {
 	arguments->data->blocked = 0;
 	
 	if (arguments->count == -1) {
-		rb_sys_fail("select_internal_without_gvl:kevent");
+		if (errno != EINTR) {
+			rb_sys_fail("select_internal_without_gvl:kevent");
+		} else {
+			arguments->count = 0;
+		}
 	}
 }
 
@@ -614,7 +618,11 @@ void select_internal_with_gvl(struct select_arguments *arguments) {
 	select_internal((void *)arguments);
 	
 	if (arguments->count == -1) {
-		rb_sys_fail("select_internal_with_gvl:kevent");
+		if (errno != EINTR) {
+			rb_sys_fail("select_internal_with_gvl:kevent");
+		} else {
+			arguments->count = 0;
+		}
 	}
 }
 

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

@@ -27,7 +27,12 @@
 
 #include "pidfd.c"
 
-static const int DEBUG = 0;
+#include <linux/version.h>
+
+enum {
+	DEBUG = 0,
+	DEBUG_IO_READ = 0,
+};
 
 static VALUE IO_Event_Selector_URing = Qnil;
 
@@ -368,12 +373,28 @@ VALUE IO_Event_Selector_URing_io_wait(VALUE self, VALUE fiber, VALUE io, VALUE e
 
 #ifdef HAVE_RUBY_IO_BUFFER_H
 
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(5,16,0)
+static inline off_t io_seekable(int descriptor) {
+	return -1;
+}
+#else
+#warning Upgrade your kernel to 5.16! io_uring bugs prevent efficient io_read/io_write hooks.
+static inline off_t io_seekable(int descriptor)
+{
+	if (lseek(descriptor, 0, SEEK_CUR) == -1) {
+		return 0;
+	} else {
+		return -1;
+	}
+}
+#endif
+
 static int io_read(struct IO_Event_Selector_URing *data, VALUE fiber, int descriptor, char *buffer, size_t length) {
 	struct io_uring_sqe *sqe = io_get_sqe(data);
 
 	if (DEBUG) fprintf(stderr, "io_read:io_uring_prep_read(fiber=%p)\n", (void*)fiber);
 
-	io_uring_prep_read(sqe, descriptor, buffer, length, 0);
+	io_uring_prep_read(sqe, descriptor, buffer, length, io_seekable(descriptor));
 	io_uring_sqe_set_data(sqe, (void*)fiber);
 	io_uring_submit_now(data);
 	
@@ -391,14 +412,16 @@ VALUE IO_Event_Selector_URing_io_read(VALUE self, VALUE fiber, VALUE io, VALUE b
 	
 	void *base;
 	size_t size;
-	rb_io_buffer_get_mutable(buffer, &base, &size);
+	rb_io_buffer_get_bytes_for_writing(buffer, &base, &size);
 	
 	size_t offset = 0;
 	size_t length = NUM2SIZET(_length);
 	
 	while (true) {
 		size_t maximum_size = size - offset;
+		if (DEBUG_IO_READ) fprintf(stderr, "io_read(%d, +%ld, %ld)\n", descriptor, offset, maximum_size);
 		int result = io_read(data, fiber, descriptor, (char*)base+offset, maximum_size);
+		if (DEBUG_IO_READ) fprintf(stderr, "io_read(%d, +%ld, %ld) -> %d\n", descriptor, offset, maximum_size, result);
 		
 		if (result > 0) {
 			offset += result;
@@ -422,7 +445,7 @@ int io_write(struct IO_Event_Selector_URing *data, VALUE fiber, int descriptor,
 	
 	if (DEBUG) fprintf(stderr, "io_write:io_uring_prep_write(fiber=%p)\n", (void*)fiber);
 
-	io_uring_prep_write(sqe, descriptor, buffer, length, 0);
+	io_uring_prep_write(sqe, descriptor, buffer, length, io_seekable(descriptor));
 	io_uring_sqe_set_data(sqe, (void*)fiber);
 	io_uring_submit_pending(data);
 	
@@ -440,7 +463,7 @@ VALUE IO_Event_Selector_URing_io_write(VALUE self, VALUE fiber, VALUE io, VALUE
 	
 	const void *base;
 	size_t size;
-	rb_io_buffer_get_immutable(buffer, &base, &size);
+	rb_io_buffer_get_bytes_for_reading(buffer, &base, &size);
 	
 	size_t offset = 0;
 	size_t length = NUM2SIZET(_length);
@@ -553,6 +576,8 @@ int select_internal_without_gvl(struct select_arguments *arguments) {
 	
 	if (arguments->result == -ETIME) {
 		arguments->result = 0;
+	} else if (arguments->result == -EINTR) {
+		arguments->result = 0;
 	} else if (arguments->result < 0) {
 		rb_syserr_fail(-arguments->result, "select_internal_without_gvl:io_uring_wait_cqe_timeout");
 	} else {
@@ -563,8 +588,6 @@ int select_internal_without_gvl(struct select_arguments *arguments) {
 	return arguments->result;
 }
 
-// #define IO_EVENT_SELECTOR_URING_UDATA_INTERRUPT ((__u64) -2)
-
 static inline
 unsigned select_process_completions(struct io_uring *ring) {
 	unsigned completed = 0;
@@ -580,10 +603,6 @@ unsigned select_process_completions(struct io_uring *ring) {
 			continue;
 		}
 		
-		// if (cqe->user_data == IO_EVENT_SELECTOR_URING_UDATA_INTERRUPT) {
-		// 	io_uring_cq_advance(ring, 1);
-		// }
-		
 		VALUE fiber = (VALUE)cqe->user_data;
 		VALUE result = RB_INT2NUM(cqe->res);
 		

data/lib/io/event/selector/select.rb

@@ -158,7 +158,7 @@ module IO::Event
 						else
 							break unless result
 							
-							buffer.copy(result, offset)
+							buffer.set_string(result, offset)
 							
 							size = result.bytesize
 							offset += size
@@ -176,7 +176,7 @@ module IO::Event
 					while true
 						maximum_size = buffer.size - offset
 						
-						chunk = buffer.to_str(offset, maximum_size)
+						chunk = buffer.get_string(offset, maximum_size)
 						case result = blocking{io.write_nonblock(chunk, exception: false)}
 						when :wait_readable
 							if length > 0

data/lib/io/event/selector.rb

@@ -31,12 +31,10 @@ module IO::Event
 				end
 			end
 			
-			if self.const_defined?(:URing)
-				return URing
+			if self.const_defined?(:EPoll)
+				return EPoll
 			elsif self.const_defined?(:KQueue)
 				return KQueue
-			elsif self.const_defined?(:EPoll)
-				return EPoll
 			else
 				return Select
 			end

data/lib/io/event/version.rb

@@ -19,5 +19,5 @@
 # THE SOFTWARE.
 
 module IO::Event
-	VERSION = "0.3.2"
+	VERSION = "1.0.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: io-event
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 1.0.0
 platform: ruby
 authors:
 - Samuel Williams
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-18 00:00:00.000000000 Z
+date: 2021-12-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bake
@@ -113,14 +113,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.0.dev
+rubygems_version: 3.3.0
 signing_key:
 specification_version: 4
 summary: An event loop.