tuntap: correctly set SOCKWQ_ASYNC_NOSPACE
authorJason Wang <jasowang@redhat.com>
Tue, 22 May 2018 06:21:04 +0000 (14:21 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 23 May 2018 18:32:12 +0000 (14:32 -0400)
commit2f3ab6221e4c87960347d65c7cab9bd917d1f637
tree5a4050742518ae4b6df94df2ee624e54636770cf
parenta43ad59a908055d288e524e559494805a6c7299e
tuntap: correctly set SOCKWQ_ASYNC_NOSPACE

When link is down, writes to the device might fail with
-EIO. Userspace needs an indication when the status is resolved.  As a
fix, tun_net_open() attempts to wake up writers - but that is only
effective if SOCKWQ_ASYNC_NOSPACE has been set in the past. This is
not the case of vhost_net which only poll for EPOLLOUT after it meets
errors during sendmsg().

This patch fixes this by making sure SOCKWQ_ASYNC_NOSPACE is set when
socket is not writable or device is down to guarantee EPOLLOUT will be
raised in either tun_chr_poll() or tun_sock_write_space() after device
is up.

Cc: Hannes Frederic Sowa <hannes@stressinduktion.org>
Cc: Eric Dumazet <edumazet@google.com>
Fixes: 1bd4978a88ac2 ("tun: honor IFF_UP in tun_get_user()")
Signed-off-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/tun.c