Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Quic Client Creates Excessive Number of Quinn Endpoints #32377

Closed
ryleung-solana opened this issue Jul 5, 2023 · 6 comments
Closed

Quic Client Creates Excessive Number of Quinn Endpoints #32377

ryleung-solana opened this issue Jul 5, 2023 · 6 comments
Labels
stale [bot only] Added to stale content; results in auto-close after a week.

Comments

@ryleung-solana
Copy link
Contributor

Problem

The Quic client creates a new Quinn Endpoint for each connection pool. This can result in excessive port usage in large networks (as each Endpoint uses a separate UDP port).

Proposed Solution

The above behavior was based on performance problems in Quinn observed by @lijunwangs with a single Quinn Endpoint. We should root-cause this perf problem and propose a fix to Quinn, so that we may use a single Quinn Endpoint for all the connection pools.

@ryleung-solana
Copy link
Contributor Author

Benched on GCE with bench-tps, and did not see any significant difference. However, results may be biased by #32179 which causes inconsistent runs of low TPS in general. Would like to see this fixed and get some consistent runs before concluding that this is no longer a problem.

@apfitzge
Copy link
Contributor

The issues as described in #32179 began with changes to connection cache which affects quic.

@behzadnouri
Copy link
Contributor

From my observations at least, this still seems to be an issue.
i.e. same quinn Endpoint creates contention across connections or send/receive operations on that Endpoint..

@ryleung-solana
Copy link
Contributor Author

@behzadnouri interesting... what stats are you looking at that show this as a bottleneck?

@behzadnouri
Copy link
Contributor

@behzadnouri interesting... what stats are you looking at that show this as a bottleneck?

Testing turbine on quic, I see evidence that hints to contention on quic endpoint across connections or send/receive operations.

@github-actions github-actions bot added the stale [bot only] Added to stale content; results in auto-close after a week. label Jul 19, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jul 29, 2024
@bw-solana bw-solana reopened this Nov 15, 2024
Copy link
Contributor

This repository is no longer in use. Please re-open this issue in the agave repo: https://github.com/anza-xyz/agave

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale [bot only] Added to stale content; results in auto-close after a week.
Projects
None yet
Development

No branches or pull requests

4 participants