-
Notifications
You must be signed in to change notification settings - Fork 898
MPI_T: provide a stub implementation of events #13086
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
Conversation
@jsquyres per your request at last weeks meeting. |
related to #8057 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general, I think this looks like a great first step -- the API is there, and is conformant (even though it doesn't actually do anything). Extensions to make this API more functional would be welcome after this PR.
Any chance we could get some man pages and/or a mention somewhere in the docs about MPI_T events?
i'll add man pages to this PR. |
@hppritcha I made a few suggestions to your first cut of the man page. |
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
12d6502
to
5622889
Compare
@jsquyres please review |
@jsquyres ping |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; squash and merge!
Just for the heckuvit, I pushed more to the See Also on MPI_T.3. |
Signed-off-by: Howard Pritchard <howardp@lanl.gov>
19ea3fa
to
d0037fc
Compare
Turns out I forgot to run the pcvs checker when adding MPI_T event stubs in PR open-mpi#13086 and missed a couple of events related functions. Also it looks like these were not included in PR open-mpi#8057. With this patch, the PCVS MPI API checker passes for MPI 4.0 standard. The PCVS MPI API checker is described here https://dl.acm.org/doi/abs/10.1145/3615318.3615329 Signed-off-by: Howard Pritchard <howardp@lanl.gov>
Turns out I forgot to run the pcvs checker when adding MPI_T event stubs in PR open-mpi#13086 and missed a couple of events related functions. Also it looks like these were not included in PR open-mpi#8057. With this patch, the PCVS MPI API checker passes for MPI 4.0 standard. The PCVS MPI API checker is described here https://dl.acm.org/doi/abs/10.1145/3615318.3615329 Signed-off-by: Howard Pritchard <howardp@lanl.gov>
man pages included in this PR.