All Virtuozzo development lists (kernel + QEMU)
 help / color / mirror / Atom feed
From: Vladimir Riabchun <vladimir.riabchun@virtuozzo.com>
To: Eva Kurchatova <eva.kurchatova@virtuozzo.com>
Cc: devel@openvz.org
Subject: Re: [Devel] [PATCH vz10 4/7] selftests: net: make the veth GRO checks independent of host tunnels
Date: Mon, 21 Sep 2026 14:51:47 +0000	[thread overview]
Message-ID: <179000230736.645403.14575711999525636303.b4-review@b4> (raw)
In-Reply-To: <20260831224850.1642265-4-eva.kurchatova@virtuozzo.com>

On Tue, 01 Sep 2026 01:48:22 +0300, Eva Kurchatova <eva.kurchatova@virtuozzo.com> wrote:
> Three aggregation checks report 10 packets where they want 1 on a host
> that has a vxlan, and pass on a host that does not. The kernel is doing
> what it is written to do.
> 
> udp4_gro_receive() looks up the receiving socket only when
> udp_encap_needed_key is on, and that key is global: any UDP
> encapsulation socket on the machine, in any namespace, turns it on.
> udp_gro_receive() then aggregates for a packet that has no socket, or
> for a socket that asked for UDP GRO, and consults NETIF_F_GRO_UDP_FWD
> only in the first case. So with a vxlan somewhere on the box the lookup
> finds the test's own receiver, and rx-udp-gro-forwarding never gets a
> say.
> 
> [...]

Reviewed-by: Vladimir Riabchun <vladimir.riabchun@virtuozzo.com>

-- 
Vladimir Riabchun <vladimir.riabchun@virtuozzo.com>
_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

  reply	other threads:[~2026-09-21 14:53 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-31 22:48 [Devel] [PATCH vz10 1/7] ms/vxlan: do not reuse cached ip_hdr() value after skb_tunnel_check_pmtu() Eva Kurchatova
2026-08-31 22:48 ` [Devel] [PATCH vz10 2/7] selftests: net: run the fdb and bind tests in their own netns Eva Kurchatova
2026-09-21 14:51   ` Vladimir Riabchun
2026-08-31 22:48 ` [Devel] [PATCH vz10 3/7] selftests: net: skip what this kernel and iproute2 do not have Eva Kurchatova
2026-09-21 14:51   ` Vladimir Riabchun
2026-08-31 22:48 ` [Devel] [PATCH vz10 4/7] selftests: net: make the veth GRO checks independent of host tunnels Eva Kurchatova
2026-09-21 14:51   ` Vladimir Riabchun [this message]
2026-08-31 22:48 ` [Devel] [PATCH vz10 5/7] selftests: net: install pmtu.sh, the script pmtu_wrapper.sh runs Eva Kurchatova
2026-09-21 14:51   ` Vladimir Riabchun
2026-08-31 22:48 ` [Devel] [PATCH vz10 6/7] selftests: net: mark test_ingress_egress_chaining.sh executable Eva Kurchatova
2026-09-21 14:51   ` Vladimir Riabchun
2026-08-31 22:48 ` [Devel] [PATCH vz10 7/7] selftests: net: let the bridged PMTU tests take the ICMP they ask for Eva Kurchatova
2026-09-21 14:51   ` Vladimir Riabchun
2026-09-18 10:50 ` [Devel] [PATCH vz10 1/7] ms/vxlan: do not reuse cached ip_hdr() value after skb_tunnel_check_pmtu() Vasileios Almpanis
2026-09-21 14:51 ` Vladimir Riabchun

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=179000230736.645403.14575711999525636303.b4-review@b4 \
    --to=vladimir.riabchun@virtuozzo.com \
    --cc=devel@openvz.org \
    --cc=eva.kurchatova@virtuozzo.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.