[dpdk-dev] [PATCH v4 0/3] Add TSO on tunneling packet

2016-10-09 Thread Thomas Monjalon
2016-09-27 17:29, Ananyev, Konstantin: > > > From: Tan, Jianfeng > > > > Patch 1: mbuf: add Tx side tunneling type Patch 2: net/i40e: add TSO > > > > support on tunneling packet Patch 3: app/testpmd: fix Tx offload on > > > > tunneling packet > > Acked-by: Konstantin Ananyev > I think you need to

[dpdk-dev] [PATCH 0/6] vhost: add Tx zero copy support

2016-10-09 Thread Yuanhan Liu
On Mon, Aug 29, 2016 at 08:32:55AM +, Xu, Qian Q wrote: > I just ran a PVP test, nic receive packets then forwards to vhost PMD, and > virtio user interface. I didn't see any performance gains in this scenario. > All packet size from 64B to 1518B > performance haven't got benefit from this p

[dpdk-dev] [PATCH v2] drivers: prefix driver REGISTER macro with RTE PMD

2016-10-09 Thread Thomas Monjalon
2016-10-09 15:12, Shreyansh Jain: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-10-08 23:35, Shreyansh Jain: > > > +PMDINFO_TO_O = if grep -E 'RTE_PMD_REGISTER_PCI\([0-9a-zA-Z,_\. > > ]+\)|RTE_PMD_REGISTER_VDEV\([0-9a-zA-Z,_\. ]+\)' $<;\ > > > + then \ > > > > I don'

[dpdk-dev] [PATCH 0/6] vhost: add Tx zero copy support

