From 894a517a12411c4dfbdeda6b9e90df26f5855d2e Mon Sep 17 00:00:00 2001 From: Krishnan Parthasarathi Date: Fri, 22 May 2015 00:37:58 +0530 Subject: socket: throttle only connected transport BUG: 1224016 Change-Id: I2352c6661ae5aee8cf06e572ec0462b76901d46f Signed-off-by: Krishnan Parthasarathi Reviewed-on: http://review.gluster.org/10887 Reviewed-by: Raghavendra G Tested-by: Raghavendra G Tested-by: NetBSD Build System Tested-by: Gluster Build System --- rpc/rpc-transport/socket/src/socket.c | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) (limited to 'rpc') diff --git a/rpc/rpc-transport/socket/src/socket.c b/rpc/rpc-transport/socket/src/socket.c index ece2bc8379a..e6ef7d2c220 100644 --- a/rpc/rpc-transport/socket/src/socket.c +++ b/rpc/rpc-transport/socket/src/socket.c @@ -3567,8 +3567,21 @@ socket_throttle (rpc_transport_t *this, gf_boolean_t onoff) will never read() any more data until throttling is turned off. */ - priv->idx = event_select_on (this->ctx->event_pool, priv->sock, - priv->idx, (int) !onoff, -1); + pthread_mutex_lock (&priv->lock); + { + + /* Throttling is useless on a disconnected transport. In fact, + * it's dangerous since priv->idx and priv->sock are set to -1 + * on a disconnected transport, which breaks epoll's event to + * registered fd mapping. */ + + if (priv->connected == 1) + priv->idx = event_select_on (this->ctx->event_pool, + priv->sock, + priv->idx, (int) !onoff, + -1); + } + pthread_mutex_unlock (&priv->lock); return 0; } -- cgit