Add CUDA support #7436
Merged
Add CUDA support #7436
Conversation
This file contains 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
added 22 commits
April 17, 2024 10:03
dgarske
requested changes
Apr 17, 2024
| @@ -0,0 +1,156 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
You'll need to add this file to scripts/include.am.
There was a problem hiding this comment.
Yup. Already done locally. Didn't want to trigger another build
wolfcrypt/src/port/cuda/README.md
Outdated
| @@ -0,0 +1,4 @@ | |||
| You will need to have the CUDA libraries and toolchains installed to be able to use this. This code was tested with the following: | |||
There was a problem hiding this comment.
Could we expand the documentation? Seems like there should be a bit more here
There was a problem hiding this comment.
Sure. I'll add my environment setup.
|
Retest this please |
dgarske
approved these changes
Apr 23, 2024
|
@night1rider will you plan to take this for a test-drive? |
Yes, I will schedule a job on the HPC to run. |
|
Ran it in a singularity container here is my results for the H100 @dgarske @bandi13 : H100 Benchmark |
kareem-wolfssl
pushed a commit
to kareem-wolfssl/wolfssl
that referenced
this pull request
Jun 27, 2024
* Redirect the AesEncrypt_C call to device * Fix function declarations * Force CC=nvcc with CUDA * Don't let C++ mangle function names * Add larger parallelization * Add in memory copy to device * `nvcc` does not support '-Wall' nor '-Wno-unused' * Add in README.md * Clean up script to output color coded data * Fix Asymmetric cipher comparisons * Add in standard output parsing in addition to the CSV * Add option to output results in a CSV --------- Co-authored-by: Andras Fekete <[email protected]>
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.
Added in support for AesEncrypt_C in CUDA