Skip to content

Conversation

@hppritcha
Copy link
Member

Squash compiler warnings now showing up in the
query methods for the mtls. Cast pointers to the different
mtl module specific types to the mca_base_module_t.

Also, fix up a missing extern in mtl_psm_types.h.
This was causing "multiple definition" errors when building
the mca_mtl_psm.so shared library.

@yburette
@regrant
@jladd-mlnx

Squash compiler warnings now showing up in the
query methods for the mtls.  Cast pointers to the different
mtl module specific types to the mca_base_module_t.

Also, fix up a missing extern in mtl_psm_types.h.
This was causing "multiple definition" errors when building
the mca_mtl_psm.so shared library.
@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/210/
Test PASSed.

@yburette
Copy link
Member

yburette commented Feb 5, 2015

It looks good to me. I verified for the OFI MTL. Shall we wait for the other MTL writers to confirm before merging?

Yohann

@hppritcha
Copy link
Member Author

The log from the mlnx jenkins looks good for mxm mtl. We'll wait on sanity check from @regrant .

@tkordenbrock
Copy link
Member

This looks good for the Portals4 MTL.

todd

On Thu, Feb 5, 2015 at 10:52 AM, Yohann Burette notifications@github.com
wrote:

It looks good to me. I verified for the OFI MTL. Shall we wait for the
other MTL writers to confirm before merging?

Yohann


Reply to this email directly or view it on GitHub
#378 (comment).

hppritcha added a commit that referenced this pull request Feb 6, 2015
@hppritcha hppritcha merged commit 45e7c7f into open-mpi:master Feb 6, 2015
@hppritcha hppritcha deleted the topic/mtl_query_cast_fix branch May 15, 2015 03:37
jsquyres pushed a commit to jsquyres/ompi that referenced this pull request Nov 10, 2015
…ry_xrc

configury: fix a typo in XRC support detection
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.

4 participants