Skip to content

Conversation

@hjelmn
Copy link
Member

@hjelmn hjelmn commented Apr 21, 2015

This commit also fixes a problem with the lazy opening of topo
components. The topo framework incorrectly: 1) checked if the topo
framework was open by checking the length of the components list, and
2) called the framework open directly instead of using
mca_base_framework_open.

fixes #544

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

…/open

This commit also fixes a problem with the lazy opening of topo
components. The topo framework incorrectly: 1) checked if the topo
framework was open by checking the length of the components list, and
2) called the framework open directly instead of using
mca_base_framework_open.

fixes open-mpi#544

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

Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com/jenkins/job/gh-ompi-master-pr/468/
Test PASSed.

@adrianreber
Copy link
Member

Fixes my error as seen in #544

@hjelmn
Copy link
Member Author

hjelmn commented Apr 21, 2015

Great. Merging.

hjelmn added a commit that referenced this pull request Apr 21, 2015
mca/base: provide functions to determine if a framework is registered/open
@hjelmn hjelmn merged commit 47ef12b into open-mpi:master Apr 21, 2015
jsquyres pushed a commit to jsquyres/ompi that referenced this pull request Nov 10, 2015
@hjelmn hjelmn deleted the topo_fix branch May 23, 2016 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mpi_test_suite segfault since e794658

3 participants