diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2026-05-28 15:02:06 +1000 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2026-06-04 06:35:02 +0200 |
| commit | 630e9bf1decf94618a036a020b7c920c8ab6126c (patch) | |
| tree | 304683fce52d4bd711e6e1e97f373911f5f01a1f /contrib/qemu | |
| parent | 4b2823784aab04a70dfc295b16fd6f0592955790 (diff) | |
| download | passt-630e9bf1decf94618a036a020b7c920c8ab6126c.tar passt-630e9bf1decf94618a036a020b7c920c8ab6126c.tar.gz passt-630e9bf1decf94618a036a020b7c920c8ab6126c.tar.bz2 passt-630e9bf1decf94618a036a020b7c920c8ab6126c.tar.lz passt-630e9bf1decf94618a036a020b7c920c8ab6126c.tar.xz passt-630e9bf1decf94618a036a020b7c920c8ab6126c.tar.zst passt-630e9bf1decf94618a036a020b7c920c8ab6126c.zip | |
tcp_splice: Remove never-invoked SO_RCVLOWAT logic
tcp_splice_forward() contains some logic to use the SO_RCVLOWAT
setsockopt(). This appears to be aimed at interrupt (epoll) mitigation, so
that we're not always waking for a socket that's getting frequent small
amounts of data.
However, the logic is never invoked, and hasn't been since at least
2022_07_14.b86cd00: it's conditional on
readlen > (long)c->tcp.pipe_size / 10
However, immediately before that we've invoked 'continue' if:
readlen >= (long)c->tcp_pipe_size * 10 / 100
which is a strictly weaker condition.
While it's possible we want to restore a working version of that interrupt
mitigation at some point, for the time being this logic just confuses the
picture and makes some other cleanups more awkward. We haven't had it
for over 3 years, so it's clearly not vital.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'contrib/qemu')
0 files changed, 0 insertions, 0 deletions
