Skip to content

Commit 28dfa36

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]>
1 parent c19426a commit 28dfa36

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
@@ -275,11 +275,8 @@ btl_ugni_component_register(void)
275275
/*
276276
* see def. of ALIGNMENT_MASK to figure this one out
277277
*/
278-
if (GNI_DEVICE_GEMINI == device_type) {
279-
mca_btl_ugni_module.super.btl_get_alignment = 4;
280-
} else {
281-
mca_btl_ugni_module.super.btl_get_alignment = 0;
282-
}
278+
/* both gemini and aries have a 4-byte alignment requirement on remote addresses */
279+
mca_btl_ugni_module.super.btl_get_alignment = 4;
283280

284281
/* threshold for put */
285282
mca_btl_ugni_module.super.btl_min_rdma_pipeline_size = 8 * 1024;

0 commit comments

Comments
 (0)