Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Commit a6cf208

Browse files
committed
coll/libnbc: do not exchange data between roots in ompi_coll_libnbc_ireduce_scatter_inter()
this is now useless since the scatter is done via the local communicator (back-ported from commit open-mpi/ompi@1462450)
1 parent 5dbf964 commit a6cf208

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

ompi/mca/coll/libnbc/nbc_ireduce_scatter.c

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -243,16 +243,6 @@ int ompi_coll_libnbc_ireduce_scatter_inter (void* sendbuf, void* recvbuf, int *r
243243
tbuf = lbuf; lbuf = rbuf; rbuf = tbuf;
244244
}
245245

246-
/* exchange data with remote root for scatter phase (we *could* use the local communicator to do the scatter) */
247-
res = NBC_Sched_recv (rbuf, true, count, datatype, 0, schedule);
248-
if (NBC_OK != res) { free(handle->tmpbuf); printf("Error in NBC_Sched_recv() (%i)\n", res); return res; }
249-
250-
res = NBC_Sched_send (lbuf, true, count, datatype, 0, schedule);
251-
if (NBC_OK != res) { free(handle->tmpbuf); printf("Error in NBC_Sched_send() (%i)\n", res); return res; }
252-
253-
res = NBC_Sched_barrier(schedule);
254-
if (NBC_OK != res) { free(handle->tmpbuf); printf("Error in NBC_Sched_barrier() (%i)\n", res); return res; }
255-
256246
/* do the local scatterv with the local communicator */
257247
res = NBC_Sched_copy (lbuf, true, recvcounts[0], datatype, recvbuf, false,
258248
recvcounts[0], datatype, schedule);

0 commit comments

Comments
 (0)