Skip to content

ompi/oshmem/spml/ucx: Fix cleanup when disconnected: avoid double free #12327

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

iyastreb
Copy link
Contributor

@iyastreb iyastreb commented Feb 13, 2024

Signed-off-by: Ilia Yastrebov [email protected]
(cherry picked from commit 92b82f6)

This is the v5.0.x version of main PR #12297

@github-actions github-actions bot added this to the v5.0.2 milestone Feb 13, 2024
Copy link

Hello! The Git Commit Checker CI bot found a few problems with this PR:

d6daa90: ompi/oshmem/spml/ucx: Fix cleanup when disconnecte...

  • check_cherry_pick: does not include a cherry pick message (did you need to bot:notacherrypick?)

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@iyastreb iyastreb force-pushed the oshmem/mca/ucx/fix-double-cleanup-5.0.x branch from d6daa90 to 50f9256 Compare February 13, 2024 09:10
Copy link

Hello! The Git Commit Checker CI bot found a few problems with this PR:

50f9256: ompi/oshmem/spml/ucx: Fix cleanup when disconnecte...

  • check_cherry_pick: does not include a cherry pick message (did you need to bot:notacherrypick?)

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

1 similar comment
Copy link

Hello! The Git Commit Checker CI bot found a few problems with this PR:

50f9256: ompi/oshmem/spml/ucx: Fix cleanup when disconnecte...

  • check_cherry_pick: does not include a cherry pick message (did you need to bot:notacherrypick?)

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@iyastreb iyastreb force-pushed the oshmem/mca/ucx/fix-double-cleanup-5.0.x branch 3 times, most recently from 98c0035 to 37d50fc Compare February 13, 2024 13:49
@jsquyres jsquyres modified the milestones: v5.0.2, v5.0.3 Feb 13, 2024
Signed-off-by: Ilia Yastrebov <[email protected]>
(cherry picked from commit 92b82f6)
Signed-off-by: Ilia Yastrebov <[email protected]>
@iyastreb iyastreb force-pushed the oshmem/mca/ucx/fix-double-cleanup-5.0.x branch from 37d50fc to 07a52f7 Compare February 14, 2024 08:26
@janjust janjust merged commit 8b4e629 into open-mpi:v5.0.x Feb 15, 2024
@iyastreb iyastreb deleted the oshmem/mca/ucx/fix-double-cleanup-5.0.x branch February 15, 2024 17:34
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