Skip to content

Segv on NBC reduce with user-defined op #966

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
Closed
jsquyres opened this issue Oct 2, 2015 · 0 comments · Fixed by #967
Closed

Segv on NBC reduce with user-defined op #966

jsquyres opened this issue Oct 2, 2015 · 0 comments · Fixed by #967
Assignees
@hjelmn
Labels
bug Severity: blocker
Milestone
v1.10.1

Comments

@jsquyres
Copy link
Member

jsquyres commented Oct 2, 2015

As reported by Georg Geiser here: http://www.open-mpi.org/community/lists/users/2015/10/27769.php

@hjelmn graciously said he would fix.
@jsquyres jsquyres added this to the Open MPI v1.10.1 milestone Oct 2, 2015
hjelmn added a commit to hjelmn/ompi that referenced this issue Oct 2, 2015
@hjelmn
op: allow user operations in ompi_3buff_op_reduce
57d3b83 
This commit allows user operations to be used in the
ompi_3buff_op_reduce function. This fixes an issue identified in:

http://www.open-mpi.org/community/lists/devel/2014/04/14586.php

and

http://www.open-mpi.org/community/lists/users/2015/10/27769.php

The fix is to copy source1 into the target then call the user op
function with source2 and target.

Fixes open-mpi#966

Signed-off-by: Nathan Hjelm <[email protected]>
@hjelmn hjelmn mentioned this issue Oct 2, 2015
Merged
bosilca pushed a commit to bosilca/ompi that referenced this issue Oct 8, 2015
@hjelmn @bosilca
op: allow user operations in ompi_3buff_op_reduce
9903694 
This commit allows user operations to be used in the
ompi_3buff_op_reduce function. This fixes an issue identified in:

http://www.open-mpi.org/community/lists/devel/2014/04/14586.php

and

http://www.open-mpi.org/community/lists/users/2015/10/27769.php

The fix is to copy source1 into the target then call the user op
function with source2 and target.

Fixes open-mpi#966

Signed-off-by: Nathan Hjelm <[email protected]>
bosilca pushed a commit to bosilca/ompi that referenced this issue Oct 8, 2015
@hjelmn @bosilca
op: allow user operations in ompi_3buff_op_reduce
2924995 
This commit allows user operations to be used in the
ompi_3buff_op_reduce function. This fixes an issue identified in:

http://www.open-mpi.org/community/lists/devel/2014/04/14586.php

and

http://www.open-mpi.org/community/lists/users/2015/10/27769.php

The fix is to copy source1 into the target then call the user op
function with source2 and target.

Fixes open-mpi#966

Signed-off-by: Nathan Hjelm <[email protected]>
jsquyres pushed a commit to jsquyres/ompi that referenced this issue Nov 10, 2015
@hjelmn
op: allow user operations in ompi_3buff_op_reduce
e16b531 
This commit allows user operations to be used in the
ompi_3buff_op_reduce function. This fixes an issue identified in:

http://www.open-mpi.org/community/lists/devel/2014/04/14586.php

and

http://www.open-mpi.org/community/lists/users/2015/10/27769.php

The fix is to copy source1 into the target then call the user op
function with source2 and target.

Fixes open-mpi#966

(cherry picked from commit open-mpi/ompi@57d3b83)

Signed-off-by: Nathan Hjelm <[email protected]>
jsquyres pushed a commit to jsquyres/ompi that referenced this issue Aug 23, 2016
@hppritcha
Merge pull request open-mpi#966 from ggouaillardet/topic/v2.x/f08_att…
14953c8 
…r_bindings

mpi_f08: correctly implements MPI_{COMM,TYPE,WIN}_{DUP,NULL_{COPY,DEL…
jsquyres pushed a commit to jsquyres/ompi that referenced this issue Sep 19, 2016
@hjelmn
op: allow user operations in ompi_3buff_op_reduce
e1ceb4e 
This commit allows user operations to be used in the
ompi_3buff_op_reduce function. This fixes an issue identified in:

http://www.open-mpi.org/community/lists/devel/2014/04/14586.php

and

http://www.open-mpi.org/community/lists/users/2015/10/27769.php

The fix is to copy source1 into the target then call the user op
function with source2 and target.

Fixes open-mpi#966

(cherry picked from commit open-mpi/ompi@57d3b83)

Signed-off-by: Nathan Hjelm <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hjelmn hjelmn

Labels
bug Severity: blocker
Projects
None yet
Milestone
v1.10.1
Development

Successfully merging a pull request may close this issue.

op: allow user operations in ompi_3buff_op_reduce hjelmn/ompi
2 participants
@jsquyres @hjelmn