Skip to content

Commit 8e317c9

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 9afdb7b commit 8e317c9

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
@@ -982,17 +982,6 @@ ompi_mtl_ofi_component_init(bool enable_progress_threads,
982982
goto error;
983983
}
984984

985-
/**
986-
* Unfortunately the attempt to implement FI_MR_SCALABLE in the GNI provider
987-
* doesn't work, at least not well. Since we're asking for the 1.5 libfabric
988-
* API now, we have to tell GNI we want to use Mr. Basic. Using FI_MR_BASIC
989-
* rather than FI_MR_VIRT_ADDR | FI_MR_ALLOCATED | FI_MR_PROV_KEY to stay
990-
* compatible with older libfabrics.
991-
*/
992-
if (!strncmp(prov->fabric_attr->prov_name,"gni",3)) {
993-
prov->domain_attr->mr_mode = FI_MR_BASIC;
994-
}
995-
996985
/**
997986
* Create the access domain, which is the physical or virtual network or
998987
* hardware port/collection of ports. Returns a domain object that can be

0 commit comments

Comments
 (0)