Commit 33b603fd authored by David Howells's avatar David Howells

rxrpc: Requeue call for recvmsg if more data

rxrpc_recvmsg() needs to make sure that the call it has just been
processing gets requeued for further attention if the buffer has been
filled and there's more data to be consumed.  The softirq producer only
queues the call and wakes the socket if it fills the first slot in the
window, so userspace might end up sleeping forever otherwise, despite there
being data available.

This is not a problem provided the userspace buffer is big enough or it
empties the buffer completely before more data comes in.
Signed-off-by: default avatarDavid Howells <dhowells@redhat.com>
parent 91c2c7b6
...@@ -463,6 +463,10 @@ int rxrpc_recvmsg(struct socket *sock, struct msghdr *msg, size_t len, ...@@ -463,6 +463,10 @@ int rxrpc_recvmsg(struct socket *sock, struct msghdr *msg, size_t len,
flags, &copied); flags, &copied);
if (ret == -EAGAIN) if (ret == -EAGAIN)
ret = 0; ret = 0;
if (after(call->rx_top, call->rx_hard_ack) &&
call->rxtx_buffer[(call->rx_hard_ack + 1) & RXRPC_RXTX_BUFF_MASK])
rxrpc_notify_socket(call);
break; break;
default: default:
ret = 0; ret = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment