From 21fdd616bd4784e4e3571294ba39f00b24a25806 Mon Sep 17 00:00:00 2001 From: Chris Horn Date: Mon, 1 Nov 2021 15:06:31 -0500 Subject: [PATCH 1/1] LU-15186 o2iblnd: Default map_on_demand to 1 On kernels that provide global MR we default to using that exclusively even if FMR/FastReg is available. This causes an interop issue if the active side of a connection request has a higher fragment count than the passive side because FMR/FastReg may be needed to map the higher fragment count. We should change the default map_on_demand to 1 so that FMR/FastReg is used by default. map_on)demand can still be set to 0 if needed. Test-Parameters: trivial Signed-off-by: Chris Horn Change-Id: I76010a905f151efbb0b109ae6f5fba6fb7ce1956 Reviewed-on: https://review.whamcloud.com/45431 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Alexey Lyashkov Reviewed-by: Andriy Skulysh Reviewed-by: Oleg Drokin --- lnet/klnds/o2iblnd/o2iblnd_modparams.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/lnet/klnds/o2iblnd/o2iblnd_modparams.c b/lnet/klnds/o2iblnd/o2iblnd_modparams.c index fda7a4b..95e7200 100644 --- a/lnet/klnds/o2iblnd/o2iblnd_modparams.c +++ b/lnet/klnds/o2iblnd/o2iblnd_modparams.c @@ -140,13 +140,11 @@ MODULE_PARM_DESC(use_fastreg_gaps, "Enable discontiguous fastreg fragment suppor * the behavior when transmit with GAPS verses contiguous. */ #ifdef HAVE_IB_GET_DMA_MR -#define IBLND_DEFAULT_MAP_ON_DEMAND 0 #define MOD_STR "map on demand" #else -#define IBLND_DEFAULT_MAP_ON_DEMAND 1 #define MOD_STR "map on demand (obsolete)" #endif -static int map_on_demand = IBLND_DEFAULT_MAP_ON_DEMAND; +static int map_on_demand = 1; module_param(map_on_demand, int, 0444); MODULE_PARM_DESC(map_on_demand, MOD_STR); -- 1.8.3.1