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

Emend the point-to-point progress rule 'on two processes' #498

Closed
Wee-Free-Scot opened this issue Apr 13, 2021 · 1 comment
Closed

Emend the point-to-point progress rule 'on two processes' #498

Wee-Free-Scot opened this issue Apr 13, 2021 · 1 comment
Assignees
Labels
chap-p2p Point to Point Communication Chapter Committee errata Errata items for the previous MPI Standard had reading Completed the formal proposal reading mpi-4.1 For inclusion in the MPI 4.1 standard passed final vote Passed the final formal vote wg-p2p Point-to-Point Working Group
Projects

Comments

@Wee-Free-Scot
Copy link

Wee-Free-Scot commented Apr 13, 2021

Problem

Reported by Jasper Larsson Träff.

The current expression of the point-to-point progress rule starts "If a pair of matching send and receives have been initiated on two processes, ..."

The "on two processes" is not required and is confusing. A send-to-self, receive-from-self pattern is specifically excluded by this expression of the progress rule but should not be. I do not believe it was ever the intent to exclude the progress guarantee for this communication pattern. This communication pattern is possible even for blocking operations with multi-threading.

Proposal

Clarify to include the communicate-with-self pattern in the progress guarantee.

Changes to the Text

§3.5 "Progress" 1st line (MPI-4.0-RC-FEB, p54, line 42)
Remove the words "on two processes".

The process(es) involved in the communication is/are already unambiguously specified by the "source" and "dest" parameters in the point-to-point MPI procedures. The word "matching" requires that the receiving process is specified as "dest" at the sender process and that the sender process is specified (or at least covered by, in the case of a wildcard) by the "source" at the sender process. There is no loss of intended specification detail; only the potentially conflicting superfluous phrase is removed.

Impact on Implementations

Additional clarity.

Impact on Users

Additional clarity.

References and Pull Requests

Fixed by PR: https://github.com/mpi-forum/mpi-standard/pull/589 (closed prematurely)
Fixed by PR: https://github.com/mpi-forum/mpi-standard/pull/605 (replacement targeting mpi-4.x branch)

@Wee-Free-Scot Wee-Free-Scot added not ready wg-p2p Point-to-Point Working Group mpi-4.1 For inclusion in the MPI 4.1 standard chap-p2p Point to Point Communication Chapter Committee labels Apr 13, 2021
@Wee-Free-Scot Wee-Free-Scot self-assigned this Apr 13, 2021
@wesbland wesbland added this to To Do in MPI 4.1 Jun 9, 2021
@Wee-Free-Scot Wee-Free-Scot moved this from To Do to In Progress in MPI 4.1 Jun 14, 2021
@Wee-Free-Scot Wee-Free-Scot added this to the September 2021 milestone Aug 18, 2021
@Wee-Free-Scot Wee-Free-Scot added the errata Errata items for the previous MPI Standard label Aug 18, 2021
@wesbland wesbland changed the title Emend the point-to-point progress rule "on two processes" Emend the point-to-point progress rule 'on two processes' Aug 25, 2021
@wesbland wesbland moved this from In Progress to Passed 2nd Vote in MPI 4.1 Sep 13, 2021
@wesbland wesbland added the passed final vote Passed the final formal vote label Sep 13, 2021
@wesbland
Copy link
Member

This passed an errata vote on 2021-09-10:

Yes No Abstain
28 0 2

@wesbland wesbland added mpi-4.0 had reading Completed the formal proposal reading and removed mpi-4.1 For inclusion in the MPI 4.1 standard labels Sep 13, 2021
@wgropp wgropp closed this as completed Sep 13, 2021
MPI 4.1 automation moved this from Passed 2nd Vote to Merged Sep 13, 2021
@Wee-Free-Scot Wee-Free-Scot added the mpi-4.1 For inclusion in the MPI 4.1 standard label Sep 7, 2023
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 errata Errata items for the previous MPI Standard had reading Completed the formal proposal reading mpi-4.1 For inclusion in the MPI 4.1 standard passed final vote Passed the final formal vote wg-p2p Point-to-Point Working Group
Projects
No open projects
MPI 4.1
Merged
Status: Done
Development

No branches or pull requests

2 participants