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

ERRATA: Clarify behavior for invalid MPI_T binding arguments #160

Closed
mahermanns opened this issue Feb 4, 2020 · 5 comments
Closed

ERRATA: Clarify behavior for invalid MPI_T binding arguments #160

mahermanns opened this issue Feb 4, 2020 · 5 comments
Assignees
Labels
chap-tools Tools 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 passed first vote Passed the first formal vote scheduled reading Reading is scheduled for the next meeting wg-terms Semantic Terms Working Group wg-tools Tools Working Group
Milestone

Comments

@mahermanns
Copy link
Member

Problem

The Section 14.3.2 discussing binding of MPI_T variables currently does not state expected behavior when an invalid object handle is passed as binding argument.

Proposal

Explicitly state that passing invalid handles as binding arguments is erroneous.

Changes to the Text

Add explicit clarification that invalid handles are erroneous. Also add Advice to Users reminding the user that MPI_COMM_WORLD and MPI_COMM_SELF are invalid outside the INIT/FINALIZE scope.

Impact on Implementations

Behavior and potential implementation clarified.

Impact on Users

Expected arguments and potential behavior of implementation clarified.

References

PR: mpi-forum/mpi-standard/pull/161

@mahermanns mahermanns added the wg-tools Tools Working Group label Feb 4, 2020
@schulzm schulzm changed the title Clarify behavior for invalid MPI_T binding arguments ERRATA: Clarify behavior for invalid MPI_T binding arguments Feb 5, 2020
@mahermanns mahermanns added the errata Errata items for the previous MPI Standard label Feb 5, 2020
@schulzm schulzm added Chapter Committee Change Changes to be made by the respective chapter committee(s) errata Errata items for the previous MPI Standard and removed errata Errata items for the previous MPI Standard Chapter Committee Change Changes to be made by the respective chapter committee(s) labels May 13, 2020
@mahermanns mahermanns added the scheduled reading Reading is scheduled for the next meeting label May 14, 2020
@mahermanns mahermanns added mpi-4.1 For inclusion in the MPI 4.1 standard wg-terms Semantic Terms Working Group and removed scheduled reading Reading is scheduled for the next meeting labels Jul 15, 2020
@wesbland wesbland closed this as completed Oct 2, 2020
@wesbland
Copy link
Member

wesbland commented Oct 7, 2020

@mahermanns I think this might have been accidentally closed when mpi-forum/mpi-standard#269 was merged last week. This is still in progress, right? Should it be reopened?

@mahermanns
Copy link
Member Author

@wesbland Yes, this issue needs further discussion, which we pushed to 4.1.

@mahermanns mahermanns reopened this Jan 7, 2021
@mahermanns
Copy link
Member Author

Tools WG Dec 2, 2021: Propose text to allow any predefined handle to be passed as a binding argument at any time in the program execution.

@mahermanns mahermanns added the chap-tools Tools Chapter Committee label Sep 30, 2022
@mahermanns
Copy link
Member Author

Tools WG Discussion 2022-11-03: With the difficulties showing up on the current PR regarding link-time vs. compile-time constants, the working group decided to propose to only allow binding to valid handles while MPI is in an initialized state. This would actually be more consistent with the handling of non-predefined handles, and might make implementation for MPI libraries and tools simpler.

@mahermanns mahermanns added this to the December 2022 milestone Nov 15, 2022
@wesbland wesbland added the scheduled reading Reading is scheduled for the next meeting label Nov 17, 2022
@wesbland wesbland added the had reading Completed the formal proposal reading label Dec 5, 2022
@wesbland
Copy link
Member

wesbland commented Dec 7, 2022

This passed an errata vote on 2022-12-07.

Yes No Abstain
29 0 4

@wesbland wesbland added passed first vote Passed the first formal vote passed final vote Passed the final formal vote labels Dec 7, 2022
@wgropp wgropp closed this as completed Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chap-tools Tools 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 passed first vote Passed the first formal vote scheduled reading Reading is scheduled for the next meeting wg-terms Semantic Terms Working Group wg-tools Tools Working Group
Projects
No open projects
Status: Done
Development

No branches or pull requests

3 participants