[issue27456] asyncio: set TCP_NODELAY flag by default
Vladimir Magamedov
report at bugs.python.org
Thu Nov 2 11:48:27 EDT 2017
Vladimir Magamedov <vladimir at magamedov.com> added the comment:
Seems like this fix is incomplete. It contains this check:
sock.type == socket.SOCK_STREAM
But sock.type is not only a type (at least in Linux and FreeBSD), it also may contain SOCK_NONBLOCK and SOCK_CLOEXEC flags. So I'm hitting the same problem: on the Linux in asyncio I have:
> sock.type == socket.SOCK_STREAM | socket.SOCK_NONBLOCK == 2049
True
So this check isn't working and TCP_NODELAY still disabled by default.
Links:
- http://man7.org/linux/man-pages/man2/socket.2.html
- https://github.com/torvalds/linux/blob/v4.13/include/linux/net.h#L77
- https://github.com/freebsd/freebsd/blob/stable/11/sys/sys/socket.h#L110
Linux has SOCK_TYPE_MASK definition equal to 0xf, but I can't find such definition in the FreeBSD sources. And I don't know how to reliably and with forward compatibility check sock.type without calling getsockopt() syscall.
Currently I have a fix in my project, where:
_sock_type_mask = 0xf if hasattr(socket, 'SOCK_NONBLOCK') else 0xffffffff
And then in my own _set_nodelay(sock) function:
sock.type & _sock_type_mask == socket.SOCK_STREAM
Should I make a pull request or someone knows more reliable check? Or it is ok to add one more syscall?
sock.getsockopt(socket.SOL_SOCKET, socket.SO_TYPE) == socket.SOCK_STREAM
----------
nosy: +vmagamedov
_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue27456>
_______________________________________
More information about the Python-bugs-list
mailing list