From 0018ac356e6d99d85768b222ec002c5a1b92b8a5 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Mon, 22 Dec 2014 00:59:36 +0000 Subject: reduce syscalls in recvmsg invocations Favor IO.select over IO#wait since the latter makes another ioctl syscall (which we'll make anyways for IO#nread). Having BasicSocket#recvmsg and recvmsg_nonblock detect the buffer size requires extra syscalls, so pass explict maxmesglen, flags, and maxcontrollen args to elide auto-detection since we already have the buffered amount from IO#nread. This cuts more syscalls from the "dtas-tl cat" invocation with larger tracklists. --- lib/dtas/unix_client.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'lib/dtas/unix_client.rb') diff --git a/lib/dtas/unix_client.rb b/lib/dtas/unix_client.rb index 40dedcc..7382ea6 100644 --- a/lib/dtas/unix_client.rb +++ b/lib/dtas/unix_client.rb @@ -38,9 +38,9 @@ class DTAS::UNIXClient # :nodoc: end def res_wait(timeout = nil) - @to_io.wait(timeout) + IO.select([@to_io], nil, nil, timeout) nr = @to_io.nread nr > 0 or raise EOFError, "unexpected EOF from server" - @to_io.recvmsg[0] + @to_io.recvmsg(nr, 0, 0)[0] end end -- cgit v1.2.3-24-ge0c7