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

Revise restriction for MPI_T functions to return error classes #385

Closed
mahermanns opened this issue Dec 8, 2020 · 4 comments
Closed

Revise restriction for MPI_T functions to return error classes #385

mahermanns opened this issue Dec 8, 2020 · 4 comments
Assignees
Labels
chap-tools Tools Chapter Committee mpi-4.1 For inclusion in the MPI 4.1 standard wg-tools Tools Working Group
Projects

Comments

@mahermanns
Copy link
Member

mahermanns commented Dec 8, 2020

Problem

Before MPI 4.0 and the introduction of sessions, MPI_Error_class was not in the set of functions callable before MPI initialization. Therefore, in MPI 3.0 we introduced the restriction on functions of the Tool Information Interface (MPI_T) to return error classes instead of error codes.

The rationale for this restrictions---which is not correct for MPI 4.0 onwards---has been removed without replacement.

This issue shall provide discussion ground for a suitable clarification of the situation.

Proposal

tbd.

Changes to the Text

tbd.

Impact on Implementations

Depends on the solution we decide for.

Impact on Users

Depends on the solution we decide for.

References

@mahermanns mahermanns added wg-tools Tools Working Group mpi-4.1 For inclusion in the MPI 4.1 standard labels Dec 8, 2020
@mahermanns mahermanns self-assigned this Dec 8, 2020
@schulzm
Copy link

schulzm commented Dec 13, 2020

The non existence of error handling routines before Init was only part (or rather a side effect) of the use of error classes in MPI_T - the main goals were to have a more defined set of return values for tools to operate with and the non fatality in all cases, to not impact MPI operations of a tool return goes wrong. If I remember, we started with something completely new and then later said, might as well just call those error classes

@mahermanns
Copy link
Member Author

@schulzm Revisiting this after the Sep 2022 meeting, would you propose to leave the semantics as it is (apart from changes that are necessary to fulfill #542 and close this issue?

This would mean everything would stay the same for MPI_T function, always return something of MPI_T_ERR_* sorts. Correct?

@schulzm
Copy link

schulzm commented Nov 3, 2022

I think this is indeed resolved by 542 and we can probably close this one - the rationale should in any case be deleted, but this seems to have happened in 542 already

@mahermanns
Copy link
Member Author

Closing this issue as it will be resolved by the fixes for #542.

@wesbland wesbland closed this as not planned Won't fix, can't repro, duplicate, stale Nov 9, 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 mpi-4.1 For inclusion in the MPI 4.1 standard wg-tools Tools Working Group
Projects
No open projects
MPI 4.1
To Do
Status: Done
Development

No branches or pull requests

3 participants