Skip to content

Implement a TLS perf server with poll system call.#330

Merged
lealem47 merged 1 commit intowolfSSL:masterfrom
JamesMenetrey:tls_poll_perf
Sep 26, 2022
Merged

Implement a TLS perf server with poll system call.#330
lealem47 merged 1 commit intowolfSSL:masterfrom
JamesMenetrey:tls_poll_perf

Conversation

@JamesMenetrey
Copy link
Copy Markdown
Contributor

Dear WolfSSL,

For research purposes, I have ported WolfSSL into an environment that does not support the EPOLL system calls, but only the POLL system call.

The next step for me is to evaluate how WolfSSL performs in that environment. Nonetheless, I observed that the TLS server performance implementation is only available using EPOLL calls. As such, I propose an alternative implementation which only relies on POLL calls. Obviously, this implementation is based on the EPOLL one and works with the already provided TLS client performance implementation.

I propose this PR with the POLL implementation to help people in the same situation as me.

Feel free to suggest edits or improvements if you feel they are needed, I'll happily comply. Also, a contributor agreement may be needed for contributing to WolfSSL code repositories. I'm ready to sign it if required.

Cheers

@dgarske
Copy link
Copy Markdown
Contributor

dgarske commented Jul 26, 2022

Thanks @JamesMenetrey , for your work on this. We will review.

@dgarske dgarske self-assigned this Jul 26, 2022
@dgarske
Copy link
Copy Markdown
Contributor

dgarske commented Sep 2, 2022

Hi @JamesMenetrey would you be willing to sign a contributor agreement? If so please email support@wolfssl.com and we will share the document to sign. These changes are excellent. Another engineer @lealem47 is going to review them.

@dgarske dgarske assigned lealem47 and unassigned dgarske Sep 2, 2022
@JamesMenetrey
Copy link
Copy Markdown
Contributor Author

JamesMenetrey commented Sep 8, 2022

Hello @dgarske! Great to hear, thanks! I have sent an email over there :)

@embhorn
Copy link
Copy Markdown
Member

embhorn commented Sep 20, 2022

Hello @JamesMenetrey

Please return the signed contributor agreement in order for us to proceed with this PR. I messaged you from our ZenDesk portal.

@JamesMenetrey
Copy link
Copy Markdown
Contributor Author

Dear @embhorn,

I just returned the document to the ZenDesk portal.

Thanks!

@embhorn
Copy link
Copy Markdown
Member

embhorn commented Sep 26, 2022

Okay to test

@embhorn embhorn requested review from embhorn and lealem47 and removed request for embhorn September 26, 2022 15:41
@embhorn
Copy link
Copy Markdown
Member

embhorn commented Sep 26, 2022

Hi @JamesMenetrey

You have been approved as a contributor for the wolfSSL project!

Copy link
Copy Markdown
Contributor

@lealem47 lealem47 left a comment

Choose a reason for hiding this comment

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

Thank you for your wonderful work @JamesMenetrey .

@lealem47 lealem47 merged commit 3d37f3e into wolfSSL:master Sep 26, 2022
@JamesMenetrey
Copy link
Copy Markdown
Contributor Author

Great! I'm glad it has been merged! Thanks for the review!

Cheers

yota22721 pushed a commit to yota22721/wolfssl-examples that referenced this pull request Jan 25, 2025
Implement a TLS perf server with poll system call.
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