Skip to content

Conversation

@abouteiller
Copy link
Member

@abouteiller abouteiller commented Jun 17, 2020

MPI-4 will introduce the concept of the 'initial error handler'. This is the error handler that is set before MPI is initialized and after MPI is finalized, and can be selected from mpiexec/SPAWN info key parameters.

This also adds MPI_ERROR_CLASS, and MPI_ERROR_STRING to the functions that can be called before/after MPI_INIT and are always thread safe.

More details from the MPI Forum site:
https://github.com/mpi-forum/mpi-standard/pull/50
mpi-forum/mpi-issues#102 (if you do not have access to the standard repo).

@abouteiller
Copy link
Member Author

Updated, ready for review.

Copy link
Member

@bosilca bosilca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor comment regarding the indentation. Once fixed this can be merged.

@abouteiller abouteiller force-pushed the mpi-next/init-errh branch from 9566ef0 to ec6fd4e Compare July 8, 2020 20:28
@abouteiller
Copy link
Member Author

bot:retest

MPI_INIT/FINALIZE

Signed-off-by: Aurélien Bouteiller <[email protected]>

make lazy initialization opal unlikely

Signed-off-by: Aurelien Bouteiller <[email protected]>
value from prun populated paremeters

Signed-off-by: Aurélien Bouteiller <[email protected]>

Allow errhandlers to invoke the initial error handler before MPI_INIT

Signed-off-by: Aurelien Bouteiller <[email protected]>

Indentation

Signed-off-by: Aurelien Bouteiller <[email protected]>
Signed-off-by: Aurélien Bouteiller <[email protected]>

update to the predefined initial error handler selection

Signed-off-by: Aurelien Bouteiller <[email protected]>
Signed-off-by: Aurélien Bouteiller <[email protected]>

Use the same env to transmit the initial error handler to spawnees

Signed-off-by: Aurelien Bouteiller <[email protected]>
@abouteiller abouteiller merged commit 816acbd into open-mpi:master Jul 21, 2020
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