Skip to content

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

Merged
merged 2 commits into from
Mar 4, 2025

Conversation

hppritcha
Copy link
Member

@hppritcha hppritcha commented Feb 7, 2025

man pages included in this PR.

@hppritcha
Copy link
Member Author

@jsquyres per your request at last weeks meeting.

@hppritcha
Copy link
Member Author

related to #8057

@jsquyres jsquyres modified the milestones: v6, v6.0.0 Feb 12, 2025
jsquyres
jsquyres previously approved these changes Feb 12, 2025
Copy link
Member

@jsquyres jsquyres left a 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?

@hppritcha
Copy link
Member Author

i'll add man pages to this PR.

@jsquyres
Copy link
Member

@hppritcha I made a few suggestions to your first cut of the man page.

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
@hppritcha
Copy link
Member Author

@jsquyres please review

@hppritcha
Copy link
Member Author

@jsquyres ping

jsquyres
jsquyres previously approved these changes Feb 28, 2025
Copy link
Member

@jsquyres jsquyres left a 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!

@jsquyres
Copy link
Member

jsquyres commented Mar 1, 2025

Just for the heckuvit, I pushed more to the See Also on MPI_T.3.

Signed-off-by: Howard Pritchard <howardp@lanl.gov>
@hppritcha hppritcha force-pushed the mpi_t_events_stub branch from 19ea3fa to d0037fc Compare March 1, 2025 19:39
@hppritcha hppritcha merged commit cbb200d into open-mpi:main Mar 4, 2025
15 checks passed
@kingshuk00 kingshuk00 mentioned this pull request Mar 10, 2025
@devreal devreal moved this to Done in MPI 4.1 compliance Mar 11, 2025
hppritcha added a commit to hppritcha/ompi that referenced this pull request Apr 15, 2025
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>
vogma pushed a commit to vogma/ompi that referenced this pull request Apr 22, 2025
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants