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

HTTP server connection stability issues #213

Closed
malthe opened this issue May 20, 2015 · 3 comments
Closed

HTTP server connection stability issues #213

malthe opened this issue May 20, 2015 · 3 comments

Comments

@malthe
Copy link
Contributor

malthe commented May 20, 2015

The server stays up, but is generally unable to handle:

weighttp -k -n 100000 -c 1000 http://127.0.0.1:50000

Typically, more than 85% of the requests do succeed, but various failure modes can happen that still causes a large number of requests to fail such as:

error: read() failed: Connection reset by peer (104)
error: connect() failed: Connection refused (111)

I did not look further into these, but it seems to be a socket handling issue.

@sylvanc sylvanc added the bug label Jun 1, 2015
@sylvanc
Copy link
Contributor

sylvanc commented Jun 5, 2015

After much investigation and twiddling, this seems to be working on Linux, FreeBSD and Windows now. On OSX there is still an outstanding issue: if a TCPListener accepts connections fast enough, there is some (as yet not identified) OS resource than be exhausted, leading to a KEVENT_READ not happening on the listening socket for ~30 seconds.

Under these circumstances, netstat -mm seems to show mbufs are not exhausted. The connecting socket is shows as SYN_SENT in lsof, and netstat -a -L shows the listening socket has a 0 qlen and a 0 incqlen. This is annoying.

Can you confirm that connections aren't being dropped for you, other than this OSX "lockup" issue?

@sylvanc
Copy link
Contributor

sylvanc commented Jun 5, 2015

We're looking into a Windows problem as well now.

@sylvanc
Copy link
Contributor

sylvanc commented Nov 12, 2015

Latest round should have this fixed.

@sylvanc sylvanc closed this as completed Nov 12, 2015
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