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

[DRAFT][WIP] Use the threadpool for sending - posttask is too slow #4367

Draft
wants to merge 1 commit into
base: 25.lts.1+
Choose a base branch
from

Conversation

jellefoks
Copy link
Member

@jellefoks jellefoks commented Nov 5, 2024

Status of draft:

  • Threadpool for sending packets doesn't work because the lock for PostTask costs as much as the packet writing. Probably needs to be done more efficiently, maybe a pipe such as I made in [DRAFT] was slow now not so much getting real close #3358, or lockless. It will need an extension for the former, but the latter will still need a very efficient wait & wake approach, and what we have all uses mutexes (pthread_cond_wait, and base:WaitableEvent). An eventfd is probably more efficient than a pipe, and can be used to further improve the MessagePump so that is probably a good next choice.
  • I'm experimenting with moving the QUIC packet processing off-thread.

@jellefoks jellefoks changed the title [DRAFT][WIP] Performance galore? [DRAFT][WIP] Use the threadpool for sending - posttask is too slow Dec 3, 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.

1 participant