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

Add failing unit tests to reproduce issue #312 #313

Closed
wants to merge 1 commit into from

Conversation

tardyp
Copy link
Contributor

@tardyp tardyp commented Feb 12, 2020

reproduces #312

The problem looks to be reproduced on the darwin side of your CI, but not on linux.

https://travis-ci.org/MagicStack/uvloop/jobs/649663884?utm_medium=notification&utm_source=github_status

@tardyp
Copy link
Contributor Author

tardyp commented Feb 14, 2020

After afternoon of tracing through the code, I am giving up for the day :(

I am observing that the protocol is pausing after sending the first round of buffer.

_get_write_buffer_size at

cdef size_t _get_write_buffer_size(self):

returns 211072, which is my initial buffer minus 8192.

__uv_stream_on_write_impl() is never called

Then.. I got lost in https://github.com/libuv/libuv/blob/07ad32138f4d2285ba2226b5e20462b27b091a59/src/unix/stream.c#L1398

fantix pushed a commit to fantix/uvloop that referenced this pull request May 9, 2020
fantix pushed a commit to fantix/uvloop that referenced this pull request May 9, 2020
fantix pushed a commit to fantix/uvloop that referenced this pull request May 13, 2020
@fantix fantix closed this in #342 May 14, 2020
fantix pushed a commit that referenced this pull request May 14, 2020
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