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

v11: Update Intel MPI flags #632

Closed
wants to merge 4 commits into from

Conversation

mathomp4
Copy link
Member

Testing by @wmputman and @pchakraborty on SLES15 at NCCS show that using:

setenv I_MPI_FABRICS shm:ofi
setenv I_MPI_OFI_PROVIDER psm3

is all that is needed. So this PR changes the flags on SLES15 to reflect this. For now it should keep the SLES12 flags as they were before.

This should be used only in concert with GEOS-ESM/GEOSgcm#804 which moves GEOSgcm to Intel MPI 2021.13. It might work with older Intel MPI...but I've only tested with that version.

NOTE: Once Cascade Lake moves to SLES15, this might need to be revisited as perhaps these are Milan-only flags.

@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Jul 24, 2024
@mathomp4 mathomp4 self-assigned this Jul 24, 2024
@mathomp4 mathomp4 marked this pull request as ready for review July 30, 2024 15:50
@mathomp4 mathomp4 requested a review from a team as a code owner July 30, 2024 15:50
@mathomp4 mathomp4 added the Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs label Jul 31, 2024
Copy link

This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged.

@mathomp4 mathomp4 closed this Oct 22, 2024
@mathomp4 mathomp4 deleted the feature/mathomp4/update-intelmpi-flags-v11 branch October 22, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch. Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant