Skip to content

Commit 2ad0f14

Browse files
committed
spml/ucx: PR fixes 2.0
Signed-off-by: Michal Shalev <mshalev.nvidia.com>
1 parent 6dc87d1 commit 2ad0f14

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

oshmem/mca/spml/ucx/spml_ucx.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -693,15 +693,15 @@ int mca_spml_ucx_add_procs(oshmem_group_t* group, size_t nprocs)
693693
}
694694

695695
indices = malloc(nprocs * sizeof(*indices));
696-
if (!indices) {
696+
if (NULL == indices) {
697697
goto error;
698698
}
699699

700700
for (i = 0; i < nprocs; i++) {
701701
indices[i] = i;
702702
}
703703

704-
srand((unsigned int)time(NULL));
704+
srand(time(NULL));
705705

706706
/* Get the EP connection requests for all the processes from modex */
707707
for (proc_index = nprocs - 1; proc_index >= 0; --proc_index) {
@@ -719,7 +719,7 @@ int mca_spml_ucx_add_procs(oshmem_group_t* group, size_t nprocs)
719719
err = ucp_ep_create(mca_spml_ucx_ctx_default.ucp_worker[0], &ep_params,
720720
&mca_spml_ucx_ctx_default.ucp_peers[indices[proc_index]].ucp_conn);
721721
if (UCS_OK != err) {
722-
SPML_UCX_ERROR("ucp_ep_create(proc=%d/%zu) failed: %s", proc_index, nprocs,
722+
SPML_UCX_ERROR("ucp_ep_create(proc=%d/%zu, index=%u) failed: %s", proc_index, nprocs, indices[proc_index],
723723
ucs_status_string(err));
724724
goto error2;
725725
}

0 commit comments

Comments
 (0)