Catch and ignore EINVAL errors from setsockopt.

This seems to happen a lot in benchmark runs when sockets are closed
after a request.

diff --git a/tornado/iostream.py b/tornado/iostream.py
index 7a5c380da2397949022c7ca3fe457899bb384d03..1dfe06a686d0d7d68d871875bbe38805930be6cd 100644 (file)
--- a/tornado/iostream.py
+++ b/tornado/iostream.py
@@ -742,8 +742,15 @@ class IOStream(BaseIOStream):
     def set_nodelay(self, value):
         if (self.socket is not None and
             self.socket.family in (socket.AF_INET, socket.AF_INET6)):
-            self.socket.setsockopt(socket.IPPROTO_TCP,
-                                   socket.TCP_NODELAY, 1 if value else 0)
+            try:
+                self.socket.setsockopt(socket.IPPROTO_TCP,
+                                       socket.TCP_NODELAY, 1 if value else 0)
+            except socket.error as e:
+                # Sometimes setsockopt will fail if the socket is closed
+                # at the wrong time.  This can happen with HTTPServer
+                # resetting the value to false between requests.
+                if e.errno != errno.EINVAL:
+                    raise
 
 
 class SSLIOStream(IOStream):