Skip to content

Expanding static memory DMA offset feature#191

Merged
bigbrett merged 4 commits intowolfSSL:mainfrom
JacobBarthelmeh:staticmemory_2
Nov 3, 2025
Merged

Expanding static memory DMA offset feature#191
bigbrett merged 4 commits intowolfSSL:mainfrom
JacobBarthelmeh:staticmemory_2

Conversation

@JacobBarthelmeh
Copy link
Copy Markdown
Contributor

@JacobBarthelmeh JacobBarthelmeh commented Sep 25, 2025

Make use of static memory DMA with:

  • CMAC
  • SHA224
  • SHA384
  • SHA512
  • ML-DSA

@JacobBarthelmeh JacobBarthelmeh self-assigned this Sep 25, 2025
@JacobBarthelmeh JacobBarthelmeh force-pushed the staticmemory_2 branch 3 times, most recently from 1c2293d to 53d7586 Compare September 26, 2025 22:35
@JacobBarthelmeh JacobBarthelmeh marked this pull request as ready for review September 26, 2025 22:39
Copy link
Copy Markdown
Contributor

@bigbrett bigbrett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great additions, I think there might be a few tweaks/bugs

Comment thread benchmark/bench_modules/wh_bench_mod_cmac.c
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread benchmark/bench_modules/wh_bench_mod_cmac.c
Comment thread src/wh_client_crypto.c
@JacobBarthelmeh
Copy link
Copy Markdown
Contributor Author

rebasing onto main to resolve merge conflicts

@JacobBarthelmeh JacobBarthelmeh force-pushed the staticmemory_2 branch 2 times, most recently from 5cd1285 to 14ebc6c Compare October 17, 2025 21:16
Copy link
Copy Markdown
Contributor

@bigbrett bigbrett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work Jacob, a few more tweaks I noticed just for consistency. Also please resolve merge conflicts

Comment thread benchmark/bench_modules/wh_bench_mod_cmac.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
fix for hash dma additions

mldsa using DMA offset feature

remove unused flag and add malloc to cmac benchmark

add macro guards around DMA additions

fix typo for state address with cmac

run git-clang-format
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
Comment thread src/wh_client_crypto.c Outdated
@JacobBarthelmeh JacobBarthelmeh removed their assignment Oct 31, 2025
@bigbrett bigbrett self-requested a review November 3, 2025 16:37
@bigbrett bigbrett merged commit d3f43c9 into wolfSSL:main Nov 3, 2025
15 checks passed
jackctj117 pushed a commit that referenced this pull request Nov 3, 2025
* add cmac and start of other hash algos

fix for hash dma additions

mldsa using DMA offset feature

remove unused flag and add malloc to cmac benchmark

add macro guards around DMA additions

fix typo for state address with cmac

run git-clang-format

* add return value checks, free in benchmark buffer on error, add macro guards

* fix typo and add fix return value checks

* fix flag for read/write with DMA operations
billphipps pushed a commit to billphipps/wolfHSM that referenced this pull request Nov 3, 2025
* add cmac and start of other hash algos

fix for hash dma additions

mldsa using DMA offset feature

remove unused flag and add malloc to cmac benchmark

add macro guards around DMA additions

fix typo for state address with cmac

run git-clang-format

* add return value checks, free in benchmark buffer on error, add macro guards

* fix typo and add fix return value checks

* fix flag for read/write with DMA operations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants