Skip to content

Commit a46e5da

Browse files
authored
Merge pull request #6744 from brminich/topic/all2all_linear_sync_fix
COLL/BASE: Fix linear sync all2all
2 parents 85c3311 + 79006f4 commit a46e5da

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

ompi/mca/coll/base/coll_base_alltoall.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -399,22 +399,22 @@ int ompi_coll_base_alltoall_intra_linear_sync(const void *sbuf, int scount,
399399
prcv = (char *) rbuf;
400400
psnd = (char *) sbuf;
401401

402-
/* Post first batch or ireceive and isend requests */
402+
/* Post first batch of irecv and isend requests */
403403
for (nreqs = 0, nrreqs = 0, ri = (rank + 1) % size; nreqs < total_reqs;
404404
ri = (ri + 1) % size, ++nrreqs) {
405-
nreqs++;
406405
error = MCA_PML_CALL(irecv
407406
(prcv + (ptrdiff_t)ri * rext, rcount, rdtype, ri,
408407
MCA_COLL_BASE_TAG_ALLTOALL, comm, &reqs[nreqs]));
408+
nreqs++;
409409
if (MPI_SUCCESS != error) { line = __LINE__; goto error_hndl; }
410410
}
411411
for (nsreqs = 0, si = (rank + size - 1) % size; nreqs < 2 * total_reqs;
412-
si = (si + size - 1) % size, ++nsreqs) {
413-
nreqs++;
412+
si = (si + size - 1) % size, ++nsreqs) {
414413
error = MCA_PML_CALL(isend
415414
(psnd + (ptrdiff_t)si * sext, scount, sdtype, si,
416415
MCA_COLL_BASE_TAG_ALLTOALL,
417416
MCA_PML_BASE_SEND_STANDARD, comm, &reqs[nreqs]));
417+
nreqs++;
418418
if (MPI_SUCCESS != error) { line = __LINE__; goto error_hndl; }
419419
}
420420

0 commit comments

Comments
 (0)