-
Notifications
You must be signed in to change notification settings - Fork 376
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
Update chicoma-cpu gnu and nvidia builds #6705
Update chicoma-cpu gnu and nvidia builds #6705
Conversation
I have successfully run:
|
I'll rebase once that goes in. |
"LDFLAGS_OPT = -O3 -m64" \ | ||
"LDFLAGS_OPT = -O3 -m64 $(GNU_CRAY_LDFLAGS)" \ | ||
"FFLAGS_DEBUG = -g -m64 -ffree-line-length-none -fconvert=big-endian -ffree-form -fbounds-check -fbacktrace -ffpe-trap=invalid,zero,overflow -ffpe-summary=none $${EXTRA_FFLAGS}" \ | ||
"CFLAGS_DEBUG = -g -m64" \ | ||
"CXXFLAGS_DEBUG = -g -m64" \ | ||
"LDFLAGS_DEBUG = -g -m64" \ | ||
"LDFLAGS_DEBUG = -g -m64 $(GNU_CRAY_LDFLAGS)" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@matthewhoffman and @mark-petersen, this is what I want your review on. If you want to look at other changes or run tests, that would be appreciated but it's up to you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xylar , this solution for Chicoma looks good and seems reasonable given the current constraints. Thanks for sorting through this issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks very much @xylar! I used the head of compass and tested this branch with gnu, both debug and optimized. It passes all tests for the nightly test suite for MPAS-Ocean standalone.
I also tested E3SM on this branch with
cd cime/scripts
./create_test SMS_Ld3.T62_oQU240.GMPAS-NYF.chicoma-cpu_gnu \
-p t24_coastal_ocean --walltime 00:30:00
and it passes all steps.
Thanks for testing, but how did that work? I'm working on Compass support in MPAS-Dev/compass#865 but I wouldn't have expected anyone to be able to build standalone MPAS-Ocean or create a working Compass enviornment on Chicoma until that goes in. And that requires a mache update as well. |
This should be set to: ``` export GNU_CRAY_LDFLAGS="-Wl,--enable-new-dtags" ``` on Chicoma-CPU with gcc.
This is to match proposed updates to Perlmutter-CPU E3SM-Project#6702
c34336e
to
0cb78ea
Compare
Update chicoma-cpu gnu and nvidia builds Following the recent DST, this merge updates the module files and environment variables on LANL machine Chicoma-CPU. We note that these updates work well for gnu and nvidia compilers but not yet for intel, which we are continuing to work on. A separate update will be needed to address Chicoma-GPU as well. An environment variable has also been added to the MPAS standalone framework to accommodate overriding the library path for standard c++ on Chicoma. [BFB] for all E3SM testing
Passes:
and verified tests build and run on chicoma-cpu using gnu and nvidia Merged to next |
merged to master |
Thanks @jonbob! |
Primary purpose is to get updates for Chicoma builds from E3SM-Project#6705 * e3sm/master: (904 commits) Add overrun qos for pm-cpu/pm-gpu Update gcc and nvidia to newer versions Revert chicoma-gpu changes (for now) Add GNU_CRAY_LDFLAGS flag to MPAS standalone builds Update nvidia modules Updates to get E3SM working on chicoma-cpu Add latest cmake Update chicoma-cpu modules revert removing <env name="FI_CXI_RX_MATCH_MODE">software</env> as I hit merge conflict remove kdreg2 change as it's already on master bump nvidia compiler version on pm-cpu Fix for bad leaf C:N ratios during transient land simulations load Core/24.07 module and remove versions from git, submodule, and zlib modules remove special flags for gnu builds as kdreg2 seems to fix the issue For pm-cpu, update compiler versions for GCC, nvidia, AMD compilers. For those compilers, update other module versions to now be same as Intel uses. Various updates to muller-cpu/muller-gpu/alvarez. rename gh/ci-w to gh-w/ci add concurrency check for standalone updates to miniforge settings upgrades and fixes to workflows Upgrade NetCDF modules for Intel and GNU compilers on Chrysalis ...
Following the recent DST, this merge updates the module files and environment variables on Chicoma-CPU. We note that these updates work well for
gnu
andnvidia
compilers but not yet forintel
, which we are continuing to work on. A separate update will be needed to address Chicoma-GPU as well.An environment variable has also been added to the MPAS standalone framework to accommodate overriding the library path for standard c++ on Chicoma.
See E3SM-Ocean-Discussion#112 for relevant discussion
[BFB] for all E3SM testing