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

docs: some functions (currently) require mpirun #12992

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jsquyres
Copy link
Member

Add a note to several MPI API man pages that they (currently) require being launched via mpirun / mpiexec, and that launching in other environments may result in undefined behavior.

@hppritcha
Copy link
Member

MPI_COMM_CREATE_FROM_GROUP also needs to be updated.

@jsquyres jsquyres force-pushed the pr/add-mpirun-only-notes branch from 2c47673 to 3be03c2 Compare December 23, 2024 18:09
Add a note to several MPI API man pages that they (currently) require
being launched via mpirun / mpiexec, and that launching in other
environments may result in undefined behavior.

Signed-off-by: Jeff Squyres <[email protected]>
@jsquyres jsquyres force-pushed the pr/add-mpirun-only-notes branch from 3be03c2 to 3e32715 Compare December 23, 2024 18:09
@jsquyres
Copy link
Member Author

MPI_COMM_CREATE_FROM_GROUP also needs to be updated.

Thanks! Updated.

@rhc54
Copy link
Contributor

rhc54 commented Dec 23, 2024

Not a function, but perhaps should point out that the FT feature also requires mpirun. And of course, anything that calls connect/accept somewhere in its code path is going to hit this limitation.

You'll also likely need to do a bit of an audit for things that call nextcid at some point - I'm not sure when the PMIx code down there gets executed, but you might hit it way down in the call stack.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants