andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

glusterd: Disable ping-timer between glusterd and brick process

Browse Source 
When there are too many IO happening, brick process epoll thread
will be busy and fails to respond to the glusterd pick packet within
30sec.
Also epoll thread can be blocked by a big-lock.

Solution is to disable ping-timer by default and only enable where ever
required

Later when the epoll thread model changed and made lighter,
we need to revert back this change. http://review.gluster.com/3842 is
one such approach.

Change-Id: I7f80ad3eb00f7d9c4d4527305932f7cf4920e73f
BUG: 1097224
Signed-off-by: Vijaikumar M <vmallika@redhat.com>
Reviewed-on: http://review.gluster.org/7753
Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
This commit is contained in:
Vijaikumar M 2014-05-13 17:36:01 +05:30 committed by Vijay Bellur
parent 1dd80a2e77
commit c13c1fd5a4
6 changed files with 19 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
rpc/rpc-lib/src/rpc-clnt-ping.c
View File

@ -147,7 +147,7 @@ rpc_clnt_ping_cbk (struct rpc_req *req, struct iovec *iov, int count,
} else {
/* timer expired and transport bailed out */
gf_log (this->name, GF_LOG_WARNING,
"timer must have expired");
"socket disconnected");
}
conn->ping_started = 0;
@ -226,7 +226,8 @@ rpc_clnt_start_ping (void *rpc_ptr)
conn = &rpc->conn;
if (conn->ping_timeout == 0) {
gf_log (THIS->name, GF_LOG_INFO, "ping timeout is 0, returning");
gf_log (THIS->name, GF_LOG_DEBUG, "ping timeout is 0,"
" returning");
return;
}

1
rpc/rpc-lib/src/rpc-clnt-ping.h
View File

@ -14,5 +14,6 @@
#include "config.h"
#endif
#define RPC_DEFAULT_PING_TIMEOUT 30
void
rpc_clnt_start_ping (void *rpc_ptr);

8
rpc/rpc-lib/src/rpc-clnt.c
View File

@ -984,9 +984,11 @@ rpc_clnt_connection_init (struct rpc_clnt *clnt, glusterfs_ctx_t *ctx,
gf_log (name, GF_LOG_DEBUG,
"setting ping-timeout to %d", conn->ping_timeout);
} else {
gf_log (name, GF_LOG_INFO,
"defaulting ping-timeout to 30secs");
conn->ping_timeout = 30;
/*TODO: Once the epoll thread model is fixed,
change the default ping-timeout to 30sec */
gf_log (name, GF_LOG_DEBUG,
"disable ping-timeout");
conn->ping_timeout = 0;
}
trans = rpc_transport_load (ctx, options, name);

3
xlators/mgmt/glusterd/src/glusterd-volgen.c
View File

@ -2250,6 +2250,9 @@ volgen_graph_build_clients (volgen_graph_t *graph, glusterd_volinfo_t *volinfo,
"%s", brick->brick_id);
if (!xl)
goto out;
ret = xlator_set_option (xl, "ping-timeout", "42");
if (ret)
goto out;
ret = xlator_set_option (xl, "remote-host", brick->hostname);
if (ret)
goto out;

9
xlators/mgmt/glusterd/src/glusterd.c
View File

@ -40,6 +40,7 @@
#include "glusterd-locks.h"
#include "common-utils.h"
#include "run.h"
#include "rpc-clnt-ping.h"
#include "syncop.h"
@ -1623,10 +1624,10 @@ struct volume_options options[] = {
.description = "directory where the bricks for the snapshots will be created"
},
{ .key = {"ping-timeout"},
.type = GF_OPTION_TYPE_INT,
.min = 1,
.max = 100,
.default_value = "30",
.type = GF_OPTION_TYPE_TIME,
.min = 0,
.max = 300,
.default_value = TOSTRING(RPC_DEFAULT_PING_TIMEOUT),
},
{ .key = {NULL} },
};

2
xlators/protocol/client/src/client.c
View File

@ -2841,7 +2841,7 @@ struct volume_options options[] = {
},
{ .key = {"ping-timeout"},
.type = GF_OPTION_TYPE_TIME,
.min = 1,
.min = 0,
.max = 1013,
.default_value = "42",
.description = "Time duration for which the client waits to "