Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-24 Thread Gao Zhenyu
; > > -Original Message- > > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > > boun...@openvswitch.org] On Behalf Of Gao Zhenyu > > Sent: Wednesday, August 23, 2017 4:12 PM > > To: Loftus, Ciara > > Cc: d...@openvswitch.org; us...@dpdk.org > &

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-24 Thread O Mahony, Billy
23, 2017 4:12 PM > To: Loftus, Ciara > Cc: d...@openvswitch.org; us...@dpdk.org > Subject: Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX > cksum in ovs-dpdk side > > Yes, maintaining only one implementation is resonable. > However making ovs-dpdk to support vhost

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-23 Thread Gao Zhenyu
Thanks, > Ciara > > > > > > > I have some other comments inline. > > > > > > Thanks, > > > Ciara > > “ > > > > > > > > From: Gao Zhenyu > > Date: Wednesday, August 16, 2017 at 6:38 AM > > To: "Loftus, C

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-23 Thread Loftus, Ciara
st 16, 2017 at 6:38 AM > To: "Loftus, Ciara" > Cc: "b...@ovn.org" , "Chandran, Sugesh" > , "ktray...@redhat.com" > , Darrell Ball , > "d...@openvswitch.org" > Subject: Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX &g

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-16 Thread Darrell Ball
c: "b...@ovn.org" , "Chandran, Sugesh" , "ktray...@redhat.com" , Darrell Ball , "d...@openvswitch.org" Subject: Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side Hi Loftus, I had submitted a new version, please see https://

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-16 Thread Gao Zhenyu
Hi Loftus, I had submitted a new version, please see https://patchwork.ozlabs.org/patch/802070/ It move the cksum to vhost receive side. Thanks Zhenyu Gao 2017-08-10 12:35 GMT+08:00 Gao Zhenyu : > I see, for flows in phy-phy setup, they should not be calculate cksum. > I will revise my pa

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-09 Thread Gao Zhenyu
I see, for flows in phy-phy setup, they should not be calculate cksum. I will revise my patch to do the cksum for vhost port only. I will send a new patch next week. Thanks Zhenyu Gao 2017-08-08 17:53 GMT+08:00 Loftus, Ciara : > > > > Hi Loftus, > > > > Thanks for testing and the comments! > > C

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-08 Thread Loftus, Ciara
> > Hi Loftus, > > Thanks for testing and the comments! > Can you show more details about your phy-vm-phy,phy-phy setup and > testing steps? Then I can reproduce it to see if I can solve this pps problem. You're welcome. I forgot to mention my tests were with 64B packets. For phy-phy the setup

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-04 Thread Gao Zhenyu
Hi Loftus, Thanks for testing and the comments! Can you show more details about your phy-vm-phy,phy-phy setup and testing steps? Then I can reproduce it to see if I can solve this pps problem. BTW, how about throughput, did you saw improvment? I would like to implement vhost->vhost part. Thanks

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-04 Thread Loftus, Ciara
> > Currently, the dpdk-vhost side in ovs doesn't support tcp/udp tx cksum. > So L4 packets's cksum were calculated in VM side but performance is not > good. > Implementing tcp/udp tx cksum in ovs-dpdk side improves throughput and > makes virtio-net frontend-driver support NETIF_F_SG as well > >

[ovs-dev] [PATCH v1] netdev-dpdk: Implement TCP/UDP TX cksum in ovs-dpdk side

2017-08-01 Thread Zhenyu Gao
Currently, the dpdk-vhost side in ovs doesn't support tcp/udp tx cksum. So L4 packets's cksum were calculated in VM side but performance is not good. Implementing tcp/udp tx cksum in ovs-dpdk side improves throughput and makes virtio-net frontend-driver support NETIF_F_SG as well Signed-off-by: Zh