-
Notifications
You must be signed in to change notification settings - Fork 106
Specify CUDA version for login node builds #697
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
Conversation
Mat Colgrove's advice: https://forums.developer.nvidia.com/t/cannot-compile-on-login-node/282759 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #697 +/- ##
=======================================
Coverage 42.99% 42.99%
=======================================
Files 61 61
Lines 16053 16053
Branches 1796 1796
=======================================
Hits 6902 6902
Misses 8167 8167
Partials 984 984 ☔ View full report in Codecov by Sentry. |
@max-Hawkins let me know the status of this. Will merge when it is ready (it passed Phoenix tests), just need to confirm it works fine on Delta - perhaps you already checked this. |
Tested on both Delta and Phoenix after removing the entire build dir. All we had to do was set the NVHPC_CUDA_HOME environment variable. It just took me longer than it should have to realize this. |
Description
Currently, specifying the MFC_CUDA_CC flag can cause builds to fail on login nodes where the system can't determine the CUDA version to build for. This adds a compiler flag to specify the CUDA version explicitly. To ensure that rearranging the build process doesn't break, I force CMAKE to find CUDA when the MFC_CUDA_CC flag is set.
Fixes #(687)
Fixes #687
Type of change
Please delete options that are not relevant.
Scope
If you cannot check the above box, please split your PR into multiple PRs that each have a common goal.
How Has This Been Tested?
Load modules for specific system
rm -rf build/staging
export MFC_CUDA_CC=XX
where XX is a compute capabilityBuild GPU MFC on Phoenix and Delta
See above
Checklist
docs/
)examples/
that demonstrate my new feature performing as expected.They run to completion and demonstrate "interesting physics"
./mfc.sh format
before committing my codeIf your code changes any code source files (anything in
src/simulation
)To make sure the code is performing as expected on GPU devices, I have:
nvtx
ranges so that they can be identified in profiles./mfc.sh run XXXX --gpu -t simulation --nsys
, and have attached the output file (.nsys-rep
) and plain text results to this PR./mfc.sh run XXXX --gpu -t simulation --omniperf
, and have attached the output file and plain text results to this PR.