On Wed, Aug 9, 2023 at 11:19 AM Simon Horman <ho...@ovn.org> wrote: > > On Wed, Aug 09, 2023 at 06:42:21PM +0800, Dexia Li via dev wrote: > > The netdev receiving packets will be encapsulated if output port > > is tunnel type. Only support userspace vxlan or geneve tunnel tso > > and inner csum offload. Calculate innter l2/l3/l4 len before > > encapsulated and calculate out l2/l3 len after encapsulated. > > > > This func only support userspace tunnel tso so linux devices is > > not included. > > > > Signed-off-by: Dexia Li <dexia...@jaguarmicro.com> > > Hi Dexia, > > unfortunately I'm seeing the following check fail in CI with this patch > applied. > > - "dpif-netdev - check tx packet checksum offloading" > > Link: > https://github.com/ovsrobot/ovs/actions/runs/5808145405/job/15744351170#step:11:5577
Hello Dexia, Thank you for the contribution, TSO support for vxlan and geneve tunnels is really lacking right now. I haven't had a chance to thoroughly look at the code; but I tried testing out this patch in a vxlan setup and found that the checksums weren't being calculated. You note in the commit message that support is only extended to a limited usecase. Could you explain more about the setup that this works in? Thank you, M > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev