All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jakub Kicinski <kuba@kernel.org>
To: Paolo Abeni <pabeni@redhat.com>
Cc: Neal Cardwell <ncardwell@google.com>,
	Matthieu Baerts <matttbe@kernel.org>,
	Eric Dumazet <edumazet@google.com>,
	Simon Horman <horms@kernel.org>,
	Kuniyuki Iwashima <kuniyu@google.com>,
	Willem de Bruijn <willemb@google.com>,
	netdev@vger.kernel.org, eric.dumazet@gmail.com,
	"David S . Miller" <davem@davemloft.net>
Subject: Re: [PATCH net-next 0/8] tcp: receiver changes
Date: Tue, 15 Jul 2025 07:54:40 -0700	[thread overview]
Message-ID: <20250715075440.49a35d7c@kernel.org> (raw)
In-Reply-To: <9fb159c3-0151-49ac-91bf-1be8301bdf18@redhat.com>

On Tue, 15 Jul 2025 15:52:33 +0200 Paolo Abeni wrote:
> On 7/15/25 3:33 PM, Jakub Kicinski wrote:
> > On Tue, 15 Jul 2025 06:28:29 -0700 Jakub Kicinski wrote:  
> >> # (null):17: error handling packet: timing error: expected outbound packet at 0.074144 sec but happened at -1752585909.757339 sec; tolerance 0.004000 sec
> >> # script packet:  0.074144 S. 0:0(0) ack 1 <mss 1460,nop,wscale 0>
> >> # actual packet: -1752585909.757339 S.0 0:0(0) ack 1 <mss 1460,nop,wscale 0>  
> > 
> > This is definitely compiler related, I rebuilt with clang and the build
> > error goes away. Now I get a more sane failure:
> > 
> > # tcp_rcv_big_endseq.pkt:41: error handling packet: timing error: expected outbound packet at 1.230105 sec but happened at 1.190101 sec; tolerance 0.005046 sec
> > # script packet:  1.230105 . 1:1(0) ack 54001 win 0 
> > # actual packet:  1.190101 . 1:1(0) ack 54001 win 0 
> > 
> > $ gcc --version
> > gcc (GCC) 15.1.1 20250521 (Red Hat 15.1.1-2)
> > 
> > I don't understand why the ack is supposed to be delayed, should we
> > just do this? (I think Eric is OOO, FWIW)
> > 
> > diff --git a/tools/testing/selftests/net/packetdrill/tcp_rcv_big_endseq.pkt b/tools/testing/selftests/net/packetdrill/tcp_rcv_big_endseq.pkt
> > index 7e170b94fd36..3848b419e68c 100644
> > --- a/tools/testing/selftests/net/packetdrill/tcp_rcv_big_endseq.pkt
> > +++ b/tools/testing/selftests/net/packetdrill/tcp_rcv_big_endseq.pkt
> > @@ -38,7 +38,7 @@
> >  
> >  // If queue is empty, accept a packet even if its end_seq is above wup + rcv_wnd
> >    +0 < P. 4001:54001(50000) ack 1 win 257
> > -  +.040 > .  1:1(0) ack 54001 win 0
> > +  +0 > .  1:1(0) ack 54001 win 0
> >  
> >  // Check LINUX_MIB_BEYOND_WINDOW has been incremented 3 times.
> >  +0 `nstat | grep TcpExtBeyondWindow | grep -q " 3 "`  
> 
> The above looks sane to me, but I Neal or Willem ack would be appreciated.

Posted officially here to get it queued to the CI already:
https://lore.kernel.org/all/20250715142849.959444-1-kuba@kernel.org/

  reply	other threads:[~2025-07-15 14:54 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-11 11:39 [PATCH net-next 0/8] tcp: receiver changes Eric Dumazet
2025-07-11 11:39 ` [PATCH net-next 1/8] tcp: do not accept packets beyond window Eric Dumazet
2025-07-12 20:52   ` Kuniyuki Iwashima
2025-07-15  1:38   ` Jakub Kicinski
2025-07-11 11:40 ` [PATCH net-next 2/8] tcp: add LINUX_MIB_BEYOND_WINDOW Eric Dumazet
2025-07-12 20:55   ` Kuniyuki Iwashima
2025-07-11 11:40 ` [PATCH net-next 3/8] selftests/net: packetdrill: add tcp_rcv_big_endseq.pkt Eric Dumazet
2025-07-12 20:58   ` Kuniyuki Iwashima
2025-07-11 11:40 ` [PATCH net-next 4/8] tcp: call tcp_measure_rcv_mss() for ooo packets Eric Dumazet
2025-07-12 21:11   ` Kuniyuki Iwashima
2025-07-11 11:40 ` [PATCH net-next 5/8] selftests/net: packetdrill: add tcp_ooo_rcv_mss.pkt Eric Dumazet
2025-07-12 21:42   ` Kuniyuki Iwashima
2025-07-11 11:40 ` [PATCH net-next 6/8] tcp: add const to tcp_try_rmem_schedule() and sk_rmem_schedule() skb Eric Dumazet
2025-07-12 21:43   ` Kuniyuki Iwashima
2025-07-11 11:40 ` [PATCH net-next 7/8] tcp: stronger sk_rcvbuf checks Eric Dumazet
2025-07-12 21:54   ` Kuniyuki Iwashima
2025-12-15 10:19   ` Christian Ebner
2025-12-18  9:31     ` Christian Ebner
2025-12-18 10:10       ` Eric Dumazet
2025-12-18 12:28         ` Christian Ebner
2025-12-18 13:19           ` Eric Dumazet
2025-12-18 14:58             ` Christian Ebner
2025-12-19  8:23               ` Eric Dumazet
2025-12-19  8:45                 ` Eric Dumazet
2025-12-19 10:00                   ` Christian Ebner
2025-12-19 10:12                     ` Eric Dumazet
2026-01-25 21:11   ` [regression] " Simon Baatz
2025-07-11 11:40 ` [PATCH net-next 8/8] selftests/net: packetdrill: add tcp_rcv_toobig.pkt Eric Dumazet
2025-07-12 21:57   ` Kuniyuki Iwashima
2025-07-15  2:20 ` [PATCH net-next 0/8] tcp: receiver changes patchwork-bot+netdevbpf
2025-07-15  8:25 ` Paolo Abeni
2025-07-15  9:21   ` Matthieu Baerts
2025-07-15 10:14     ` Paolo Abeni
2025-07-15 10:40       ` Matthieu Baerts
2025-07-15 13:28       ` Jakub Kicinski
2025-07-15 13:33         ` Jakub Kicinski
2025-07-15 13:52           ` Paolo Abeni
2025-07-15 14:54             ` Jakub Kicinski [this message]
2025-07-15 14:48           ` Kuniyuki Iwashima
2025-07-15 13:50         ` Paolo Abeni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250715075440.49a35d7c@kernel.org \
    --to=kuba@kernel.org \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=eric.dumazet@gmail.com \
    --cc=horms@kernel.org \
    --cc=kuniyu@google.com \
    --cc=matttbe@kernel.org \
    --cc=ncardwell@google.com \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=willemb@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.