2016-10-09 Thread linhaifeng
? 2016/8/23 16:10, Yuanhan Liu ??: > The basic idea of Tx zero copy is, instead of copying data from the > desc buf, here we let the mbuf reference the desc buf addr directly. Is there problem when push vlan to the mbuf which reference the desc buf addr directly? We know if guest use virtio_net(k

[dpdk-dev] [PATCH v3 7/7] net/vhost: add an option to enable dequeue zero copy

2016-10-09 Thread Yuanhan Liu
Add an option, dequeue-zero-copy, to enable this feature in vhost-pmd. Signed-off-by: Yuanhan Liu Reviewed-by: Maxime Coquelin --- drivers/net/vhost/rte_eth_vhost.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/net/vhost/rte_eth_vhost.c b/drivers/net/vhost/rte_eth_

[dpdk-dev] [PATCH v3 6/7] examples/vhost: add an option to enable dequeue zero copy

2016-10-09 Thread Yuanhan Liu
Add an option, --dequeue-zero-copy, to enable dequeue zero copy. One thing worth noting while using dequeue zero copy is the nb_tx_desc has to be small enough so that the eth driver will hit the mbuf free threshold easily and thus free mbuf more frequently. The reason behind that is, when dequeue

[dpdk-dev] [PATCH v3 5/7] vhost: add a flag to enable dequeue zero copy

2016-10-09 Thread Yuanhan Liu
Dequeue zero copy is disabled by default. Here add a new flag ``RTE_VHOST_USER_DEQUEUE_ZERO_COPY`` to explictily enable it. Signed-off-by: Yuanhan Liu Reviewed-by: Maxime Coquelin --- v2: - update release log - doc dequeue zero copy in detail --- doc/guides/prog_guide/vhost_lib.rst| 35

[dpdk-dev] [PATCH v3 4/7] vhost: add dequeue zero copy

2016-10-09 Thread Yuanhan Liu
The basic idea of dequeue zero copy is, instead of copying data from the desc buf, here we let the mbuf reference the desc buf addr directly. Doing so, however, has one major issue: we can't update the used ring at the end of rte_vhost_dequeue_burst. Because we don't do the copy here, an update of

[dpdk-dev] [PATCH v3 3/7] vhost: introduce last avail idx for dequeue

2016-10-09 Thread Yuanhan Liu
So far, we retrieve both the used ring and avail ring idx by the var last_used_idx; it won't be a problem because the used ring is updated immediately after those avail entries are consumed. But that's not true when dequeue zero copy is enabled, that used ring is updated only when the mbuf is cons

[dpdk-dev] [PATCH v3 2/7] vhost: get guest/host physical address mappings

2016-10-09 Thread Yuanhan Liu
So that we can convert a guest physical address to host physical address, which will be used in later Tx zero copy implementation. MAP_POPULATE is set while mmaping guest memory regions, to make sure the page tables are setup and then rte_mem_virt2phy() could yield proper physical address. Signed

[dpdk-dev] [PATCH v3 1/7] vhost: simplify memory regions handling

2016-10-09 Thread Yuanhan Liu
Due to history reason (that vhost-cuse comes before vhost-user), some fields for maintaining the vhost-user memory mappings (such as mmapped address and size, with those we then can unmap on destroy) are kept in "orig_region_map" struct, a structure that is defined only in vhost-user source file.

[dpdk-dev] [PATCH v3 0/7] vhost: add dequeue zero copy support

2016-10-09 Thread Yuanhan Liu
This patch set enables vhost dequeue zero copy. The majority work goes to patch 4: "vhost: add dequeue zero copy". The basic idea of dequeue zero copy is, instead of copying data from the desc buf, here we let the mbuf reference the desc buf addr directly. The major issue behind that is how and w

[dpdk-dev] [PATCH v2] drivers: prefix driver REGISTER macro with RTE PMD

2016-10-09 Thread Shreyansh Jain
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Sunday, October 09, 2016 1:52 AM > To: Shreyansh Jain > Cc: david.marchand at 6wind.com; dev at dpdk.org; nhorman at tuxdriver.com > Subject: Re: [PATCH v2] drivers: prefix driver REGISTE

[dpdk-dev] [PATCH] net/virtio: add missing driver name

2016-10-09 Thread Yuanhan Liu
On Fri, Oct 07, 2016 at 06:57:41PM +0530, Shreyansh Jain wrote: > On Friday 07 October 2016 06:33 PM, David Marchand wrote: > >The driver name has been lost with the eal rework. > >Restore it. > > > >Fixes: c830cb295411 ("drivers: use PCI registration macro") > > > >Signed-off-by: David Marchand >

[dpdk-dev] [RFC] libeventdev: event driven programming model framework for DPDK

2016-10-09 Thread Jerin Jacob
On Fri, Oct 07, 2016 at 10:40:03AM +, Hemant Agrawal wrote: > Hi Jerin/Narender, Hi Hemant, Thanks for the review. > > Thanks for the proposal and discussions. > > I agree with many of the comment made by Narender. Here are some > additional comments. > > 1. rte_event_sche

[dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue

2016-10-09 Thread Wang, Zhihong
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Wang, Zhihong > Sent: Wednesday, September 28, 2016 12:45 AM > To: Yuanhan Liu ; Jianbo Liu > > Cc: Maxime Coquelin ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue > > > >

[dpdk-dev] [PATCH v7] net/virtio: add set_mtu in virtio

2016-10-09 Thread Yuanhan Liu
On Thu, Sep 29, 2016 at 04:31:30PM -0400, Dey wrote: > /* > * dev_ops for virtio, bare necessities for basic operation > */ > @@ -677,7 +685,6 @@ static const struct eth_dev_ops virtio_eth_dev_ops = { > .allmulticast_enable = virtio_dev_allmulticast_enable, > .allmulticast_disa

[dpdk-dev] [PATCH] doc: announce ABI change for ethtool app enhance

2016-10-09 Thread Qiming Yang
This patch adds a notice that the ABI change for ethtool app to get the NIC firmware version in the 17.02 release. Signed-off-by: Qiming Yang --- doc/guides/rel_notes/deprecation.rst | 4 1 file changed, 4 insertions(+) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_no

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-10-09 Thread Yuanhan Liu
On Thu, Oct 06, 2016 at 02:37:27PM +, Xu, Qian Q wrote: > this function copy_desc_to_mbuf has changed on the dpdk-next-virtio repo. > Based on current dpdk-next-virtio repo, the commit ID is as below: > commit b4f7b43cd9d3b6413f41221051d03a23bc5f5fbe > Author: Zhiyong Yang > Date: Thu Sep