* [PATCH net-next] selftests/net: packetdrill: add CONFIG_MAX_SKB_FRAGS=17
@ 2026-03-24 14:25 Eric Dumazet
2026-03-24 14:53 ` Willem de Bruijn
0 siblings, 1 reply; 2+ messages in thread
From: Eric Dumazet @ 2026-03-24 14:25 UTC (permalink / raw)
To: David S . Miller, Jakub Kicinski, Paolo Abeni
Cc: Simon Horman, Neal Cardwell, Kuniyuki Iwashima, Willem de Bruijn,
netdev, eric.dumazet, Eric Dumazet
Before adding tests about RWIN, enforce the default linux value
for the number of fragments per skb.
With 17 fragments, sizeof(skb_shared_info) == 320.
With 45 fragments, sizeof(skb_shared_info) == 768.
Signed-off-by: Eric Dumazet <edumazet@google.com>
---
tools/testing/selftests/net/packetdrill/config | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/testing/selftests/net/packetdrill/config b/tools/testing/selftests/net/packetdrill/config
index c4a19a78552111df690d2c97c0f3a8abe2176ada..8c739db4752c3abaaa8862ea2aeb661a2d5f0575 100644
--- a/tools/testing/selftests/net/packetdrill/config
+++ b/tools/testing/selftests/net/packetdrill/config
@@ -1,6 +1,7 @@
CONFIG_HZ=1000
CONFIG_HZ_1000=y
CONFIG_IPV6=y
+CONFIG_MAX_SKB_FRAGS=17
CONFIG_NET_NS=y
CONFIG_NET_SCH_FIFO=y
CONFIG_NET_SCH_FQ=y
--
2.53.0.983.g0bb29b3bc5-goog
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net-next] selftests/net: packetdrill: add CONFIG_MAX_SKB_FRAGS=17
2026-03-24 14:25 [PATCH net-next] selftests/net: packetdrill: add CONFIG_MAX_SKB_FRAGS=17 Eric Dumazet
@ 2026-03-24 14:53 ` Willem de Bruijn
0 siblings, 0 replies; 2+ messages in thread
From: Willem de Bruijn @ 2026-03-24 14:53 UTC (permalink / raw)
To: Eric Dumazet, David S . Miller, Jakub Kicinski, Paolo Abeni
Cc: Simon Horman, Neal Cardwell, Kuniyuki Iwashima, Willem de Bruijn,
netdev, eric.dumazet, Eric Dumazet
Eric Dumazet wrote:
> Before adding tests about RWIN, enforce the default linux value
> for the number of fragments per skb.
>
> With 17 fragments, sizeof(skb_shared_info) == 320.
>
> With 45 fragments, sizeof(skb_shared_info) == 768.
>
> Signed-off-by: Eric Dumazet <edumazet@google.com>
> ---
> tools/testing/selftests/net/packetdrill/config | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/testing/selftests/net/packetdrill/config b/tools/testing/selftests/net/packetdrill/config
> index c4a19a78552111df690d2c97c0f3a8abe2176ada..8c739db4752c3abaaa8862ea2aeb661a2d5f0575 100644
> --- a/tools/testing/selftests/net/packetdrill/config
> +++ b/tools/testing/selftests/net/packetdrill/config
> @@ -1,6 +1,7 @@
> CONFIG_HZ=1000
> CONFIG_HZ_1000=y
> CONFIG_IPV6=y
> +CONFIG_MAX_SKB_FRAGS=17
> CONFIG_NET_NS=y
> CONFIG_NET_SCH_FIFO=y
> CONFIG_NET_SCH_FQ=y
Why do we need to make this explicit, if it is the default?
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-03-24 14:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-24 14:25 [PATCH net-next] selftests/net: packetdrill: add CONFIG_MAX_SKB_FRAGS=17 Eric Dumazet
2026-03-24 14:53 ` Willem de Bruijn
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox