[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

[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

[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

[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

[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|

[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

[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

[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.

[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 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

[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.

[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] 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

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

2016-10-09 Thread Shreyansh Jain
All macros related to driver registeration renamed from DRIVER_* to RTE_PMD_* This includes: DRIVER_REGISTER_PCI -> RTE_PMD_REGISTER_PCI DRIVER_REGISTER_PCI_TABLE -> RTE_PMD_REGISTER_PCI_TABLE DRIVER_REGISTER_VDEV -> RTE_PMD_REGISTER_VDEV DRIVER_REGISTER_PARAM_STRING ->

[dpdk-dev] [PATCH v1] drivers: prefix driver REGISTER macro with RTE EAL

2016-10-09 Thread Shreyansh Jain
On 10/8/2016 11:31 PM, Shreyansh Jain wrote: > All macros related to driver registeration renamed from DRIVER_* > to RTE_PMD_* > > This includes: > > DRIVER_REGISTER_PCI -> RTE_PMD_REGISTER_PCI > DRIVER_REGISTER_PCI_TABLE -> RTE_PMD_REGISTER_PCI_TABLE > DRIVER_REGISTER_VDEV ->

[dpdk-dev] [PATCH v1] drivers: prefix driver REGISTER macro with RTE EAL

2016-10-09 Thread Shreyansh Jain
All macros related to driver registeration renamed from DRIVER_* to RTE_PMD_* This includes: DRIVER_REGISTER_PCI -> RTE_PMD_REGISTER_PCI DRIVER_REGISTER_PCI_TABLE -> RTE_PMD_REGISTER_PCI_TABLE DRIVER_REGISTER_VDEV -> RTE_PMD_REGISTER_VDEV DRIVER_REGISTER_PARAM_STRING ->