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

Add TLS connector/acceptor benchmarks using valgrind #367

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

hargut
Copy link
Contributor

@hargut hargut commented Aug 30, 2024

This PR tries to provide a solution for the issue described in #358.

For further details please refer to the documentation in pingora-core/benches/tls_benchmarks.md.

hargut added 3 commits August 30, 2024 18:51
for further details please refer to the documentation in
pingora-core/benches/tls_benchmarks.md
@eaufavor eaufavor added the Performance Performance of efficiency. label Aug 30, 2024
@hargut
Copy link
Contributor Author

hargut commented Sep 6, 2024

During comparing the TLS implementations I've noticed that there was no option to run multiple instances (e.g. listeners, connectors) in parallel.

Added another set of benchmarks that does parallel checks, starting multiple connectors/acceptors and issuing request in a parallel manner.
Enabled DHAT runs to as well have more details on heap usage.

Refactored file layout to start with main() at the top.

@johnhurt johnhurt self-assigned this Nov 15, 2024
@hargut
Copy link
Contributor Author

hargut commented Dec 18, 2024

Hi @johnhurt,

do you see any value for the future in this PR / setup?
For me it has served the purpose for learning and I'm currently not intending to use it any further.

For anyone interested some results in comparing Rustls with OpenSSL using this PR had been published in the Rustls ticket. #29 (comment)

Kind regards,
Harald

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Performance Performance of efficiency.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants