Re: [PATCH 0/8] splice() forwarding cleanups

[Stefano Brivio <sbrivio@redhat.com>]

On Thu, 28 May 2026 15:02:05 +1000
David Gibson <david@gibson.dropbear.id.au> wrote:

> As we discussed on an earlier call, while fixing bug 202 I noticed a
> number of warts in the surrounding splice() forwarding code.  Most are
> just things that are longer or harder to follow than they need to be,
> but in some cases there may be real (if unlikely to trigger) bugs.
> Here's a collection of fixes.
> 
> David Gibson (8):
>   tcp_splice: Remove never-invoked SO_RCVLOWAT logic
>   tcp_splice: Simplify EPOLLRDHUP / eof / FIN handling
>   tcp_splice: Improve EOF exit condition for the loop
>   tcp_splice: Remove goto from forwarding loop
>   tcp_splice: Simplify shutdown(2) handling
>   tcp_splice: Simplify / correct OUT_WAIT flag handling
>   tcp_splice: Remove questionable "optimisation" of pending bytes
>     tracking

Applied up to here (7/8).

>   tcp_splice: Exit forwarding earlier when stalled read side
> 
>  tcp_splice.c | 100 ++++++++++++++++-----------------------------------
>  1 file changed, 31 insertions(+), 69 deletions(-)

-- 
Stefano