From 84b012e02a567f79dc3061cd860be982f95be048 Mon Sep 17 00:00:00 2001 From: normal Date: Wed, 6 May 2015 20:30:43 +0000 Subject: stdlib: use IO#wait_*able instead of IO.select when possible In case a process encounters high-numbered FDs, this allows consistent performance on systems with ppoll support. [ruby-core:35572] * ext/socket/lib/socket.rb (connect_nonblock): use IO#wait_writable * lib/drb/drb.rb (DRB::DRbTCPSocket#alive?): use IO#wait_readable * lib/webrick/httpserver.rb (run): ditto * lib/resolv.rb (request): ditto for single socket case [ruby-core:68943] [Feature #11081] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@50432 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- lib/drb/drb.rb | 3 ++- lib/resolv.rb | 7 ++++++- lib/webrick/httpserver.rb | 3 ++- 3 files changed, 10 insertions(+), 3 deletions(-) (limited to 'lib') diff --git a/lib/drb/drb.rb b/lib/drb/drb.rb index e7a8d20211..32e5bfb9e7 100644 --- a/lib/drb/drb.rb +++ b/lib/drb/drb.rb @@ -48,6 +48,7 @@ require 'socket' require 'thread' require 'fcntl' +require 'io/wait' require 'drb/eq' # @@ -1003,7 +1004,7 @@ module DRb # Check to see if this connection is alive. def alive? return false unless @socket - if IO.select([@socket], nil, nil, 0) + if @socket.to_io.wait_readable(0) close return false end diff --git a/lib/resolv.rb b/lib/resolv.rb index e67dff9704..9083bd0b53 100644 --- a/lib/resolv.rb +++ b/lib/resolv.rb @@ -1,6 +1,7 @@ require 'socket' require 'timeout' require 'thread' +require 'io/wait' begin require 'securerandom' @@ -680,7 +681,11 @@ class Resolv if timeout <= 0 raise ResolvTimeout end - select_result = IO.select(@socks, nil, nil, timeout) + if @socks.size == 1 + select_result = @socks[0].wait_readable(timeout) ? [ @socks ] : nil + else + select_result = IO.select(@socks, nil, nil, timeout) + end if !select_result after_select = Time.now next if after_select < timelimit diff --git a/lib/webrick/httpserver.rb b/lib/webrick/httpserver.rb index 0618489c53..96bd3fa548 100644 --- a/lib/webrick/httpserver.rb +++ b/lib/webrick/httpserver.rb @@ -8,6 +8,7 @@ # # $IPR: httpserver.rb,v 1.63 2002/10/01 17:16:32 gotoyuzo Exp $ +require 'io/wait' require 'webrick/server' require 'webrick/httputils' require 'webrick/httpstatus' @@ -72,7 +73,7 @@ module WEBrick begin timeout = @config[:RequestTimeout] while timeout > 0 - break if IO.select([sock], nil, nil, 0.5) + break if sock.to_io.wait_readable(0.5) break if @status != :Running timeout -= 0.5 end -- cgit v1.2.3