Skip to content
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

Backport pr12384 #12487

Merged
merged 4 commits into from
Apr 24, 2024
Merged

Backport pr12384 #12487

merged 4 commits into from
Apr 24, 2024

Conversation

wenduwan
Copy link
Contributor

@wenduwan wenduwan commented Apr 22, 2024

Backport #12384
There was a conflict in ompi/communicator/ft/comm_ft.c that I had to manually resolve.

For intercommunicators, passing MPI_IN_PLACE to allreduce is invalid.

Signed-off-by: Lisandro Dalcin <[email protected]>
(cherry picked from commit cba47b6)
Co-authored-by: Aurelien Bouteiller <[email protected]>
Signed-off-by: Lisandro Dalcin <[email protected]>
(cherry picked from commit fdc618c)
Signed-off-by: Lisandro Dalcin <[email protected]>
(cherry picked from commit b01e156)
@dalcinl
Copy link
Contributor

dalcinl commented Apr 22, 2024

The ULFM tests are currently being skip with v5.0.x. The following manual CI execution should run these tests:
https://github.com/mpi4py/mpi4py-testing/actions/runs/8790618519/job/24123009162
This is using mpi4py/mpi4py@9a0e2f3, I'll merge that commit into the master branch once this PR is merged.

@dalcinl
Copy link
Contributor

dalcinl commented Apr 23, 2024

@wenduwan Can you also add the fixes from #12480 in here?

`ompi_comm_request_return(req)` releases the internal `req->context`.
Therefore, any `OBJ_RELEASE(context->xxx)` **MUST** be called **BEFORE**
`ompi_comm_request_return()`, otherwise it is a use-after-free.

Signed-off-by: Lisandro Dalcin <[email protected]>
(cherry picked from commit 573f35e)
@wenduwan wenduwan changed the title Backport pr12384 Backport pr12384 restart RTD Apr 24, 2024
@wenduwan wenduwan changed the title Backport pr12384 restart RTD Backport pr12384 Apr 24, 2024
@janjust janjust merged commit f6c460f into open-mpi:v5.0.x Apr 24, 2024
32 checks passed
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