Skip to content

Fix NBC iBarrier for inter-communicators. #1210

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 14, 2015

Conversation

artpol84
Copy link
Contributor

Remove send of the extra message. This bug hase triggered on
MPICH/coll/nbicbarrier test. In this test a series of communicators
are created.
This extre-message was reseived after original communicator was destroyed
and queued into non_existing_communicator_pending. When new completely
unrelated communicator with the same id as original was created this message
was pushed into the frags_cant_match queue and caused seq numbers skew and hang.

Remove send of the extra message. This bug hase triggered on
MPICH/coll/nbicbarrier test. In this test a series of communicators
are created.
This extre-message was reseived after original communicator was destroyed
and queued into non_existing_communicator_pending. When new completely
unrelated communicator with the same id as original was created this message
was pushed into the frags_cant_match queue and caused seq numbers skew and hang.
@artpol84
Copy link
Contributor Author

@hjelmn please, review

@artpol84
Copy link
Contributor Author

:bot:retest

@jsquyres jsquyres added this to the v2.0.0 milestone Dec 14, 2015
@hppritcha
Copy link
Member

@artpol84 please start assigning milestone labels to issues

@hjelmn
Copy link
Member

hjelmn commented Dec 14, 2015

👍

PR to v2.x and v1.10

hjelmn added a commit that referenced this pull request Dec 14, 2015
Fix NBC iBarrier for inter-communicators.
@hjelmn hjelmn merged commit 9d65946 into open-mpi:master Dec 14, 2015
@artpol84
Copy link
Contributor Author

@hjelmn I've cleanly applied the patch to 2.x and port it to 1.10.

@artpol84 artpol84 deleted the icbarrier_fix branch December 16, 2015 05:36
jsquyres pushed a commit to jsquyres/ompi that referenced this pull request Sep 19, 2016
ompi/op: Provide a default value for type/flags
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.

4 participants