Skip to content

Conversation

@EmmanuelBRELLE
Copy link
Contributor

MPI 5.0 specification (section 7.6) allows MPI_Intercomm_create peer_comm to be either an intracommunicator or any kind of communicator.
In the current implementation this communicator is only used by local root in PML send/recv communications.
Unless I missed something, I should be OK to release the intracommunicator constrain.

@github-actions
Copy link

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

6428260: [MPI_Intercomm_create] Peer_comm intercommunicator...

  • check_signed_off: does not contain a valid Signed-off-by line

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

@EmmanuelBRELLE EmmanuelBRELLE force-pushed the intercomm_as_peercomm_in_intercomm_create branch from 6428260 to 6ea5d76 Compare October 15, 2025 14:20
Copy link
Member

@bosilca bosilca left a comment

Choose a reason for hiding this comment

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

You are correct, the standard places no restrictions on the peer communicato for as long as the two leaders can communicate through it, which means being in different groups.

@jsquyres jsquyres merged commit d933b33 into open-mpi:main Oct 15, 2025
16 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