diff options
| author |   Eric Dumazet <edumazet@google.com> | 2018-04-16 10:33:35 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-04-16 18:26:37 -0400 |
| commit | d1361840f8c519eaee9a78ffe09e4f0a1b586846 (patch) | |
| tree | 86a904ade99a93544e0817cda7dc842b12f9b833 /net/core/sock.c | |
| parent | tc-testing: add sample action tests (diff) | |
| download | linux-dev-d1361840f8c519eaee9a78ffe09e4f0a1b586846.tar.xz linux-dev-d1361840f8c519eaee9a78ffe09e4f0a1b586846.zip | |
tcp: fix SO_RCVLOWAT and RCVBUF autotuning
Applications might use SO_RCVLOWAT on TCP socket hoping to receive
one [E]POLLIN event only when a given amount of bytes are ready in socket
receive queue.
Problem is that receive autotuning is not aware of this constraint,
meaning sk_rcvbuf might be too small to allow all bytes to be stored.
Add a new (struct proto_ops)->set_rcvlowat method so that a protocol
can override the default setsockopt(SO_RCVLOWAT) behavior.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/core/sock.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/net/core/sock.c b/net/core/sock.c index 6444525f610c..b2c3db169ca1 100644 --- a/net/core/sock.c +++ b/net/core/sock.c @@ -905,7 +905,10 @@ set_rcvbuf: case SO_RCVLOWAT: if (val < 0) val = INT_MAX; - sk->sk_rcvlowat = val ? : 1; + if (sock->ops->set_rcvlowat) + ret = sock->ops->set_rcvlowat(sk, val); + else + sk->sk_rcvlowat = val ? : 1; break; case SO_RCVTIMEO: |