Skip to content

Commit 288d248

Browse files
committed
btl/ugni: fix bug when attempting unaligned get on aries
This commit fixes a programming error when using an aries nic. The documentation of ugni shows that only the local alignment restriction for get was lifted on aries. There is still a remote address alignment restriction. Signed-off-by: Nathan Hjelm <[email protected]> (cherry picked from open-mpi/ompi@cc96097) Signed-off-by: Nathan Hjelm <[email protected]>
1 parent e577259 commit 288d248

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

opal/mca/btl/ugni/btl_ugni_component.c

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -234,11 +234,8 @@ btl_ugni_component_register(void)
234234
/*
235235
* see def. of ALIGNMENT_MASK to figure this one out
236236
*/
237-
if (GNI_DEVICE_GEMINI == device_type) {
238-
mca_btl_ugni_module.super.btl_get_alignment = 4;
239-
} else {
240-
mca_btl_ugni_module.super.btl_get_alignment = 0;
241-
}
237+
/* both gemini and aries have a 4-byte alignment requirement on remote addresses */
238+
mca_btl_ugni_module.super.btl_get_alignment = 4;
242239

243240
/* threshold for put */
244241
mca_btl_ugni_module.super.btl_min_rdma_pipeline_size = 8 * 1024;

0 commit comments

Comments
 (0)