* [PATCH nf] netfilter: flowtable: fix pmtu discovery in nf_flow_tunnel_ip6ip6_push()
@ 2026-10-19 10:16 Lorenzo Bianconi
0 siblings, 0 replies; 3+ messages in thread
From: Lorenzo Bianconi @ 2026-10-19 10:16 UTC (permalink / raw)
To: Pablo Neira Ayuso, Florian Westphal, Phil Sutter, David S. Miller,
Eric Dumazet, Jakub Kicinski, Paolo Abeni, Simon Horman,
Lorenzo Bianconi
Cc: netfilter-devel, coreteam, netdev, Lorenzo Bianconi
nf_flow_tunnel_ip6ip6_push() invokes skb_scrub_packet() before updating
the PMTU. skb_scrub_packet() drops the skb dst, so
skb_dst_update_pmtu_no_confirm() finds no dst and the PMTU update is
silently lost.
Move the PMTU update before skb_scrub_packet() so the inner route is
updated with the MTU advertised by the tunnel dst, as done in
ip6_tnl_xmit().
Fixes: 93cf357fa797 ("netfilter: flowtable: Add IP6IP6 tx sw acceleration")
Suggested-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@oss.qualcomm.com>
---
net/netfilter/nf_flow_table_ip.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/netfilter/nf_flow_table_ip.c b/net/netfilter/nf_flow_table_ip.c
index c8c29a9a1684..12faf85ee374 100644
--- a/net/netfilter/nf_flow_table_ip.c
+++ b/net/netfilter/nf_flow_table_ip.c
@@ -664,10 +664,10 @@ static int nf_flow_tunnel_ip6ip6_push(struct net *net, struct sk_buff *skb,
if (err)
return err;
- skb_scrub_packet(skb, true);
mtu = dst_mtu(&rt->dst) - sizeof(*ip6h);
mtu = max(mtu, IPV6_MIN_MTU);
skb_dst_update_pmtu_no_confirm(skb, mtu);
+ skb_scrub_packet(skb, true);
skb_push(skb, sizeof(*ip6h));
skb_reset_network_header(skb);
---
base-commit: ee2d4c2d7cc18a2cd504044a75ea614e720248eb
change-id: 20261019-nf_flow_tunnel_ip6ip6_push-fix-pmtu-ff81656e32fc
Best regards,
--
Lorenzo Bianconi <lorenzo.bianconi@oss.qualcomm.com>
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH nf] netfilter: flowtable: fix pmtu discovery in nf_flow_tunnel_ip6ip6_push()
@ 2026-10-19 10:19 netdev-bot+sinfo
2026-10-19 10:46 ` Pablo Neira Ayuso
0 siblings, 1 reply; 3+ messages in thread
From: netdev-bot+sinfo @ 2026-10-19 10:19 UTC (permalink / raw)
To: Lorenzo Bianconi
Cc: Pablo Neira Ayuso, Florian Westphal, Phil Sutter, David S. Miller,
Eric Dumazet, Jakub Kicinski, Paolo Abeni, Simon Horman,
Lorenzo Bianconi, netfilter-devel, coreteam, netdev
Hi!
This is an automated message. This series looks like a fix, but its
commit messages seem to be missing some information:
- How the issue was discovered, e.g. hit in production, hit during
development, syzbot report, manual code inspection, LLM or static
analysis tool scan.
- Whether the issue was actually triggered, or is only theoretical
(e.g. found by code inspection). If it was triggered please include
the symptoms, like the stack trace or error messages.
Please do not repost the series just to address the above. Instead,
reply to this email with the missing information, so that reviewers
can take it into account. If the series needs another revision for
other reasons, please include the information in the commit messages
then.
The evaluation is done by an LLM so it may be wrong, if you think
that is the case please reply and explain.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH nf] netfilter: flowtable: fix pmtu discovery in nf_flow_tunnel_ip6ip6_push()
2026-10-19 10:19 [PATCH nf] netfilter: flowtable: fix pmtu discovery in nf_flow_tunnel_ip6ip6_push() netdev-bot+sinfo
@ 2026-10-19 10:46 ` Pablo Neira Ayuso
0 siblings, 0 replies; 3+ messages in thread
From: Pablo Neira Ayuso @ 2026-10-19 10:46 UTC (permalink / raw)
To: netdev-bot+sinfo
Cc: Lorenzo Bianconi, Florian Westphal, Phil Sutter, David S. Miller,
Eric Dumazet, Jakub Kicinski, Paolo Abeni, Simon Horman,
Lorenzo Bianconi, netfilter-devel, coreteam, netdev
On Mon, Oct 19, 2026 at 10:19:07AM +0000, netdev-bot+sinfo@kernel.org wrote:
> Hi!
>
> This is an automated message. This series looks like a fix, but its
> commit messages seem to be missing some information:
>
> - How the issue was discovered, e.g. hit in production, hit during
> development, syzbot report, manual code inspection, LLM or static
> analysis tool scan.
>
> - Whether the issue was actually triggered, or is only theoretical
> (e.g. found by code inspection). If it was triggered please include
> the symptoms, like the stack trace or error messages.
Sashiko reported this.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-10-19 10:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-19 10:19 [PATCH nf] netfilter: flowtable: fix pmtu discovery in nf_flow_tunnel_ip6ip6_push() netdev-bot+sinfo
2026-10-19 10:46 ` Pablo Neira Ayuso
-- strict thread matches above, loose matches on Subject: below --
2026-10-19 10:16 Lorenzo Bianconi
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.