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

do we need a sessions world equivalent of MPI Abort #11

Open
hppritcha opened this issue Apr 22, 2019 · 3 comments
Open

do we need a sessions world equivalent of MPI Abort #11

hppritcha opened this issue Apr 22, 2019 · 3 comments

Comments

@hppritcha
Copy link
Contributor

at the WG meeting today Dan described the usefulness of a MPI_Sessions_abort like function.

@dholmes-epcc-ed-ac-uk
Copy link

Previously I was describing how we might want to abort "all connected processes" for a session. However, we have not found a good way to define that for MPI processes that have references to multiple sessions.

Note: the error handler MPI_ERRORS_ABORT now acts only on the local MPI process (see issue #12).

By symmetry, should this mythical MPI_Session_Abort function act only on the local MPI process?
That would make it equivalent to MPI_ABORT(MPI_COMM_SELF), except that we don't have MPI_COMM_SELF - so we'd have to get the user to create one by creating a comm from the "mpi://self" process set. Thus, it could be a useful short-hand for:

int MPI_Session_abort(MPI_Session session, int errorCode) {
    MPI_Group myTempGroup;
    MPI_Comm mySelfComm;
    MPI_Group_from_pset(session, "mpi://self", myTempGroup);
    MPI_Comm_create_from_group(myTempGroup, mySelfComm);
    MPI_Abort(mySelfComm, errorCode);
    return errorCode;
}

Suggestion: we should drop this - at least for now.

@hppritcha
Copy link
Contributor Author

hppritcha commented Jul 25, 2022

Discussed in the Sessions WG 7/25/22 meeting. Dan recalled discussion of scope: group, universe, session (connected process). We now have a definition of all connected processes for sessions so this functionality may be necessary for FT/Sessions. Drop if FT WG wants to have a session scope for error handling otherwise we'll drop.

@hppritcha
Copy link
Contributor Author

put the mpi-5 label back on per discussion at the 08/01/22 sessions wg meeting

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

No branches or pull requests

2 participants