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

net/litep2p fail to send request ChannelClogged #5645

Open
alexggh opened this issue Sep 9, 2024 · 1 comment
Open

net/litep2p fail to send request ChannelClogged #5645

alexggh opened this issue Sep 9, 2024 · 1 comment

Comments

@alexggh
Copy link
Contributor

alexggh commented Sep 9, 2024

I've been running some testing in CPU restrictive environments, describe here: #5035 (comment), and I observed that when switching to litep2p parachains blocks production slow down from ~6s average to around ~10-11s, the problems seems to be that availability protocol is way slower, I see a lot of errors like:

sub-libp2p::request-response: /89b720ba4ad8cf2a849c2c19f2a8b8911f077fbc670a69e3123fcc4e2da25454/req_chunk/2: failed to send request to PeerId("12D3KooWFycxBdu976AXgJSL4g6dxMbW8swQ3LbkHcwY1qF53QPB"): ChannelClogged  

Logs: https://grafana.teleport.parity.io/goto/1Dgn9teSg?orgId=1

FYI: @paritytech/networking

@lexnv
Copy link
Contributor

lexnv commented Sep 9, 2024

Nice catch! Thanks for reporting this to us 🙏

We have noticed some performance differences between libp2p and litep2p in terms of req-resp protocols.

It would be interesting to see how this performs with the latest version (0.7), once we merge this:

Link to the channel bound:

// Hundreds of validators will start requesting their chunks once they see a candidate
// awaiting availability on chain. Given that they will see that block at different
// times (due to network delays), 100 seems big enough to accommodate for "bursts",
// assuming we can service requests relatively quickly, which would need to be measured
// as well.
Protocol::ChunkFetchingV1 | Protocol::ChunkFetchingV2 => 100,

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

No branches or pull requests

2 participants