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

Connection freezes in high_bandwidth #75

Closed
russelltg opened this issue Jan 21, 2021 · 5 comments · Fixed by #136
Closed

Connection freezes in high_bandwidth #75

russelltg opened this issue Jan 21, 2021 · 5 comments · Fixed by #136
Milestone

Comments

@russelltg
Copy link
Owner

russelltg commented Jan 21, 2021

Steps to reproduce:

Set the default buffer sizes to be larger (this should be an option that we expose, but for now it isn't):

sudo sysctl -w net.core.rmem_default=8388608
sudo sysctl -w net.core.wmem_default=8388608

Run the high_bandwidth test:

cargo test --release --test=high_bandwidth -- --nocapture

And notice after a few seconds it freezes. You may have to tweak the bandwidth by changing the RATE_MBPS constant to get it to repro (higher if it's working properly, lower if it isn't delivering the requested bandwidth).

This may or may not be reproducible outside of Linux, haven't tried.

@robertream
Copy link
Collaborator

Is this resolved with the latest build? The latest changes might have fixed this too.

@russelltg
Copy link
Owner Author

russelltg commented Nov 24, 2021

This may be fixed, but the high_bandwidth test still freezes up when ran for long enough at high enough bandwidth, so I think I'll leave it open

@russelltg russelltg changed the title Fix extreme throttling after dropping a packet Connection freezes in high_bandwidth Nov 24, 2021
@robertream
Copy link
Collaborator

I agree, though ensure you are running it in release mode, at high speeds and too low of a latency setting, it's highly sensitive to thread timing. It might be interesting to see if this behavior could be reproduced in a "faster than real time" test.

@russelltg
Copy link
Owner Author

Yep, that would be great to repro in a deterministic way. I am running in release, I put my repro steps in the original issue

@russelltg
Copy link
Owner Author

(Blocked on #119 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants