Skip to content

v3.0: mpool/base: plug a memory leak #4583

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

ggouaillardet
Copy link
Contributor

set the key of all mpool_tree_item objects, so they can be retrieved
in mpool_base_free and then returned back to the
mca_mpool_base_tree_item_free_list free list.

Refs. #4567

Thanks Philip Blakely for the bug report.

Signed-off-by: Gilles Gouaillardet [email protected]

(cherry picked from commit 11e5f86)

@hppritcha
Copy link
Member

bot:ompi:retest

@hppritcha hppritcha added the NEWS label Dec 7, 2017
@hppritcha hppritcha self-requested a review December 7, 2017 03:58
Copy link
Member

@hppritcha hppritcha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add a NEWS update

set the key of all mpool_tree_item objects, so they can be retrieved
in mpool_base_free and then returned back to the
mca_mpool_base_tree_item_free_list free list.

Refs. open-mpi#4567

Thanks Philip Blakely for the bug report.

Signed-off-by: Gilles Gouaillardet <[email protected]>

(cherry picked from commit open-mpi/ompi@11e5f86)
Plug a memory leak in MPI_Mem_free().  Thanks to Philip Blakely for reporting.

Refs open-mpi/ompi@c1257cf

Signed-off-by: Gilles Gouaillardet <[email protected]>
Fix the tree spawn operation when the number of nodes is larger than the radix.  Thanks to Carlos Eduardo de Andrade for reporting.

Refs open-mpi/ompi@e445d47

Signed-off-by: Gilles Gouaillardet <[email protected]>
@ggouaillardet ggouaillardet force-pushed the topic/v3.0.x/mpool_base_alloc branch from cf1d7de to 61aa462 Compare December 9, 2017 01:20
@jsquyres jsquyres changed the title mpool/base: plug a memory leak v3.0: mpool/base: plug a memory leak Dec 12, 2017
@bwbarrett bwbarrett merged commit 2da02f6 into open-mpi:v3.0.x Dec 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants