Re: [PATCH stable-5.4 4/4] tcp: avoid too many retransmit packets

[Greg KH <gregkh@linuxfoundation.org>]

On Tue, Jul 16, 2024 at 08:40:40PM +0800, Jason Xing wrote:
> On Tue, Jul 16, 2024 at 7:10 PM Miguel Ojeda <ojeda@kernel.org> wrote:
> >
> > Hi Greg, Eric, all,
> >
> > I noticed this in stable-rc/queue and stable-rc/linux- for 6.1 and 6.6:
> >
> >     net/ipv4/tcp_timer.c:472:7: error: variable 'rtx_delta' is uninitialized when used here [-Werror,-Wuninitialized]
> >                     if (rtx_delta > user_timeout)
> >                         ^~~~~~~~~
> >     net/ipv4/tcp_timer.c:464:15: note: initialize the variable 'rtx_delta' to silence this warning
> >             u32 rtx_delta;
> >                         ^
> >                         = 0
> >
> > I hope that helps!
> 
> Thanks for the report!
> 
> I think it missed one small snippet of code from [1] compared to the
> latest kernel. We can init this part before using it, something like
> this:
> 
> +       rtx_delta = (u32)msecs_to_jiffies(tcp_time_stamp(tp) -
> +                       (tp->retrans_stamp ?: tcp_skb_timestamp(skb)));
> 
> Note: fully untested.
> 
> Since Eric is very busy, I decided to check and provide some useful
> information here.

Thanks all, this was probably due to my manual backporting here, let me
go check what went wrong...