Skip to content

Fix NBC iGatherv for inter-communicators. #1216

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.

Sign up for GitHub

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

Jump to bottom
Merged
hjelmn merged 1 commit into from
Dec 14, 2015
Merged

Fix NBC iGatherv for inter-communicators. #1216

hjelmn merged 1 commit into from
Dec 14, 2015

Conversation

@artpol84
Copy link
Contributor

@artpol84 artpol84 commented Dec 14, 2015

We need to use remote size to form a schedule.

@artpol84
Fix NBC iGatherv for inter-communicators.
2d0919d 
We need to use remote size to form a schedule.
@artpol84
Copy link
Contributor Author

artpol84 commented Dec 14, 2015

@hjelmn, please review.

@artpol84
Copy link
Contributor Author

artpol84 commented Dec 14, 2015

Looks like with this PR and PR# #1210 all MPICH tests that we are testing except cancelrecv are OK.

@jsquyres jsquyres added this to the v2.0.0 milestone Dec 14, 2015
@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
@hjelmn
Merge pull request #1216 from artpol84/icgatherv_fix
4b3dac5 
Fix NBC iGatherv for inter-communicators.
@hjelmn hjelmn merged commit 4b3dac5 into open-mpi:master Dec 14, 2015
@artpol84
Copy link
Contributor Author

artpol84 commented Dec 15, 2015

@hjelmn

  • I've made a PR to 2.x
  • 1.10 looks fine with regards to this bug

@artpol84 artpol84 deleted the icgatherv_fix branch December 16, 2015 05:35
artpol84 added a commit to artpol84/ompi that referenced this pull request Dec 29, 2015
@artpol84
Fix Mellanox copyrights with respect to the following PRs:
2abb297 
* open-mpi#1184
* open-mpi#1188
* open-mpi#1197
* open-mpi#1202
* open-mpi#1210
* open-mpi#1216
* open-mpi#1236
* open-mpi#1237
* open-mpi#1248
* open-mpi#1260
* open-mpi#1264
@artpol84 artpol84 mentioned this pull request Dec 29, 2015
Merged
jsquyres pushed a commit to jsquyres/ompi that referenced this pull request Sep 19, 2016
Merge pull request open-mpi#1216 from ggouaillardet/topic/v1.10/nag_c…
d99bdc6 
…onfigury

Topic/v1.10/nag configury
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@hjelmn hjelmn

Labels

bug Severity: blocker

Projects

None yet

Milestone

v2.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@artpol84 @hjelmn @jsquyres