Skip to content

ofi/common: release opal memory base framework #10975

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
merged 1 commit into from
Nov 1, 2022

Conversation

hppritcha
Copy link
Member

when shutting down the memory monitoring infrastructure in ofi common.

Signed-off-by: Howard Pritchard [email protected]

when shutting down the memory monitoring infrastructure
in ofi common.

Signed-off-by: Howard Pritchard <[email protected]>
@hppritcha
Copy link
Member Author

@wckzhang could you please review? I'd like to get this into the 5.0.0 release.

@hppritcha
Copy link
Member Author

@wckzhang could you review when you have time? we need to get this into 5.0.0.
@gpaulsen

@hppritcha hppritcha removed the request for review from wckzhang November 1, 2022 15:23
@hppritcha hppritcha merged commit a4c2d16 into open-mpi:main Nov 1, 2022
@wckzhang
Copy link
Contributor

wckzhang commented Nov 1, 2022

Can you also add closing the base framework to the opal_common_ofi_export_memory_monitor error path?

@hppritcha
Copy link
Member Author

@wckzhang good catch yes I'll add that in a new PR.

hppritcha added a commit to hppritcha/ompi that referenced this pull request Nov 5, 2022
hppritcha added a commit to hppritcha/ompi that referenced this pull request Nov 9, 2022
hppritcha added a commit to hppritcha/ompi that referenced this pull request Nov 11, 2022
related to open-mpi#10975

Signed-off-by: Howard Pritchard <[email protected]>
(cherry picked from commit c9594e3)
yli137 pushed a commit to yli137/ompi that referenced this pull request Jan 10, 2024
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.

3 participants