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

Reply via email to