Skip to content

Commit b097ff7

Browse files
committed
mtl/ofi: Remove special patch for gni provider
There should not be any Cray XC systems in production now - which is where the GNI provider nominally functioned. Meanwhile FI_MR_BASIC is deprecated since Libfabric 1.5 and will be droppped in future Libfabric 2.x versions. Remove this special handler for gni provider due to unnecessity and build warnings. Signed-off-by: Shi Jin <[email protected]> (cherry picked from commit 1c67d90)
1 parent 995bd30 commit b097ff7

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

ompi/mca/mtl/ofi/mtl_ofi_component.c

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -897,17 +897,6 @@ ompi_mtl_ofi_component_init(bool enable_progress_threads,
897897
goto error;
898898
}
899899

900-
/**
901-
* Unfortunately the attempt to implement FI_MR_SCALABLE in the GNI provider
902-
* doesn't work, at least not well. Since we're asking for the 1.5 libfabric
903-
* API now, we have to tell GNI we want to use Mr. Basic. Using FI_MR_BASIC
904-
* rather than FI_MR_VIRT_ADDR | FI_MR_ALLOCATED | FI_MR_PROV_KEY to stay
905-
* compatible with older libfabrics.
906-
*/
907-
if (!strncmp(prov->fabric_attr->prov_name,"gni",3)) {
908-
prov->domain_attr->mr_mode = FI_MR_BASIC;
909-
}
910-
911900
#if OPAL_CUDA_SUPPORT
912901
/**
913902
* Some providers do not require the use of the CUDA convertor

0 commit comments

Comments
 (0)