Skip to content

Conversation

@ggouaillardet
Copy link
Contributor

No description provided.

abouteiller and others added 2 commits July 12, 2019 10:26
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>

Correctly bubble up errors in NBC collective operations

Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>

The error field of requests needs to be rearmed at start, not at create

Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>

(cherry picked from commit open-mpi/ompi@65660e5)
MPI standard states a user MPI_Op and/or user MPI_Datatype can be free'd
after a call to a non blocking collective and before the non-blocking
collective completes.
Retain user (only) MPI_Op and MPI_Datatype when the non blocking call is
invoked, and set a request callback so they are free'd when the MPI_Request
completes.

Thanks Thomas Ponweiser for reporting this

Fixes open-mpi#2151
Fixes open-mpi#1304

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>

(cherry picked from commit open-mpi/ompi@0fe756d)
@gpaulsen
Copy link
Member

Please remember to cherry-pick -x to add the linkage to the master commits when creating the release branch PRs.

@ggouaillardet
Copy link
Contributor Author

@gpaulsen I did cherry-pick -x (!)
When I issued this PR, the "master" PR was not yet merged (GitHub permissions issues), that is why I set the WIP label.
From what I can tell, the commit message were always correct.
(and now the master PR has been merged, so this one is ready for review)

@hppritcha hppritcha added the NEWS label Jul 16, 2019
@gpaulsen
Copy link
Member

@ggouaillardet - Ah thanks.

@gpaulsen gpaulsen merged commit 507fcc9 into open-mpi:v4.0.x Jul 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants