Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Conversation

@hjelmn
Copy link
Member

@hjelmn hjelmn commented Apr 14, 2016

This commit ensures the bml is always enabled whether or not it will
be used. This ensures that any available btls communicate their modex
so that they can be used for one-sided communication.

Signed-off-by: Nathan Hjelm hjelmn@lanl.gov

(cherry picked from open-mpi/ompi@c6b1981)

Signed-off-by: Nathan Hjelm hjelmn@lanl.gov

This commit ensures the bml is always enabled whether or not it will
be used. This ensures that any available btls communicate their modex
so that they can be used for one-sided communication.

Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>

(cherry picked from open-mpi/ompi@c6b1981)

Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
@hjelmn
Copy link
Member Author

hjelmn commented Apr 14, 2016

:bot🏷️bug
:bot:milestone:v2.0.0
:bot:reviewer: @hppritcha

This is needed to support using bmls alongside osc/rdma. This commit ensures that the btls contribute their modex.

@ompiteam-bot ompiteam-bot added this to the v2.0.0 milestone Apr 14, 2016
@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1515/ for details.

@hppritcha
Copy link
Member

discussed with @jsquyres and notice that there's not the appropriate cleanup in MPI_Finalize

@jsquyres
Copy link
Member

@hjelmn What about ompi_mpi_finalize()?

@hjelmn
Copy link
Member Author

hjelmn commented Apr 18, 2016

No extra cleanup needed. mca_base_framework_close (&ompi_bml_base_framework) is already part of finalize.

:bot:nolabel:pushed-back

@hppritcha hppritcha merged commit c110ae8 into open-mpi:v2.x Apr 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants