From c046fadefd1c98b1d8e99cc56fd9f5cf683ab60f Mon Sep 17 00:00:00 2001 From: James Shimek Date: Fri, 14 Jul 2017 16:48:09 -0500 Subject: [PATCH] LU-10185 gnilnd: change default credits It has been found that a credit value of 64 reduces likelihood of file io induced congestion without producing performance degradation when a many router single compute pattern occurs. Change default compute credits to 64 based on this finding. Test-parameters: trivial Signed-off-by: Chris Horn Change-Id: I0a1f46bdb2c327a7e6dc9eeb145fe1418d691da1 Reviewed-on: https://review.whamcloud.com/29896 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: James Shimek Reviewed-by: James Simmons Reviewed-by: Doug Oucharek Reviewed-by: Oleg Drokin --- lnet/klnds/gnilnd/gnilnd.h | 2 ++ lnet/klnds/gnilnd/gnilnd_modparams.c | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/lnet/klnds/gnilnd/gnilnd.h b/lnet/klnds/gnilnd/gnilnd.h index 9ab8056..6cf451c 100644 --- a/lnet/klnds/gnilnd/gnilnd.h +++ b/lnet/klnds/gnilnd/gnilnd.h @@ -126,11 +126,13 @@ #define GNILND_SCHED_NICE 0 /* default nice value for scheduler threads */ #define GNILND_COMPUTE 1 /* compute image */ #define GNILND_FAST_RECONNECT 1 /* Fast Reconnect option */ +#define GNILND_DEFAULT_CREDITS 64 /* Default number of simultaneous transmits */ #else #define GNILND_FMABLK 1024 /* default number of mboxes per fmablk */ #define GNILND_SCHED_NICE -20 /* default nice value for scheduler threads */ #define GNILND_COMPUTE 0 /* service image */ #define GNILND_FAST_RECONNECT 0 /* Fast Reconnect option */ +#define GNILND_DEFAULT_CREDITS 256 /* Default number of simultaneous transmits */ #endif /* EXTRA_BITS are there to allow us to hide NOOP/CLOSE and anything else out of band */ diff --git a/lnet/klnds/gnilnd/gnilnd_modparams.c b/lnet/klnds/gnilnd/gnilnd_modparams.c index 3f042fa..bdf2b97 100644 --- a/lnet/klnds/gnilnd/gnilnd_modparams.c +++ b/lnet/klnds/gnilnd/gnilnd_modparams.c @@ -25,7 +25,7 @@ #include "gnilnd.h" -static int credits = 256; +static int credits = GNILND_DEFAULT_CREDITS; module_param(credits, int, 0444); MODULE_PARM_DESC(credits, "# concurrent sends"); -- 1.8.3.1