Skip to content

Optimizations for tracer horizontal mixing#538

Closed
mattdturner wants to merge 3 commits intoMPAS-Dev:ocean/developfrom
mattdturner:ocean/tracer_hmix_ocnMesh
Closed

Optimizations for tracer horizontal mixing#538
mattdturner wants to merge 3 commits intoMPAS-Dev:ocean/developfrom
mattdturner:ocean/tracer_hmix_ocnMesh

Conversation

@mattdturner
Copy link
Collaborator

Optimizations to tracer horizontal mixing that include

  • Removing pointer retrievals
  • Trimming argument lists to remove Pool variables (no longer used in these routines)
  • Replace mesh variables (previously retrieved from meshPool) with module-level variables in ocn_mesh module (relies on PR New ocean mesh structure with GPU replication #496 being merged)

This PR requires the following PRs be merged first:

Some of the changes in those PRs were either git cherry-pick'ed or manually added. As a result, once those PRs are merged there might need to be some commits squashed or changes reverted in this PR.

@mattdturner
Copy link
Collaborator Author

This was tested using a QU240 test case, and a EC60to30 test case (both generated via Compass). The results are bit-for-bit with Intel compiler on Cori.

As a result of the optimizations in this PR, there is a reduction of the time spent in tracer hmix by approx. 5%. Unfortunately, all of this runtime reduction came from the Redi code and there was no speedup to the del4 code.

Copy link
Contributor

@philipwjones philipwjones left a comment

Choose a reason for hiding this comment

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

This was b4b on Summit with PGI as well. Visual inspection looked fine

@mattdturner
Copy link
Collaborator Author

Rebased to current ocean/develop.

@mattdturner mattdturner force-pushed the ocean/tracer_hmix_ocnMesh branch from 61c2371 to 15c1548 Compare December 8, 2020 20:10
@mattdturner mattdturner deleted the ocean/tracer_hmix_ocnMesh branch March 16, 2021 16:44
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