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

mpit: optionally disable MPI_T usage in OSHMPI #112

Merged
merged 3 commits into from
May 11, 2021

Conversation

minsii
Copy link
Collaborator

@minsii minsii commented May 7, 2021

Add an environment variable to optionally disable MPI_T usage in OSHMPI.

Reason to add:
#105 reports a failure from FujitsuMPI occurred when calling MPI_T_cvar_write at OSHMPI init. It looks like FujitsuMPI does not return a proper error at the call to MPI_T_cvar_get_index. MPI_T_ERR_INVALID_NAME is expected when an MPI implementation specific CVAR does not exist.

This PR provides a workaround from OSHMPI. More checks need to be done with FujitsuMPI

@minsii
Copy link
Collaborator Author

minsii commented May 7, 2021

test:oshmpi/mpich/builtin
test:oshmpi/ompi/builtin

@minsii minsii added the TODO-dev label May 7, 2021
@minsii minsii force-pushed the pr/disable-mpit branch 2 times, most recently from 2cbd990 to 00abdea Compare May 11, 2021 21:30
@minsii
Copy link
Collaborator Author

minsii commented May 11, 2021

test:oshmpi/mpich/builtin
test:oshmpi/ompi/builtin

1 similar comment
@minsii
Copy link
Collaborator Author

minsii commented May 11, 2021

test:oshmpi/mpich/builtin
test:oshmpi/ompi/builtin

@minsii minsii force-pushed the pr/disable-mpit branch from 00abdea to 0f53b3b Compare May 11, 2021 22:34
@minsii minsii merged commit 41ad4c1 into pmodels:main May 11, 2021
@minsii minsii deleted the pr/disable-mpit branch May 11, 2021 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant