Skip to content

Fixes for benchmark with small stack (RSA was being skipped)#7268

Merged
JacobBarthelmeh merged 3 commits intowolfSSL:masterfrom
dgarske:bench_fixes
Feb 23, 2024
Merged

Fixes for benchmark with small stack (RSA was being skipped)#7268
JacobBarthelmeh merged 3 commits intowolfSSL:masterfrom
dgarske:bench_fixes

Conversation

@dgarske
Copy link
Copy Markdown
Member

@dgarske dgarske commented Feb 22, 2024

Description

Fixes for benchmark with small stack (RSA was being skipped). Added Thumb2 benchmarks for STM32H753.

Fixes ZD 17565.

Testing

./configure --enable-smallstack && make
./wolfcrypt/benchmark/benchmark -rsa

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

* remove redundant nullness checks covered by WC_ALLOC_VAR() via WC_ALLOC_DO_ON_FAILURE();
* add "exit:" logic to bench_sm4_gcm();
* add missing WC_ALLOC_VAR() in bench_sm4_ccm();
* fix early return in bench_ed448KeySign().
@douzzer
Copy link
Copy Markdown
Contributor

douzzer commented Feb 23, 2024

retest this please

Copy link
Copy Markdown
Member Author

@dgarske dgarske left a comment

Choose a reason for hiding this comment

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

Approved. Over to PR cap.

@JacobBarthelmeh JacobBarthelmeh merged commit aa444c1 into wolfSSL:master Feb 23, 2024
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.

4 participants