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

CA fails debug mode with GNU compiler #770

Closed
DeniseWorthen opened this issue Aug 25, 2021 · 4 comments · Fixed by #832
Closed

CA fails debug mode with GNU compiler #770

DeniseWorthen opened this issue Aug 25, 2021 · 4 comments · Fixed by #832
Labels
bug Something isn't working

Comments

@DeniseWorthen
Copy link
Collaborator

DeniseWorthen commented Aug 25, 2021

Description

The coupled RTs are tested with GNU on both Hera and Cheyenne. The cpld_control_p7 test from PR 765 fails in debug mode with GNU. This seems to be a known issue, since in rt_gnu.conf, the test is commented out with a note that # Both stochy and ca debug tests crash with GNU.

For the coupled P7 tests to include CA, this will need to be addressed.

To Reproduce:

Tested on hera.gnu. Compile the cpld_control_p7 test in debug mode for GNU. The run fails with Program received signal SIGFPE: Floating-point exception - erroneous arithmetic operation. and the err log points here:

 68: #0  0x2b42939e73ff in ???
 70: #0  0x2aba00fa13ff in ???
125: #1  0x21ca6d4 in __update_ca_MOD_update_cells_sgs
125:    at /scratch2/NCEPDEV/climate/Denise.Worthen/WORK/ufs_cpldv16_gnu/stochastic_physics/update_ca.F90:249
 71: #0  0x2b9d3a2ab3ff in ???
 72: #0  0x2ad2d44333ff in ???

Line 249 is:

 else
    ! don't rely on compiler to truncate integer(8) to integer(4) on
    ! overflow, do wrap around explicitly.
    count4 = mod(kstep + mype + iseed_ca + 2147483648, 4294967296) - 2147483648
 endif
@DeniseWorthen DeniseWorthen added the bug Something isn't working label Aug 25, 2021
@DeniseWorthen
Copy link
Collaborator Author

@lisa-bengtsson

@lisa-bengtsson
Copy link
Contributor

Hi @DeniseWorthen I recall this issue from Dom's test. It seems the gnu compiler doesn't like this arithmetic. We are updating this line in the new PR that addresses MPI reproducibility, and will ensure the gnu debug test works in this update.

@DeniseWorthen
Copy link
Collaborator Author

@lisa-bengtsson I expected that to be the case, but I wanted to make sure that since you were changing the code for MPI reproducibility that this issue didn't get overlooked.

@lisa-bengtsson
Copy link
Contributor

I'm on it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants