Replace MPI_Init to MPI_Init_thread when needed

kokkos / kokkos-comm

Experimental MPI Wrapper for Kokkos

https://kokkos.org/kokkos-comm/

Other

12 stars 9 forks source link

Replace MPI_Init to MPI_Init_thread when needed #93

Closed hmt23 closed 1 month ago

cwpearson commented 1 month ago

I think we're currently okay, but are we going to run into a situation where folks who just call MPI_Init and then use us are going to have problems?

dssgabriel commented 1 month ago

I think this should go in the documentation regarding initialization. We can change our unit tests to match what is the recommended MPI initialization (either MPI_Init_thread w/ MPI_THREAD_MULTIPLE, or MPI Sessions).

I am opening an issue for docs on how to initialize KokkosComm applications.