-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
8.1.0 download slower than 8.0.1 #11162
Comments
@pandada8 what's the average ping time client <=> server in this setup ? |
Around 3.5ms |
This seems to be the effect of the lower stream window size we introduced. I have a rather slow downlink, but I can see that the higher the ping time to a mirror, the less it becomes saturated. Increasing the http/2 window size - in the source - helps. The relevant place in #define H2_STREAM_WINDOW_SIZE (512 * 1024) I would be interested to know what you experience when you increase that to |
I play around with different values, given 3.5ms latency
It does improve the performance, the final difference could be a margin of error |
Thanks for the measurements. Since I want a quick fix for the upcoming patch release, I suggest I settle for a 10MB window for now. There might be a good way to adjust that more dynamically, but that is outside the current time frame. |
PR #11167 in CI now. |
Reported-by: pandada8 on github Fixes curl#11162 Closes curl#11167
I did this
When download from same url with same network, curl 8.1.0 downloads half as fast as curl 8.0.1
(try a nearby mirror to test if you want to replicate, the provided link speed may be not optimal in your location)
I also tried different endpoint in a different location (vultr vps in la). It seems the slowdown ratio may be different, but 8.1.0 IS slower than 8.0.1
I expected the following
download speed shouldn't make any difference
curl/libcurl version
I did some bisect, it seems the slowdown is introduced by 744dcf2
Indeed, the provided link is http2 enabled
operating system
Linux panda-router 6.2.11-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 13 Apr 2023 16:59:24 +0000 x86_64 GNU/Linux
The text was updated successfully, but these errors were encountered: