Add GPU architecture BLACKWELL#488
Merged
hyschive merged 10 commits intogamer-project:mainfrom Feb 13, 2026
Merged
Conversation
Contributor
|
@koarakawaii Do you think we need to install cuda13 and nvidia hpc to spock:/software? So that the config no longer need to depend to your own file. |
Contributor
Author
|
@xuanweishan : I think that will be great. I guess we will need it in the future anyway~ |
Contributor
|
It seems good to me.
|
Contributor
Author
|
@xuanweishan : conflicts have been solved~ |
Merge with lastest main branch from public repo
merge with latest main branch from public repo
xuanweishan
reviewed
Feb 11, 2026
Contributor
xuanweishan
left a comment
There was a problem hiding this comment.
@koarakawaii
I've test it with following settings
eureka-
cuda11
-
spock-
cuda12 -
cuda13on ordinary node -
cuda13on RTX5090 node
-
GX10-
cuda13
It would be ready after update the path.
-
Contributor
Author
|
@xuanweishan Thank for all the testing~ I have updated the |
xuanweishan
approved these changes
Feb 11, 2026
Contributor
xuanweishan
left a comment
There was a problem hiding this comment.
@koarakawaii Thanks for the update
@hyschive This PR is ready to go.
hyschive
approved these changes
Feb 13, 2026
Contributor
hyschive
left a comment
There was a problem hiding this comment.
@koarakawaii @xuanweishan Looks good. Thanks for the contributions and review!
This file contains hidden or 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
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.
cudaDevicePropare deprecated inCUDA 13.0; I did not check all of them, but only modified those affecting the compilation. See https://docs.nvidia.com/cuda/cuda-runtime-api/structcudaDeviceProp.html#structcudaDeviceProp for details.cudaDeviceGetAttributeAPI; but again, I did not check whether it covers all the properties we queried forcudaDevicePropcufftDxNONEtoOPTION_NONEinGAMERto be compatible withcufftDx's.hppfile.spock_intel_RTX5090.configfor RTX5090 onspock26and GB10 forgx10, which uses private.NVIDIA HPC SDKmodules instead of publicCUDAon the log-in node. The path should be directing to publicCUDA 13.0when available, if we want to install itCUDA 12.1also works for current driver version580.95.05, but I am not sure we want to fall back toCUDA 12.1if a more advanced version has been released.cufftDxongx10is under private path.