[dpdk-dev] [PATCH] hash: fix ring size

2016-09-27 Thread Pablo de Lara
Ring stores the free slots available to be used in the key table. The ring size was being increased by 1, because of the dummy slot, used for key misses, but this is not actually stored in the ring, so there is no need to increase it. Fixes: 5915699153d7 ("hash: fix scaling by reducing contention"

[dpdk-dev] [PATCH v5 0/2] net/i40e: enable 25G device

2016-09-27 Thread Zhang, Qi Z
Hi Ferruh: > -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, September 27, 2016 12:15 AM > To: Zhang, Qi Z ; Wu, Jingjing intel.com>; > Zhang, Helin > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v5 0/2] net/i40e: enable 25G device > > Hi Qi, > > On 9/26/2016 12:25 AM

[dpdk-dev] [PATCH] testpmd: fix fdir command on MAC and tunnel modes

2016-09-27 Thread Wu, Jingjing
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Frederico.Cadete- > ext at oneaccess-net.com > Sent: Tuesday, August 23, 2016 5:11 PM > To: dev at dpdk.org > Cc: frederico at cadete.eu; Frederico Cadete > Subject: [dpdk-dev] [PATCH] testpmd: fix fdir command

[dpdk-dev] [PATCH v3 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-27 Thread Yuanhan Liu
On Mon, Sep 26, 2016 at 04:53:05PM +0200, Marcin Kerlin wrote: > diff --git a/lib/librte_ether/rte_ether_version.map > b/lib/librte_ether/rte_ether_version.map > index 45ddf44..34e1109 100644 > --- a/lib/librte_ether/rte_ether_version.map > +++ b/lib/librte_ether/rte_ether_version.map > @@ -139,3

[dpdk-dev] [PATCH 1/2] vhost: enable any layout feature

2016-09-27 Thread Yuanhan Liu
On Mon, Sep 26, 2016 at 10:24:55PM +0300, Michael S. Tsirkin wrote: > On Mon, Sep 26, 2016 at 11:01:58AM -0700, Stephen Hemminger wrote: > > I assume that if using Version 1 that the bit will be ignored Yes, but I will just quote what you just said: what if the guest virtio device is a legacy devi

[dpdk-dev] [PATCH v3] vhost: Add indirect descriptors support to the TX path

2016-09-27 Thread Yuanhan Liu
On Fri, Sep 23, 2016 at 10:28:23AM +0200, Maxime Coquelin wrote: > + if (vq->desc[desc_indexes[i]].flags & VRING_DESC_F_INDIRECT) { > + desc = (struct vring_desc *)gpa_to_vva(dev, As mentioned before, this would break 32 bit OS build. It should be (struct v

[dpdk-dev] [PATCH v6 0/2] net/i40e: enable 25G device

2016-09-27 Thread Zhang Qi
The patch enable devices that support 25G link speed. It is based on previous 25G base driver update: http://dpdk.org/ml/archives/dev/2016-August/045569.html v6: - rebase to dpdk-next-net/rel_16_11. v5: - no chnage, just for adding the missing change log. v4: - correct return value check of i40e

[dpdk-dev] [PATCH v6 1/2] net/i40e: use PHY type to check PHY capability

2016-09-27 Thread Zhang Qi
In previous code, we use device ID to check PHY capability which is not extensible since there is always new device be added. Now we use PHY type to detect PHY capability. PHY type encoded all link speed the device support, it is read out through aq command "get_phy_capability" at init stage. Sign

[dpdk-dev] [PATCH v6 2/2] net/i40e: enable 25G device

2016-09-27 Thread Zhang Qi
Add code branch for 25G link speed, so 25G device will be functional. Signed-off-by: Zhang Qi --- v6: - rebase to dpdk-next-net/rel_16_11. v3: - add PHY type check macro for 25G devices. - use PHY type to check if device support 25G. drivers/net/i40e/i40e_ethdev.c | 22 ++

[dpdk-dev] [PATCH v7 0/9] enable lpm, acl and other missing libraries in ppc64le

2016-09-27 Thread gowrishankar muthukrishnan
On Friday 09 September 2016 09:29 PM, Thomas Monjalon wrote: >>> v7 changes: >>> - removed enforcing cache alignment for table hash structs and >>>instead check only for multiples of 64 bytes. >>> >> Acked-by: Cristian Dumitrescu > Applied, thanks > > We now have some AltiVec code in DPDK! > >

[dpdk-dev] [dpdk-stable] [PATCH v7 0/9] enable lpm, acl and other missing libraries in ppc64le

2016-09-27 Thread Yuanhan Liu
On Tue, Sep 27, 2016 at 12:16:32PM +0530, gowrishankar muthukrishnan wrote: > On Friday 09 September 2016 09:29 PM, Thomas Monjalon wrote: > >>>v7 changes: > >>>- removed enforcing cache alignment for table hash structs and > >>> instead check only for multiples of 64 bytes. > >>> > >>Acked-by: C

[dpdk-dev] [PATCH v3] vhost: Add indirect descriptors support to the TX path

2016-09-27 Thread Maxime Coquelin
On 09/27/2016 06:15 AM, Yuanhan Liu wrote: > On Fri, Sep 23, 2016 at 10:28:23AM +0200, Maxime Coquelin wrote: >> +if (vq->desc[desc_indexes[i]].flags & VRING_DESC_F_INDIRECT) { >> +desc = (struct vring_desc *)gpa_to_vva(dev, > > As mentioned before, this would brea

[dpdk-dev] [PATCH] cryptodev: fix compilation error in SUSE 11 SP2

2016-09-27 Thread Adrien Mazarguil
On Mon, Sep 26, 2016 at 10:50:35PM +0100, Pablo de Lara wrote: > This commit fixes following build error, which happens in SUSE 11 SP2, > with gcc 4.5.1: > > In file included from lib/librte_cryptodev/rte_cryptodev.c:71:0: > lib/librte_cryptodev/rte_cryptodev_pmd.h:76:7: > error: flexible array me

[dpdk-dev] [PATCH v7 0/2] net/i40e: enable 25G device

2016-09-27 Thread Zhang Qi
The patch enable devices that support 25G link speed. It is based on previous 25G base driver update: http://dpdk.org/ml/archives/dev/2016-August/045569.html v7: - 25G device is XXV710 not XXL710, correct it. v6: - rebase to dpdk-next-net/rel_16_11. v5: - no chnage, just for adding the missing

[dpdk-dev] [PATCH v7 1/2] net/i40e: use PHY type to check PHY capability

2016-09-27 Thread Zhang Qi
In previous code, we use device ID to check PHY capability which is not extensible since there is always new device be added. Now we use PHY type to detect PHY capability. PHY type encoded all link speed the device support, it is read out through aq command "get_phy_capability" at init stage. Sign

[dpdk-dev] [PATCH v7 2/2] net/i40e: enable 25G device

2016-09-27 Thread Zhang Qi
Add code branch for 25G link speed, so 25G device will be functional. Signed-off-by: Zhang Qi --- v6: - rebase to dpdk-next-net/rel_16_11. v3: - add PHY type check macro for 25G devices. - use PHY type to check if device support 25G. drivers/net/i40e/i40e_ethdev.c | 22 ++

[dpdk-dev] [PATCH v4] vhost: Add indirect descriptors support to the TX path

2016-09-27 Thread Maxime Coquelin
Indirect descriptors are usually supported by virtio-net devices, allowing to dispatch a larger number of requests. When the virtio device sends a packet using indirect descriptors, only one slot is used in the ring, even for large packets. The main effect is to improve the 0% packet loss benchma

[dpdk-dev] [PATCH v7 0/2] net/i40e: enable 25G device

2016-09-27 Thread Wu, Jingjing
> -Original Message- > From: Zhang, Qi Z > Sent: Tuesday, September 27, 2016 9:37 AM > To: Wu, Jingjing; Zhang, Helin > Cc: dev at dpdk.org; Zhang, Qi Z > Subject: [PATCH v7 0/2] net/i40e: enable 25G device > > The patch enable devices that support 25G link speed. > It is based on previo

[dpdk-dev] [PATCH] app/test: remove unnecessary hexdump from test_cryptodev.c

2016-09-27 Thread Arek Kusztal
This patch removes unnecesssary hexdump calls from GMAC related functions in test_cryptodev.c which hinder compilation when RTE_LOG_DEBUG is specified Signed-off-by: Arek Kusztal --- app/test/test_cryptodev.c | 4 1 file changed, 4 deletions(-) diff --git a/app/test/test_cryptodev.c b/app/

[dpdk-dev] [PATCH] app/test: remove unnecessary hexdump from test_cryptodev.c

2016-09-27 Thread Jain, Deepak K
> -Original Message- > From: Kusztal, ArkadiuszX > Sent: Tuesday, September 27, 2016 10:21 AM > To: dev at dpdk.org > Cc: Trahe, Fiona ; Jain, Deepak K > ; De Lara Guarch, Pablo > ; Griffin, John intel.com>; > Kusztal, ArkadiuszX > Subject: [PATCH] app/test: remove unnecessary hexdump fr

[dpdk-dev] [PATCH v3 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-27 Thread Kerlin, MarcinX
Hi Liu > -Original Message- > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > Sent: Tuesday, September 27, 2016 5:07 AM > To: Kerlin, MarcinX > Cc: dev at dpdk.org; De Lara Guarch, Pablo intel.com>; > thomas.monjalon at 6wind.com > Subject: Re: [dpdk-dev] [PATCH v3 1/2] librt

[dpdk-dev] [PATCH] app/test: fix compilation error when debug mode enabled

2016-09-27 Thread Kusztal, ArkadiuszX
> -Original Message- > From: De Lara Guarch, Pablo > Sent: Monday, September 26, 2016 8:14 PM > To: Kusztal, ArkadiuszX ; dev at dpdk.org > Cc: Trahe, Fiona ; Jain, Deepak K > ; Griffin, John > Subject: RE: [PATCH] app/test: fix compilation error when debug mode > enabled > > Hi Arek, >

[dpdk-dev] qos: traffic shaping at queue level

2016-09-27 Thread Nikhil Jagtap
Hi, I have a few questions about the hierarchical scheduler. I am taking a simple example here to get a better understanding. Reference example: pipe rate = 30 mbps tc 0 rate = 30 mbps traffic-type 0 being queued to queue 0, tc 0. traffic-type 1 being queued to queue 1, tc 0. Assume tra

[dpdk-dev] [PATCH] app/test: remove unnecessary conditional

2016-09-27 Thread Trahe, Fiona
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara > Sent: Monday, September 26, 2016 11:13 PM > To: dev at dpdk.org > Cc: De Lara Guarch, Pablo > Subject: [dpdk-dev] [PATCH] app/test: remove unnecessary conditional > > Regardless the result of

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

2016-09-27 Thread Yuanhan Liu
On Thu, Sep 22, 2016 at 05:01:41PM +0800, Jianbo Liu wrote: > On 22 September 2016 at 14:58, Wang, Zhihong > wrote: > > > > > >> -Original Message- > >> From: Jianbo Liu [mailto:jianbo.liu at linaro.org] > >> Sent: Thursday, September 22, 2016 1:48 PM > >> To: Yuanhan Liu > >> Cc: Wang,

[dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF management

2016-09-27 Thread Iremonger, Bernard
Hi Thomas, Bruce, > Subject: Re: [dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF > management > > 2016-09-26 15:37, Iremonger, Bernard: > > Hi Thomas, Bruce, > > > > > > > > > Subject: Re: [dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's > > > for VF management > > > > > > 20

[dpdk-dev] [PATCH v4 0/2] app/testpmd: improve multiprocess support

2016-09-27 Thread Marcin Kerlin
This patch ensure not overwrite device data in the multiprocess application. 1)Changes in the library introduces continuity in array rte_eth_dev_data[] shared between all processes. Secondary process adds new entries in free space instead of overwriting existing entries. 2)Changes in application

[dpdk-dev] [PATCH v4 0/2] app/testpmd: improve multiprocess support

2016-09-27 Thread Marcin Kerlin
This patch ensure not overwrite device data in the multiprocess application. 1)Changes in the library introduces continuity in array rte_eth_dev_data[] shared between all processes. Secondary process adds new entries in free space instead of overwriting existing entries. 2)Changes in application

[dpdk-dev] [PATCH v4 1/2] librte_ether: add protection against overwrite device data

2016-09-27 Thread Marcin Kerlin
Added prevention not overwrite device data in array rte_eth_dev_data[] for the next secondary applications. Secondary process appends in the first free place rather than at the beginning. This behavior prevents overwriting devices data of primary process by secondary process. Signed-off-by: Marcin

[dpdk-dev] [PATCH v4 2/2] app/testpmd: improve handling of multiprocess

2016-09-27 Thread Marcin Kerlin
Added lookup for pool name because secondary process should attach to mempool created by primary process rather than create new one. Added function free_shared_dev_data() used at the exit of the testpmd. This causes detach devices data from array rte_eth_dev_data[] shared between all processes. Th

[dpdk-dev] [RFC][PATCH V2 1/3] examples/vhost: Add vswitch (generic switch) framework

2016-09-27 Thread Pankaj Chauhan
On 9/19/2016 6:12 PM, Tan, Jianfeng wrote: > Hi Pankaj, > Hi Jianfeng, Sorry for delayed reply. >>> Can we hide queues inside struct vswitch_port? I mean: >>> For VMDQ switch, treat (port_id, queue_id) as a vswitch_port, so far >>> you've already stored "struct vhost_dev *" into vswitch_port.pri

[dpdk-dev] [RFC][PATCH V2 1/3] examples/vhost: Add vswitch (generic switch) framework

2016-09-27 Thread Pankaj Chauhan
On 9/26/2016 9:26 AM, Yuanhan Liu wrote: > On Tue, Sep 20, 2016 at 02:28:17PM +0530, Pankaj Chauhan wrote: >> On 9/19/2016 8:13 PM, Yuanhan Liu wrote: >>> Firstly, sorry for being late on this discussion: I just got a chance >>> to follow what you guys were talking about. >>> >>> On Tue, Sep 13, 20

[dpdk-dev] [RFC][PATCH V2 1/3] examples/vhost: Add vswitch (generic switch) framework

2016-09-27 Thread Pankaj Chauhan
On 9/26/2016 9:42 AM, Yuanhan Liu wrote: > Besides the VMDq proposal, I got few more comments for you. > > On Mon, Sep 05, 2016 at 04:24:29PM +0530, Pankaj Chauhan wrote: >> Introduce support for a generic framework for handling of switching between >> physical and vhost devices. The vswitch framew

[dpdk-dev] [RFC][PATCH V2 1/3] examples/vhost: Add vswitch (generic switch) framework

2016-09-27 Thread Yuanhan Liu
On Tue, Sep 27, 2016 at 05:14:44PM +0530, Pankaj Chauhan wrote: > On 9/26/2016 9:42 AM, Yuanhan Liu wrote: > >Besides the VMDq proposal, I got few more comments for you. > > > >On Mon, Sep 05, 2016 at 04:24:29PM +0530, Pankaj Chauhan wrote: > >>Introduce support for a generic framework for handling

[dpdk-dev] [RFC][PATCH V2 1/3] examples/vhost: Add vswitch (generic switch) framework

2016-09-27 Thread Yuanhan Liu
On Tue, Sep 27, 2016 at 05:05:41PM +0530, Pankaj Chauhan wrote: > >Hi Pankaj, > > > >Again, apologize for late response: you see I was busy ;) Besides, I > >need some time to think about it. > > > > Hi YLiu, > > No issues with delayed response :) Thanks! > > >Generally, I think your ideal prop

[dpdk-dev] [PATCH v4] vhost: Add indirect descriptors support to the TX path

2016-09-27 Thread Yuanhan Liu
On Tue, Sep 27, 2016 at 10:42:49AM +0200, Maxime Coquelin wrote: > Indirect descriptors are usually supported by virtio-net devices, > allowing to dispatch a larger number of requests. > > When the virtio device sends a packet using indirect descriptors, > only one slot is used in the ring, even f

[dpdk-dev] [PATCH v3] net: fix build error with clang

2016-09-27 Thread Yuanhan Liu
Can any PMD guys review it? It blocks a virtio patchset apply... Thanks. --yliu On Mon, Sep 26, 2016 at 12:29:13PM +0800, Yuanhan Liu wrote: > Interestingly, clang and gcc has different prototype for _mm_prefetch(). > For gcc, we have > >_mm_prefetch (const void *__P, enum _mm_hint

[dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF management

2016-09-27 Thread Bruce Richardson
On Tue, Sep 27, 2016 at 11:31:06AM +0100, Iremonger, Bernard wrote: > Hi Thomas, Bruce, > > > > > Subject: Re: [dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF > > management > > > > 2016-09-26 15:37, Iremonger, Bernard: > > > Hi Thomas, Bruce, > > > > > > > > > > > > > Subject: Re

[dpdk-dev] [PATCH v2] net/i40e: remove weak symbols

2016-09-27 Thread Zoltan Kiss
On 14/09/16 15:42, Ferruh Yigit wrote: > On 7/20/2016 6:11 PM, Zoltan Kiss wrote: >> Using weak symbols have a few issues with static linking: >> >> - normally the linker searches the .o files already linked, if your weak >>one is there, it won't check if there is a strong version >> - unless t

[dpdk-dev] [PATCH 1/1] ixgbe: fix a log

2016-09-27 Thread Bruce Richardson
On Wed, Sep 21, 2016 at 12:07:32PM +0100, Remy Horton wrote: > > On 27/08/2016 04:28, Yunjian Wang wrote: > >Signed-off-by: Yunjian Wang > >--- > > drivers/net/ixgbe/ixgbe_ethdev.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > Acked-by: Remy Horton Applied to dpdk-next-net/rel_1

[dpdk-dev] [PATCH] net/enic: add support for Rx queue count function

2016-09-27 Thread Bruce Richardson
On Mon, Sep 19, 2016 at 11:50:29AM -0700, Nelson Escobar wrote: > Add support to enic for rte_eth_rx_queue_count(). > > Signed-off-by: Nelson Escobar > Reviewed-by: John Daley > --- > drivers/net/enic/enic_ethdev.c | 31 ++- > 1 file changed, 30 insertions(+), 1 dele

[dpdk-dev] [PATCH v3] i40e: enable i40e pmd on ARM platform

2016-09-27 Thread Bruce Richardson
On Fri, Sep 23, 2016 at 01:13:17AM +, Zhang, Qi Z wrote: > Hi > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jianbo Liu > > Sent: Friday, August 5, 2016 5:06 PM > > To: dev at dpdk.org; Zhang, Helin ; Wu, Jingjing > > > > Cc: Jianbo Liu > > Subje

[dpdk-dev] [PATCH v3] app/test: add AES GCM performance test to cryptodev

2016-09-27 Thread Arek Kusztal
This patch adds AES Galois Counter Mode performance test case for cryptodev QAT and AESNI GCM. Test is performed with different buffer sizes, burst size of 32 and 128b key. Test vectors are placed in app/test/test_cryptodev_perf_vectors.h file. Signed-off-by: Arek Kusztal --- This patch depends o

[dpdk-dev] [PATCH] bnxt: Update MAINTAINERS file for bnxt driver

2016-09-27 Thread Bruce Richardson
On Wed, Sep 21, 2016 at 03:49:20PM -0700, Stephen Hurd wrote: > On Wed, Sep 21, 2016 at 3:27 PM, Ajit Khaparde > wrote: > > > Add Ajit Khaparde as the maintainer of the bnxt PMD > > > > CC: Stephen Hurd > > Signed-off-by: Ajit Khaparde > > Reviewed-by: David Christensen > > --- > > MAINTAINER

[dpdk-dev] [PATCH] net/mlx5: support Mellanox OFED 3.4

2016-09-27 Thread Bruce Richardson
On Fri, Sep 23, 2016 at 11:14:59AM +0200, Adrien Mazarguil wrote: > On Wed, Sep 21, 2016 at 03:48:12PM +0200, Nelio Laranjeiro wrote: > > Some macros are renamed by Mellanox OFED 3.4. > > > > Signed-off-by: Nelio Laranjeiro > > Acked-by: Adrien Mazarguil > Applied to dpdk-next-net/rel_16_11 /

[dpdk-dev] [PATCH] net/mlx5: return RSS hash result in mbuf

2016-09-27 Thread Bruce Richardson
On Wed, Sep 14, 2016 at 10:16:05AM +0200, Nelio Laranjeiro wrote: > Signed-off-by: Nelio Laranjeiro > --- > drivers/net/mlx5/mlx5_rxq.c | 1 + > drivers/net/mlx5/mlx5_rxtx.c | 6 +- > drivers/net/mlx5/mlx5_rxtx.h | 2 ++ > 3 files changed, 8 insertions(+), 1 deletion(-) > > diff --git a/dri

[dpdk-dev] [PATCH v1 0/4] Generalize PCI specific EAL function/structures

2016-09-27 Thread Shreyansh Jain
(I rebased these over HEAD 7b3c4f3) These patches were initially part of Jan's original series on SoC Framework ([1],[2]). An update to that series, without these patches, was posted here [3]. Main motivation for these is aim of introducing a non-PCI centric subsystem in EAL. As of now the first

[dpdk-dev] [PATCH v1 1/4] eal: generalize PCI kernel driver enum to EAL

2016-09-27 Thread Shreyansh Jain
From: Jan Viktorin Signed-off-by: Jan Viktorin Signed-off-by: Shreyansh Jain -- Changes since v0: - fix compilation error due to missing include --- lib/librte_eal/common/include/rte_dev.h | 12 lib/librte_eal/common/include/rte_pci.h | 10 +- 2 files changed, 13 inserti

[dpdk-dev] [PATCH v1 2/4] eal: generalize PCI map/unmap resource to EAL

2016-09-27 Thread Shreyansh Jain
From: Jan Viktorin The functions pci_map_resource, pci_unmap_resource are generic so the pci_* prefix can be omitted. The functions are moved to the eal_common_dev.c so they can be reused by other infrastructure. Signed-off-by: Jan Viktorin Signed-off-by: Shreyansh Jain --- lib/librte_eal/bsd

[dpdk-dev] [PATCH v1 3/4] eal/linux: generalize PCI kernel unbinding driver to EAL

2016-09-27 Thread Shreyansh Jain
From: Jan Viktorin Generalize the PCI-specific pci_unbind_kernel_driver. It is now divided into two parts. First, determination of the path and string identification of the device to be unbound. Second, the actual unbind operation which is generic. Signed-off-by: Jan Viktorin Signed-off-by: Shr

[dpdk-dev] [PATCH v1 4/4] eal/linux: generalize PCI kernel driver extraction to EAL

2016-09-27 Thread Shreyansh Jain
From: Jan Viktorin Generalize the PCI-specific pci_get_kernel_driver_by_path. The function is general enough, we have just moved it to eal.c, changed the prefix to rte_eal and provided it privately to other parts of EAL. Signed-off-by: Jan Viktorin Signed-off-by: Shreyansh Jain --- lib/librte

[dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add API's for VF management

2016-09-27 Thread Iremonger, Bernard
Hi Bruce, > > > > > Subject: Re: [dpdk-dev] [RFC PATCH v2 3/5] librte_ether: add > > > > > API's for VF management > > > > > > > > > > 2016-09-23 17:02, Iremonger, Bernard: > > > > > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > > > > > 2016-09-23 09:53, Richardson, Bruce:

[dpdk-dev] [PATCH 3/3] net/enic: enable link check interrupts

2016-09-27 Thread Bruce Richardson
On Mon, Sep 19, 2016 at 11:50:11AM -0700, Nelson Escobar wrote: > Signed-off-by: Nelson Escobar > Reviewed-by: John Daley > --- > doc/guides/nics/enic.rst | 3 ++- > drivers/net/enic/enic_ethdev.c | 2 +- > drivers/net/enic/enic_main.c | 19 ++- > 3 files changed, 21 in

[dpdk-dev] [PATCH 1/3] net/enic: cleanup interrupt setup when stopping port

2016-09-27 Thread Bruce Richardson
On Mon, Sep 19, 2016 at 11:50:09AM -0700, Nelson Escobar wrote: > enic_disable() wasn't calling rte_intr_disable() or > rte_intr_callback_unregister(). If stopping/starting a port, the > latter omission would result in the same interrupt callback being > registered multiple times, which would then

[dpdk-dev] [PATCH] net/mlx5: return RSS hash result in mbuf

2016-09-27 Thread Nélio Laranjeiro
Hi Bruce, On Tue, Sep 27, 2016 at 03:11:10PM +0100, Bruce Richardson wrote: > On Wed, Sep 14, 2016 at 10:16:05AM +0200, Nelio Laranjeiro wrote: > > Signed-off-by: Nelio Laranjeiro > > --- > > drivers/net/mlx5/mlx5_rxq.c | 1 + > > drivers/net/mlx5/mlx5_rxtx.c | 6 +- > > drivers/net/mlx5/ml

[dpdk-dev] [PATCH] net/enic: fix another case of memory not being freed

2016-09-27 Thread Bruce Richardson
On Thu, Sep 22, 2016 at 10:02:45AM -0700, John Daley wrote: > From: Nelson Escobar > > The function vnic_dev_free_desc_ring() didn't actually free memory. Fix > this by first changing vnic_dev_alloc_desc_ring() to use the common > allocation function, then in vnic_dev_free_desc_ring call the comm

[dpdk-dev] [PATCH] net/enic: support scatter Rx in the MTU update function

2016-09-27 Thread Bruce Richardson
On Thu, Sep 22, 2016 at 10:02:46AM -0700, John Daley wrote: > Re-initialize Rq's when MTU is changed. This allows for more > efficient use of mbufs when moving from an MTU that is greater > than the mbuf size to one that is less. And moves to using Rx > scatter mode when moving from an MTU less tha

[dpdk-dev] dpdk bonding net driver

2016-09-27 Thread Vasily Philipov
Hi all, I am trying to run testpmd using bonding: I have downloaded DPDK 16.07 and run the following command line: dpdk-install/bin/testpmd -c 0xf -n 4 --vdev 'eth_bond0,mode=0, slave=:08:00.0' -- --port-topology=chained And this is the error I get: EAL: Detected 28 lcore(s

[dpdk-dev] [PATCH v7 0/2] net/i40e: enable 25G device

2016-09-27 Thread Bruce Richardson
On Tue, Sep 27, 2016 at 08:57:57AM +, Wu, Jingjing wrote: > > > > -Original Message- > > From: Zhang, Qi Z > > Sent: Tuesday, September 27, 2016 9:37 AM > > To: Wu, Jingjing; Zhang, Helin > > Cc: dev at dpdk.org; Zhang, Qi Z > > Subject: [PATCH v7 0/2] net/i40e: enable 25G device > >

[dpdk-dev] [PATCH v2] net/mlx5: return RSS hash result in mbuf

2016-09-27 Thread Nelio Laranjeiro
Signed-off-by: Nelio Laranjeiro --- doc/guides/nics/mlx5.rst | 1 + drivers/net/mlx5/mlx5_rxq.c | 1 + drivers/net/mlx5/mlx5_rxtx.c | 21 - drivers/net/mlx5/mlx5_rxtx.h | 2 ++ 4 files changed, 20 insertions(+), 5 deletions(-) diff --git a/doc/guides/nics/mlx5.rst b/d

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

2016-09-27 Thread Dey, Souvik
Hi All, Any further updates/comments on this ? -- Regards, Souvik -Original Message- From: Yuanhan Liu [mailto:yuanhan@linux.intel.com] Sent: Sunday, September 25, 2016 11:21 PM To: Stephen Hemminger Cc: Dey, Souvik ; Kavanagh, Mark B ; dev at dpdk.org Subject: Re: [PATCH v6

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

2016-09-27 Thread Wang, Zhihong
> -Original Message- > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > Sent: Tuesday, September 27, 2016 6:21 PM > To: Jianbo Liu > Cc: Wang, Zhihong ; Maxime Coquelin > ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 0/5] vhost: optimize enqueue > > On Thu, Sep 22, 20

[dpdk-dev] [PATCH v2] app/testpmd: fix RSS-hash-key size

2016-09-27 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mohammad Abdul Awal > Sent: Friday, August 5, 2016 4:35 PM > To: dev at dpdk.org > Cc: Awal, Mohammad Abdul > Subject: [dpdk-dev] [PATCH v2] app/testpmd: fix RSS-hash-key size > > RSS hash-key-size is retrieved

[dpdk-dev] [PATCH v2] net/mlx5: return RSS hash result in mbuf

2016-09-27 Thread Ferruh Yigit
On 9/27/2016 3:53 PM, Nelio Laranjeiro wrote: > Signed-off-by: Nelio Laranjeiro <...> > @@ -1286,12 +1291,13 @@ mlx5_rx_burst(void *dpdk_rxq, struct rte_mbuf **pkts, > uint16_t pkts_n) > &(*rxq->cqes)[rxq->cq_ci & cqe_cnt].cqe64; > unsigned int i = 0; > unsigned int rq

[dpdk-dev] [PATCH v3] net: fix build error with clang

2016-09-27 Thread Chen, Jing D
Hi, > -Original Message- > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com] > Sent: Tuesday, September 27, 2016 5:20 AM > To: dev at dpdk.org > Cc: Jerin Jacob ; Chen, Jing D > ; Liang, Cunming ; > Richardson, Bruce ; Thomas Monjalon > > Subject: Re: [PATCH v3] net: fix build er

[dpdk-dev] [PATCH v5 3/3] app/testpmd: support tunneled TSO in csum fwd engine

2016-09-27 Thread Ananyev, Konstantin
> -Original Message- > From: Tan, Jianfeng > Sent: Monday, September 26, 2016 2:49 PM > To: dev at dpdk.org > Cc: Ananyev, Konstantin ; Wu, Jingjing > ; Tan, Jianfeng > ; Zhe Tao > Subject: [PATCH v5 3/3] app/testpmd: support tunneled TSO in csum fwd engine > > Add a new command "tunne

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

2016-09-27 Thread Ananyev, Konstantin
> > > -Original Message- > > > From: Tan, Jianfeng > > > Sent: Monday, August 1, 2016 11:57 AM > > > To: dev at dpdk.org > > > Cc: thomas.monjalon at 6wind.com; De Lara Guarch, Pablo; Ananyev, > > > Konstantin; Wu, Jingjing; Zhang, Helin; Tan, Jianfeng; Tao, Zhe > > > Subject: [PATCH v4 0

[dpdk-dev] [PATCH] eal: check cpu flags at init

2016-09-27 Thread Flavio Leitner
On Mon, Sep 26, 2016 at 11:43:37AM -0400, Aaron Conole wrote: > My only concern is whether this change would be considered ABI > breaking. I wouldn't think so, since it doesn't seem as though an > application would want to call this explicitly (and is spelled out as > such), but I can't be sure th

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

2016-09-27 Thread Stephen Hemminger
On Fri, 23 Sep 2016 15:17:37 + "Dey, Souvik" wrote: > Hi Liu/Mark/Stephen, > > I have tried to modify the code with all of your latest > comments. Do let me know if this looks fine or you have more comments. > > > > Changes done : > > -- max frame ize is compare to VIRTIO_

[dpdk-dev] [PATCH v2 0/3] fix virtio_user issues

2016-09-27 Thread Jianfeng Tan
v2: -- Patch 1: Enable 1st queue pair unconditionaly, instead of depending on VHOST_USER_GET_PROTOCOL_FEATURES. -- Patch 3: address Stephen's comment on confusion laying in two local variables named ret and result. Now we just keep one. Patch 1: fix issue when using virtio_user with OVS-DP

[dpdk-dev] [PATCH v2 1/3] net/virtio_user: fix queue pair not enabled

2016-09-27 Thread Jianfeng Tan
When virtio_user is used with OVS-DPDK (with mq disabled), it cannot receive any packets. It's because when vhost provides VHOST_USER_GET_PROTOCOL_FEATURES, all queue pairs are initialized in the disabled state. Quote QEMU/docs/specs/vhost-user.txt: If VHOST_USER_F_PROTOCOL_FEATURES has not be

[dpdk-dev] [PATCH v2 2/3] net/virtio_user: fix wrong sequence of messages

2016-09-27 Thread Jianfeng Tan
When virtio_user is used with VPP's native vhost user, it cannot send/receive any packets. The root cause is that vpp-vhost-user translates the message VHOST_USER_SET_FEATURES as puting this device into init state, aka, zero all related structures. However, previous code puts this message at last

[dpdk-dev] [PATCH v2 3/3] net/virtio_user: fix dev not freed after init error

2016-09-27 Thread Jianfeng Tan
Currently, when virtio_user device fails to be started (e.g., vhost unix socket does not exit), the init function does not return struct rte_eth_dev (and some other structs) back to ether layer. And what's more, it does not report the error to upper layer. The fix is to free those structs and repo

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

2016-09-27 Thread Thomas Monjalon
2016-09-28 01:52, Tan, Jianfeng: > Hi Konstantin, > > > On 9/28/2016 1:29 AM, Ananyev, Konstantin wrote: > > > [...] > > Acked-by: Konstantin Ananyev > > I think you need to rebase your first one: mbuf: add Tx side tunneling type > > against the mainline. > > The first one can be applied by 3-w

[dpdk-dev] [PATCH 1/2] vhost: enable any layout feature

2016-09-27 Thread Stephen Hemminger
On Tue, 27 Sep 2016 11:11:58 +0800 Yuanhan Liu wrote: > On Mon, Sep 26, 2016 at 10:24:55PM +0300, Michael S. Tsirkin wrote: > > On Mon, Sep 26, 2016 at 11:01:58AM -0700, Stephen Hemminger wrote: > > > I assume that if using Version 1 that the bit will be ignored > > Yes, but I will just quot

[dpdk-dev] [PATCH] testpmd: fix fdir command on MAC and tunnel modes

2016-09-27 Thread Frederico Cadete
On Tue, Sep 27, 2016 at 4:42 AM, Wu, Jingjing wrote: > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Frederico.Cadete- >> ext at oneaccess-net.com >> Sent: Tuesday, August 23, 2016 5:11 PM >> To: dev at dpdk.org >> Cc: frederico at cadete.eu; Frederico

[dpdk-dev] [PATCH 1/2] vhost: enable any layout feature

2016-09-27 Thread Michael S. Tsirkin
On Tue, Sep 27, 2016 at 11:11:58AM +0800, Yuanhan Liu wrote: > On Mon, Sep 26, 2016 at 10:24:55PM +0300, Michael S. Tsirkin wrote: > > On Mon, Sep 26, 2016 at 11:01:58AM -0700, Stephen Hemminger wrote: > > > I assume that if using Version 1 that the bit will be ignored > > Yes, but I will just quo

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

2016-09-27 Thread Dey, Souvik
Hi Stephen, So what should be my next steps , should I submit a v7 for this patch or you suggest otherwise. -- Regards, Souvik -Original Message- From: Stephen Hemminger [mailto:step...@networkplumber.org] Sent: Tuesday, September 27, 2016 2:57 PM To: Dey, Souvik Cc: Kavanagh,

[dpdk-dev] [PATCH] hash: fix ring size

2016-09-27 Thread De Lara Guarch, Pablo
> -Original Message- > From: De Lara Guarch, Pablo > Sent: Monday, September 26, 2016 4:52 PM > To: dev at dpdk.org > Cc: Richardson, Bruce; De Lara Guarch, Pablo > Subject: [PATCH] hash: fix ring size > > Ring stores the free slots available to be used in the key table. > The ring size

[dpdk-dev] [PATCH 0/3] Hash library fixes

2016-09-27 Thread Edupuganti, Saikrishna
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara > Sent: Friday, August 26, 2016 2:30 PM > To: dev at dpdk.org > Cc: Richardson, Bruce ; De Lara Guarch, Pablo > > Subject: [dpdk-dev] [PATCH 0/3] Hash library fixes > > This patchset includes so