Hi Mike,

Sorry for missing this letter for so log time.

We use ovs vxlan and geneve tunnel tso in DPU scenario.
Front virtio port in host set tso flag by ethtool -K ens26 tso on,
Our Smart NIC has its own cpu where ovs running on it. Ovs has two
Ports, one is vhost-user port, the other is Physical NIC which we want to 
Excete hardware tso. The vhost-user port receive packet from host virtio port.
According to virtio flag which is set by ethtool, vhost-user port set the 
RTE_MBUF_F_TX_TCP_SEG flag in rte mbuf ol_flags. When packets transmit 
From vhost-user port to physical nic, large packets will be cutted by mtu in 
Physical nic.

It is more Clearer by seeing the drawing in the attachment.


Thanks 
Dexia


-----邮件原件-----
发件人: Mike Pattrick <m...@redhat.com> 
发送时间: 2023年8月10日 21:48
收件人: Simon Horman <ho...@ovn.org>
抄送: Dexia Li <dexia...@jaguarmicro.com>; ovs-dev@openvswitch.org; 
i.maxim...@ovn.org
主题: Re: [ovs-dev] [PATCH] userspace: support vxlan and geneve tunnel tso

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/1574435117
> 0#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