Re: [ovs-dev] The Patch netdev-dpdk: add TSO support for vhost-user ports

2016-10-18 Thread Michael Qiu
[+cc DPDK jianfeng] For the gap, is it been fixed by below commit? commit c59faf3fe8fe6dbd99e8cd9d703839728264c5ae Author: Jianfeng Tan Date: Mon Aug 1 03:56:54 2016 + net/i40e: support TSO on tunneling packet I just tested your vlan patch, the same thing happens as vxlan tunnel T

Re: [ovs-dev] The Patch netdev-dpdk: add TSO support for vhost-user ports

2016-10-17 Thread Kavanagh, Mark B
> >OK, Thanks for your info. > >BTW, you mentioned the gap, is it the tunneling tso can't work correctly for >multi-segments >mbuf, and after some packets, guest is unable to xmit packets(dpdk driver >failed to work any >more)? At the time, the gap was that the tunnel header was not replicated a

Re: [ovs-dev] The Patch netdev-dpdk: add TSO support for vhost-user ports

2016-10-14 Thread 网易
OK, Thanks for your info. BTW, you mentioned the gap, is it the tunneling tso can't work correctly for multi-segments mbuf, and after some packets, guest is unable to xmit packets(dpdk driver failed to work any more)? Thanks, Michael 2016年10月14日,下午5:15,Kavanagh, Mark B : >> >> >> Hi, Mark

Re: [ovs-dev] The Patch netdev-dpdk: add TSO support for vhost-user ports

2016-10-14 Thread Kavanagh, Mark B
> >Hi, Mark > > >I have reviewed your RFC patch, I found one place could have issue, see >below: > > >+if (v > ETHER_MAX_LEN) { >+b->mbuf.data_len = >+(uint16_t)(b->mbuf.buf_len - b->mbuf.data_off); >+} else { >+b->mbuf.data_len = (uint16_t)v; >+} >+ > >Here,

Re: [ovs-dev] The Patch netdev-dpdk: add TSO support for vhost-user ports

2016-10-14 Thread Kavanagh, Mark B
> > >Hi, Mark > > >OK, once it's ready, pls let me know and I'm glade to help to test it. > > >BTW, you mentioned the gap, is that the TSO and CSUM of tunnel offload? >I'm familiar with it, and I hope I could do something on it if possible. Yes, that's what it was. Commits have recently been int

Re: [ovs-dev] The Patch netdev-dpdk: add TSO support for vhost-user ports

2016-10-13 Thread Michael Qiu
Hi, Mark I have reviewed your RFC patch, I found one place could have issue, see below: +if (v > ETHER_MAX_LEN) { +b->mbuf.data_len = +(uint16_t)(b->mbuf.buf_len - b->mbuf.data_off); +} else { +b->mbuf.data_len = (uint16_t)v; +} + Here, what will happ

Re: [ovs-dev] The Patch netdev-dpdk: add TSO support for vhost-user ports

2016-10-11 Thread Michael Qiu
Hi, Mark OK, once it's ready, pls let me know and I'm glade to help to test it. BTW, you mentioned the gap, is that the TSO and CSUM of tunnel offload? I'm familiar with it, and I hope I could do something on it if possible. 2016/10/11 22:34, Kavanagh, Mark B : Hi, all This patch is v

Re: [ovs-dev] The Patch netdev-dpdk: add TSO support for vhost-user ports

2016-10-11 Thread Kavanagh, Mark B
> >Hi, all > > >This patch is very important for users want to improve the performance >of the large packets. > > >But you know, in data center, lots of networks using vxlan, so if it >supports vxlan, then it will be very useful. > > >Would you guys has a plan to support it? I would like to help t