100% CPU with SSL fix #552

Merged
merged 1 commit into from Jun 29, 2012

2 participants

@vigasin

When some connection was reset Tornado started to use 100% of CPU. That was caused by not closed socket which constantly generated socket.error with errno.ECONNRESET code instead handled errno.ECONNABORTED.

Platform the error detected: windows. However other platforms probably affected too.

@bdarnell
tornadoweb member

Can you reproduce this with a simple test case? It's always good to get more test cases for platform-specific behavior, but if it's difficult to reproduce don't worry about it.

@bdarnell bdarnell merged commit ff12f29 into tornadoweb:master Jun 29, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment