Skip to content

v5: MPI 4: Add MPI_COMM_TYPE_HW_UNGUIDED and MPI_COMM_TYPE_HW_GUIDED #10703

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 5 commits into from
Sep 13, 2022

Conversation

jjhursey
Copy link
Member

No description provided.

@jjhursey jjhursey changed the title V5 hw guided v5: MPI 4: Add MPI_COMM_TYPE_HW_UNGUIDED and MPI_COMM_TYPE_HW_GUIDED Aug 23, 2022
@jjhursey
Copy link
Member Author

This cherry-picks:

@dalcinl
Copy link
Contributor

dalcinl commented Aug 23, 2022

@jjhursey Just FYI, all is good on mpi4py's side [link]

@jjhursey jjhursey force-pushed the v5-hw-guided branch 2 times, most recently from 16e410d to 9b95f39 Compare September 7, 2022 14:07
@jjhursey
Copy link
Member Author

jjhursey commented Sep 7, 2022

The main PR #10702 has been merged. I updated this PR with one additional commit from that PR. It is ready for review.

@jjhursey jjhursey marked this pull request as ready for review September 7, 2022 14:08
@jjhursey
Copy link
Member Author

jjhursey commented Sep 7, 2022

FYI: I just pushed on a missing help message PR #10778 from main

 * `MPI_COMM_TYPE_HW_GUIDED` supports all of the existing `OMPI_COMM_TYPE_`
   options.
 * `MPI_COMM_TYPE_HW_UNGUIDED` is recognized, but not supported so it returns
   `MPI_COMM_NULL` indidicating that the MPI library cannot split the
   communicator any further.

Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 19e24fa)
 * The wrong split type was being passed to the
   `ompi_comm_split_type_get_part` function. It was passing the
   "original" (`MPI_COMM_TYPE_HW_GUIDED`) split type and not the
   "converted" (e.g., `MPI_COMM_TYPE_SHARED`) split type. This resulted
   in an error indicating that the split type was invalid.

Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 6f1c128)
Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 349cb75)
Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 6ddcc58)
Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 2b70dcc)
@awlauria awlauria merged commit da2a901 into open-mpi:v5.0.x Sep 13, 2022
@jjhursey jjhursey deleted the v5-hw-guided branch September 13, 2022 20:44
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