Skip to content

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