Skip to content

{phys}[nvofbf/2024.9] VASP v6.4.3 w/ hdf5#21641

Open
sassy-crick wants to merge 1 commit intoeasybuilders:developfrom
sassy-crick:20241014144659_new_pr_VASP643
Open

{phys}[nvofbf/2024.9] VASP v6.4.3 w/ hdf5#21641
sassy-crick wants to merge 1 commit intoeasybuilders:developfrom
sassy-crick:20241014144659_new_pr_VASP643

Conversation

@sassy-crick
Copy link
Collaborator

@sassy-crick sassy-crick commented Oct 14, 2024

(created using eb --new-pr)

Requires:

Currently the patches have the CUDA version hard-coded in it (see comment in the EC files) and a better way needs to be found here.
Help from @bedroge and @Thyre for helpful discussions is much appreciated!

…24.9-hdf5.eb and patches: vasp-6.4.3_nv-flags.patch, vasp-6.4.3_hdf5.patch
@tanmoy1989
Copy link
Contributor

Hi, this is just to add-in that while building VASP/6.4.3 using this PR we noticed that it only works for L40 GPUs because of the use of only cc89 parameter against -gpu flag in the makefile.include coming from vasp-6.4.3_nv-flags.patch. To make the build more robust if cc80 is added then that seems to be working well on both A100s and L40s. So I propose the following modification in lines 15-17 in vasp-6.4.3_nv-flags.patch

+CC          = mpicc  -acc -gpu=cc80,cc89,cuda12.6 -mp
+FC          = mpif90 -acc -gpu=cc80,cc89,cuda12.6 -mp
+FCL         = mpif90 -acc -gpu=cc80,cc89,cuda12.6 -mp -c++libs

@sassy-crick
Copy link
Collaborator Author

sassy-crick commented Aug 18, 2025

There is another PR open for VASP-6.5.1: PR #23188
I would suggest to leave that one open until the NVHPC toolchain has settled a bit and then look into it again, and either close it or improve it and merge it.

@sassy-crick sassy-crick added the 2024a issues & PRs related to 2024a common toolchains label Aug 18, 2025
@Thyre Thyre added the manual_download Sources require manual download label Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2024a issues & PRs related to 2024a common toolchains manual_download Sources require manual download update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants