Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Espressif benchmark update #7037

Merged
merged 4 commits into from
Dec 8, 2023

Conversation

gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented Dec 6, 2023

Description

This PR Updates the wolfcrypt benchmark.c to take into account various issues for different Espressif SoC devices.

This is a companion PR to #7035 and all of the following are now supported:

  • esp32
  • esp32c2
  • esp32c3
  • esp32c6
  • esp32s2
  • esp32s3
  • esp32h2

A new DEBUG_WOLFSSL_BENCHMARK_TIMING macro is introduced to enable verbose timing related debugging.

A new MILLION_VALUE macro is introduced for the fixed value of 1,000,000.0 (1000000.0)

As the CPU Cycle Counter is known to overflow during benchmarks on slower, embedded systems, a new multi-overflow implementation now calculated approximate CPU cycle counts when needed using the FreeRTOS tick counts, instead. As the TickCounts are not guaranteed to start at the "beginning of time" for RTOS, we lose some granularity, depending on configTICK_RATE_HZ.

See #6234 for a roadmap of Espressif updates.

Fixes zd# n/a

Testing

How did you test?

Tested with Espressif apps and:

./configure --enable-all
make clean
make && make test
 ./wolfcrypt/benchmark/benchmark

Checklist

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

@dgarske dgarske removed their request for review December 6, 2023 23:51
@dgarske dgarske removed their assignment Dec 6, 2023
@JacobBarthelmeh
Copy link
Contributor

@gojimmypi picked up a merge conflict

@gojimmypi
Copy link
Contributor Author

@JacobBarthelmeh I've resolved the merge conflict.

@JacobBarthelmeh JacobBarthelmeh merged commit 0ba3646 into wolfSSL:master Dec 8, 2023
108 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants