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

mtl/ofi: Make sure the resources are managed by the provider. #771

Merged
merged 1 commit into from
Nov 18, 2015

Conversation

yburette
Copy link
Member

Merged from master

bot:milestone:v1.10.2
bot:label:bug

@ompiteam-bot ompiteam-bot added this to the v1.10.2 milestone Nov 17, 2015
@mellanox-github
Copy link

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

@yburette
Copy link
Member Author

@rhc54 @jsquyres @miked-mellanox
I looked at the console output. The build seems to be successful but there is a memory check error in oshmem.
Sorry guys, I'm not sure what this has to do with my fix to OFI MTL. Any idea?

@igor-ivanov
Copy link
Member

@yburette valgrind issue you obsreved in jenkins is common for all last PRs and probably relates pml/yalla. We are looking for a fix.

@yburette
Copy link
Member Author

@igor-ivanov OK, I'm glad I wasn't doing anything wrong. Thank you :-)

rhc54 pushed a commit that referenced this pull request Nov 18, 2015
mtl/ofi: Make sure the resources are managed by the provider.
@rhc54 rhc54 merged commit 82f8261 into open-mpi:v1.10 Nov 18, 2015
alinask pushed a commit to alinask/ompi-release that referenced this pull request Dec 10, 2015
opal/lifo: add missing opal_atomic_wmb and remove unnecessary opal_atomic_rmb
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