Skip to content

Conversation

@hjelmn
Copy link
Member

@hjelmn hjelmn commented Jun 22, 2015

OPAL_ENABLE_MULTI_THREADS is always on. The correct value to check is
OMPI_ENABLE_THREAD_MULTIPLE.

Signed-off-by: Nathan Hjelm hjelmn@me.com

@hjelmn
Copy link
Member Author

hjelmn commented Jun 22, 2015

@jsquyres If the message queue fix in in either 1.10 or 1.8 this will need to be PR'd there as well.

@rhc54
Copy link
Contributor

rhc54 commented Jun 22, 2015

Crud - it is in both branches

Sent from my iPhone

On Jun 22, 2015, at 10:05 AM, Nathan Hjelm notifications@github.com wrote:

@jsquyres If the message queue fix in in either 1.10 or 1.8 this will need to be PR'd there as well.


Reply to this email directly or view it on GitHub.

@mellanox-github
Copy link

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

…tiple support

OPAL_ENABLE_MULTI_THREADS is always on. The correct value to check is
OMPI_ENABLE_THREAD_MULTIPLE.

Signed-off-by: Nathan Hjelm <hjelmn@me.com>
@hjelmn hjelmn force-pushed the fix_thread_multiple_check branch from 3e6f802 to 284dd6b Compare June 23, 2015 01:17
@hjelmn
Copy link
Member Author

hjelmn commented Jun 23, 2015

Fixed one more check.

@mellanox-github
Copy link

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

hjelmn added a commit that referenced this pull request Jun 23, 2015
pml/ob1: do not use OPAL_ENABLE_MULTI_THREADS to determine thread multiple support
@hjelmn hjelmn merged commit ac51acb into open-mpi:master Jun 23, 2015
@hjelmn hjelmn deleted the fix_thread_multiple_check 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.

3 participants