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

MPI_START / MPI_STARTALL semantics clarification #452

Open
tonyskjellum opened this issue Feb 24, 2021 · 1 comment
Open

MPI_START / MPI_STARTALL semantics clarification #452

tonyskjellum opened this issue Feb 24, 2021 · 1 comment
Assignees
Labels
chap-p2p Point to Point Communication Chapter Committee mpi-5 For inclusion in the MPI 5.0 standard

Comments

@tonyskjellum
Copy link

Problem

Although MPI_START and MPI_STARTALL take persistent requests by reference, the implementation has never been intended to change the handles (the bits in the request itself; the underlying internals are not at issue). Stating this fact is important, because the user has no expectation that handles change values. In some sense, MPI_START should have taken an IN argument for the request, but the API wasn't defined that way in MPI-1.0.

Proposal

Persistent requests don't change their value when used in START/STARTALL operations.

Changes to the Text

Text will be added to indicate that MPI_START and MPI_START_ALL receive persistent requests by reference, but may not
change the requests (const pointer, const array is implied).

Impact on Implementations

No known implementations do this unwanted behavior.

Impact on Users

No expected impact. The lack of this clarification could cause major impacts if a future implementation
chose to use this loophole.

References and Pull Requests

TBD.

@tonyskjellum tonyskjellum added the chap-p2p Point to Point Communication Chapter Committee label Feb 24, 2021
@tonyskjellum tonyskjellum self-assigned this Feb 24, 2021
@wesbland wesbland changed the title MPI-4.1: MPI_START / MPI_STARTALL semantics clarification MPI_START / MPI_STARTALL semantics clarification Feb 24, 2021
@wesbland wesbland added the mpi-4.1 For inclusion in the MPI 4.1 standard label Feb 24, 2021
@wesbland
Copy link
Member

I’m going to propose moving this to MPI 5.0. There’s more discussion to be had here, but it sounds like there’s nothing in progress. If someone objects and thinks we’ll be ready to read this soon, leave a comment and we can discuss bringing it back into MPI 4.1.

@wesbland wesbland added mpi-5 For inclusion in the MPI 5.0 standard and removed mpi-4.1 For inclusion in the MPI 4.1 standard labels Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chap-p2p Point to Point Communication Chapter Committee mpi-5 For inclusion in the MPI 5.0 standard
Projects
Status: To Do
Development

No branches or pull requests

2 participants