From 748e993d1f30197c533933ddae889b317ccd00d3 Mon Sep 17 00:00:00 2001 From: Milind Changire Date: Tue, 27 Nov 2018 11:43:04 +0530 Subject: rpc: bump up server.event-threads Problem: A single event-thread causes performance issues in the system. Solution: Bump up event-threads to 2 to make the system more performant. This helps in making the system more responsive and helps avoid the ping-timer-expiry problem as well. However, setting the event-threads to 2 is not the only thing required to avoid ping-timer-expiry issues. Change-Id: Idb0fd49e078db3bd5085dd083b0cdc77b59ddb00 fixes: bz#1653277 Signed-off-by: Milind Changire --- libglusterfs/src/glusterfs.h | 2 +- xlators/nfs/server/src/nfs.c | 2 +- xlators/protocol/server/src/server.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libglusterfs/src/glusterfs.h b/libglusterfs/src/glusterfs.h index 4d5bac322fd..305635fe078 100644 --- a/libglusterfs/src/glusterfs.h +++ b/libglusterfs/src/glusterfs.h @@ -270,7 +270,7 @@ enum gf_internal_fop_indicator { #define GLUSTERFS_RPC_REPLY_SIZE 24 -#define STARTING_EVENT_THREADS 1 +#define STARTING_EVENT_THREADS 2 #define DEFAULT_VAR_RUN_DIRECTORY DATADIR "/run/gluster" #define DEFAULT_GLUSTERFSD_MISC_DIRETORY DATADIR "/lib/misc/glusterfsd" diff --git a/xlators/nfs/server/src/nfs.c b/xlators/nfs/server/src/nfs.c index a20fab3ef79..6f55ed2f0e5 100644 --- a/xlators/nfs/server/src/nfs.c +++ b/xlators/nfs/server/src/nfs.c @@ -2030,7 +2030,7 @@ struct volume_options options[] = { .type = GF_OPTION_TYPE_SIZET, .min = 1, .max = 32, - .default_value = "1", + .default_value = "2", .description = "Specifies the number of event threads to execute in" "in parallel. Larger values would help process" " responses faster, depending on available processing" diff --git a/xlators/protocol/server/src/server.c b/xlators/protocol/server/src/server.c index 81aa72fbbc0..737e8e0907c 100644 --- a/xlators/protocol/server/src/server.c +++ b/xlators/protocol/server/src/server.c @@ -1840,7 +1840,7 @@ struct volume_options server_options[] = { .type = GF_OPTION_TYPE_INT, .min = 1, .max = 1024, - .default_value = "1", + .default_value = "2", .description = "Specifies the number of event threads to execute " "in parallel. Larger values would help process" " responses faster, depending on available processing" -- cgit