From 4c58aa8ec7d1ab0fb725f90498fdab35ac461483 Mon Sep 17 00:00:00 2001 From: usa Date: Fri, 14 Feb 2014 12:02:01 +0000 Subject: * ext/socket: revert r44943 because it causes errors on some linux platforms. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_3@44954 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ext/socket/basicsocket.c | 2 +- ext/socket/init.c | 4 ++-- ext/socket/rubysocket.h | 16 ---------------- ext/socket/udpsocket.c | 2 +- 4 files changed, 4 insertions(+), 20 deletions(-) (limited to 'ext') diff --git a/ext/socket/basicsocket.c b/ext/socket/basicsocket.c index 1530d677a3..6f942e3a55 100644 --- a/ext/socket/basicsocket.c +++ b/ext/socket/basicsocket.c @@ -566,7 +566,7 @@ rsock_bsock_send(int argc, VALUE *argv, VALUE sock) GetOpenFile(sock, fptr); arg.fd = fptr->fd; arg.flags = NUM2INT(flags); - while (rsock_maybe_fd_writable(arg.fd), + while (rb_thread_fd_writable(arg.fd), (n = (int)BLOCKING_REGION_FD(func, &arg)) < 0) { if (rb_io_wait_writable(arg.fd)) { continue; diff --git a/ext/socket/init.c b/ext/socket/init.c index 8e418230be..54452e4b99 100644 --- a/ext/socket/init.c +++ b/ext/socket/init.c @@ -129,7 +129,7 @@ rsock_s_recvfrom(VALUE sock, int argc, VALUE *argv, enum sock_recv_type from) RBASIC(str)->klass = 0; while (rb_io_check_closed(fptr), - rsock_maybe_wait_fd(arg.fd), + rb_thread_wait_fd(arg.fd), (slen = BLOCKING_REGION_FD(recvfrom_blocking, &arg)) < 0) { if (!rb_io_wait_readable(fptr->fd)) { rb_sys_fail("recvfrom(2)"); @@ -503,7 +503,7 @@ rsock_s_accept(VALUE klass, int fd, struct sockaddr *sockaddr, socklen_t *len) arg.sockaddr = sockaddr; arg.len = len; retry: - rsock_maybe_wait_fd(fd); + rb_thread_wait_fd(fd); fd2 = (int)BLOCKING_REGION_FD(accept_blocking, &arg); if (fd2 < 0) { switch (errno) { diff --git a/ext/socket/rubysocket.h b/ext/socket/rubysocket.h index 9a5f15c542..ab05270be6 100644 --- a/ext/socket/rubysocket.h +++ b/ext/socket/rubysocket.h @@ -306,20 +306,4 @@ void rsock_init_addrinfo(void); void rsock_init_sockopt(void); void rsock_init_socket_init(void); -/* - * It is safe on Linux to attempt using a socket without waiting on it in - * all cases. For some syscalls (e.g. accept/accept4), blocking on the - * syscall instead of relying on select/poll allows the kernel to use - * "wake-one" behavior and avoid the thundering herd problem. - * This is likely safe on all other *nix-like systems, so this whitelist - * can be expanded by interested parties. - */ -#if defined(__linux__) -static inline int rsock_maybe_fd_writable(int fd) { return 1; } -static inline void rsock_maybe_wait_fd(int fd) { } -#else /* some systems (mswin/mingw) need these. ref: r36946 */ -# define rsock_maybe_fd_writable(fd) rb_thread_fd_writable((fd)) -# define rsock_maybe_wait_fd(fd) rb_thread_wait_fd((fd)) -#endif - #endif diff --git a/ext/socket/udpsocket.c b/ext/socket/udpsocket.c index b66c45dee4..0ba4371f1a 100644 --- a/ext/socket/udpsocket.c +++ b/ext/socket/udpsocket.c @@ -176,7 +176,7 @@ udp_send(int argc, VALUE *argv, VALUE sock) retry: arg.to = res->ai_addr; arg.tolen = res->ai_addrlen; - rsock_maybe_fd_writable(arg.fd); + rb_thread_fd_writable(arg.fd); n = (int)BLOCKING_REGION_FD(rsock_sendto_blocking, &arg); if (n >= 0) { freeaddrinfo(res0); -- cgit v1.2.3