Re: [ovs-dev] [PATCH] tunnel: Do not carry source port from a previous tunnel.

2023-12-02 Thread Vladislav Odintsov
Hi Ilya, thanks for the patch! I’ve validated it against my setup with geneve tunnels hairpin. With [0] applied, tc offloading works as expected: no warnings/errors in ovs-vswitchd.log, and only first packets of a flow reach system/CPU/are visible in tcpdump. Also, ovs-appctl dpctl/dump-flows

Re: [ovs-dev] [PATCH v2] netdev-offload-tc: Fix offload of tunnel key tp_src.

2023-12-02 Thread Vladislav Odintsov
Hi Ilya, thanks for the added test! I’ve got one question about it, psb. > On 2 Dec 2023, at 02:06, Ilya Maximets wrote: > > There is no TCA_TUNNEL_KEY_ENC_SRC_PORT in the kernel, so the offload > should not be attempted if OVS_TUNNEL_KEY_ATTR_TP_SRC is requested > by OVS. Current code just