Skip to content

Commit

Permalink
Merge pull request #12888 from hppritcha/beef_up_sessions_smoke_test
Browse files Browse the repository at this point in the history
sessions: make sure to only finalize class
  • Loading branch information
hppritcha authored Oct 29, 2024
2 parents c393881 + 70e51fe commit afc970c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 12 deletions.
6 changes: 1 addition & 5 deletions examples/hello_sessions_c.c
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,10 @@ int main(int argc, char** argv) {
MPI_Info info;
MPI_Session s1, s2;

#if 0
/* need PR https://github.com/open-mpi/ompi/pull/12868 to be merged in
* before this can be uncommented.
*/
MPI_Info_create(&info);
#endif
MPI_Session_init(MPI_INFO_NULL, MPI_ERRORS_RETURN, &s1);
MPI_Session_finalize(&s1);
MPI_Session_init(MPI_INFO_NULL, MPI_ERRORS_RETURN, &s2);
MPI_Session_finalize(&s2);
MPI_Info_free(&info);
}
7 changes: 0 additions & 7 deletions ompi/instance/instance.c
Original file line number Diff line number Diff line change
Expand Up @@ -954,13 +954,6 @@ static int ompi_mpi_instance_finalize_common (void)

ompi_mpi_instance_release ();

if (0 == opal_initialized) {
/* if there is no MPI_T_init_thread that has been MPI_T_finalize'd,
* then be gentle to the app and release all the memory now (instead
* of the opal library destructor */
opal_class_finalize ();
}

return OMPI_SUCCESS;
}

Expand Down

0 comments on commit afc970c

Please sign in to comment.