Skip to content

Commit 971f697

Browse files
committed
Merge pull request open-mpi#1208 from hjelmn/v2.x_ugni
v2.x ugni bug fixes
2 parents cedc56e + dad3d47 commit 971f697

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

opal/mca/btl/ugni/btl_ugni_add_procs.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,6 @@ struct mca_btl_base_endpoint_t *mca_btl_ugni_get_ep (struct mca_btl_base_module_
244244
do {
245245
rc = opal_hash_table_get_value_uint64 (&ugni_module->id_to_endpoint, proc_id, (void **) &ep);
246246
if (OPAL_SUCCESS == rc) {
247-
OPAL_THREAD_UNLOCK(&ugni_module->endpoint_lock);
248247
break;
249248
}
250249

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)