From 8879d72884396de4ae2ff41649de5f2d916d272b Mon Sep 17 00:00:00 2001 From: Milind Changire Date: Fri, 13 Apr 2018 10:53:46 +0530 Subject: [PATCH 257/260] rpc: rearm listener socket early Problem: On node reboot, when glusterd starts volumes, a setup with a large number of bricks might cause SYN Flooding and connections to be dropped if the connections are not accepted quickly enough. Solution: accept() the connection and rearm the listener socket early to receive more connection requests as soon as possible. mainline: > Reviewed-on: https://review.gluster.org/19833 > Change-Id: Ibed421e50284c3f7a8fcdb4de7ac86cf53d4b74e > fixes: bz#1564600 > Signed-off-by: Milind Changire Change-Id: I4bf45b8a12f738ed9b398246e22dfd29e6b1010c BUG: 1563804 Signed-off-by: Milind Changire Reviewed-on: https://code.engineering.redhat.com/gerrit/135513 Tested-by: RHGS Build Bot Reviewed-by: Raghavendra Gowdappa Reviewed-by: Sunil Kumar Heggodu Gopala Acharya --- rpc/rpc-transport/socket/src/socket.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/rpc/rpc-transport/socket/src/socket.c b/rpc/rpc-transport/socket/src/socket.c index a6be911..b98efdc 100644 --- a/rpc/rpc-transport/socket/src/socket.c +++ b/rpc/rpc-transport/socket/src/socket.c @@ -2746,6 +2746,9 @@ socket_server_event_handler (int fd, int idx, int gen, void *data, if (poll_in) { new_sock = accept (priv->sock, SA (&new_sockaddr), &addrlen); + if (ctx) + event_handled (ctx->event_pool, fd, idx, gen); + if (new_sock == -1) { gf_log (this->name, GF_LOG_WARNING, "accept on %d failed (%s)", @@ -2968,8 +2971,6 @@ socket_server_event_handler (int fd, int idx, int gen, void *data, } } out: - event_handled (ctx->event_pool, fd, idx, gen); - if (cname && (cname != this->ssl_name)) { GF_FREE(cname); } -- 1.8.3.1