renaming abortutils as cam_abortutils to enable running with COSP #89
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 6 commits
January 30, 2015 13:06
Build with COSP depends on cam_abortutils. The ChangeLog for a later version of CAM indicates that abortutils was renamed as cam_abortutils in order to avoid conflicts with an abortutils module defined and used in CLM.
All references to abortutils in CAM were changed to cam_abortutils. Most of these were of the form "use abortutils", but a few instances were also in comments and build logic.
…(2nd try) Previous commit changed all instances in atm/cam/src . These changes also include atm/cam/bld.
…ompiler COSP does not work in all configurations of interest on Titan unless update to a newer version of PGI. (This is also required when building and running with CLM4.5 on Titan.)
…ompiler (2nd try) pgi/14.10 is not yet a OLCF-supported module, and the module load committed previously is now broken. This one works (today).
When running COSP the stacksize needs to be at least 128M when running the ne120 water cycle science case (rather than the 64M required previously).
This was referenced Mar 3, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The scripts update that came with the CLM4.5 update also added a build dependency on a module called cam_abortutils. This does not exist in the version of CAM we are using. Checking the ChangeLog, cam_abortutils is a rename of the abortutils module, to avoid a name conflict with a similarly named module in CLM. On the advice of the software engineers, the decision was made to implement this renaming in CAM (rather than change the dependency in the build logic):
a) renaming abortutils as cam_abortutils
b) replacing all instances of "abortutils" with "cam_abortutils" in CAM
For COSP to run on Titan for the ne120 water cycle benchmark, two more changes were required:
c) replacing pgi/14.2.0 with pgi/14.10.home (on Titan)
d) increasing the thread stack size from 64M to 128M (on Titan)
[should BFB except for the change in compiler version on Titan; only tested build and successful run on Titan; used grep to look for all instances of abortutils and I believe that I found all of them in CAM, and all others are in CLM, and should be changed.]
[unnamed subtask of JIRA task PG-46]