[dpdk-dev] [PATCH 15/16] net/avp: device start and stop operations

2017-02-24 Thread Allain Legacy
Adds support for device start and stop functions. This allows an application to control the administrative state of an AVP device. Stopping the device will notify the host application to stop sending packets on that device's receive queues. Signed-off-by: Allain Legacy Signed-off-by: Matt Peter

[dpdk-dev] [PATCH 16/16] doc: adds information related to the AVP PMD

2017-02-24 Thread Allain Legacy
Updates the documentation and feature lists for the AVP PMD device. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- MAINTAINERS | 1 + doc/guides/nics/avp.rst | 99 doc/guides/nics/features/avp.ini | 17 +++

[dpdk-dev] [PATCH 14/16] net/avp: device promiscuous functions

2017-02-24 Thread Allain Legacy
Adds support for setting and clearing promiscuous mode on an AVP device. When enabled the _mac_filter function will allow packets destined to any MAC address to be processed by the receive functions. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 34 +

[dpdk-dev] [PATCH 13/16] net/avp: device statistics operations

2017-02-24 Thread Allain Legacy
Adds support for device get/set operations against an AVP device so that an application can query and reset statistics on an AVP device. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- config/common_base | 1 + drivers/net/avp/avp_ethdev.c | 81 +++

[dpdk-dev] [PATCH 08/16] net/avp: device initialization

2017-02-24 Thread Allain Legacy
Adds support for initialization newly probed AVP PCI devices. Initial queue translations are setup in preparation for device configuration. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 770 +++ 1 file changed

[dpdk-dev] [PATCH 11/16] net/avp: packet receive functions

2017-02-24 Thread Allain Legacy
Adds function required for receiving packets from the host application via AVP device queues. Both the simple and scattered functions are supported. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/Makefile | 1 + drivers/net/avp/avp_ethdev.c | 469

[dpdk-dev] [PATCH 10/16] net/avp: queue setup and release

2017-02-24 Thread Allain Legacy
Adds queue management operations so that an appliation can setup and release the transmit and receive queues. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 148 ++- 1 file changed, 147 insertions(+), 1 deletion

[dpdk-dev] [PATCH 09/16] net/avp: device configuration

2017-02-24 Thread Allain Legacy
Adds support for "dev_configure" operations to allow an application to configure the device. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 136 +++ 1 file changed, 136 insertions(+) diff --git a/drivers/net/av

[dpdk-dev] [PATCH 12/16] net/avp: packet transmit functions

2017-02-24 Thread Allain Legacy
Adds support for packet transmit functions so that an application can send packets to the host application via an AVP device queue. Both the simple and scattered functions are supported. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/avp_ethdev.c | 347

[dpdk-dev] [PATCH 04/16] net/avp: add PMD version map file

2017-02-24 Thread Allain Legacy
Adds a default ABI version file for the AVP PMD. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/avp/rte_pmd_avp_version.map | 4 1 file changed, 4 insertions(+) create mode 100644 drivers/net/avp/rte_pmd_avp_version.map diff --git a/drivers/net/avp/rte_pmd_avp_ve

[dpdk-dev] [PATCH 07/16] net/avp: driver registration

2017-02-24 Thread Allain Legacy
Adds the initial framework for registering the driver against the support PCI device identifiers. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- config/common_linuxapp | 1 + config/defconfig_i686-native-linuxapp-gcc| 5 + config/defconfig_i686-native-

[dpdk-dev] [PATCH 05/16] net/avp: debug log macros

2017-02-24 Thread Allain Legacy
Adds a header file with log macros for the AVP PMD Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- config/common_base | 4 drivers/net/avp/avp_logs.h | 59 ++ 2 files changed, 63 insertions(+) create mode 100644 drivers/net/

[dpdk-dev] [PATCH 03/16] maintainers: claim responsibility for AVP PMD

2017-02-24 Thread Allain Legacy
Updating the maintainers file to claim the AVP PMD driver on behalf of Wind River Systems, Inc. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- MAINTAINERS | 5 + 1 file changed, 5 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 24e0eff..992ffa5 100644 --- a/MAINTAIN

[dpdk-dev] [PATCH 06/16] drivers/net: adds driver makefiles for AVP PMD

2017-02-24 Thread Allain Legacy
Adds a default Makefile to the driver directory but does not include any source files. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- drivers/net/Makefile | 1 + drivers/net/avp/Makefile | 52 2 files changed, 53 insertions(+)

[dpdk-dev] [PATCH 02/16] net/avp: public header files

2017-02-24 Thread Allain Legacy
Adds public/exported header files for the AVP PMD. The AVP device is a shared memory based device. The structures and constants that define the method of operation of the device must be visible by both the PMD and the host DPDK application. They must not change without proper version controls an

[dpdk-dev] [PATCH 00/16] Wind River Systems AVP PMD

2017-02-24 Thread Allain Legacy
This patch series submits an initial version of the AVP PMD from Wind River Systems. The series includes shared header files, driver implementation, and changes to documentation files in support of this new driver. The AVP driver is a shared memory based device. It is intended to be used as a PM

[dpdk-dev] [PATCH 01/16] config: adds attributes for the AVP PMD

2017-02-24 Thread Allain Legacy
Updates the common base configuration file to include a top level config attribute for the AVP PMD. Signed-off-by: Allain Legacy Signed-off-by: Matt Peters --- config/common_base | 5 + 1 file changed, 5 insertions(+) diff --git a/config/common_base b/config/common_base index aeee13e..912b

Re: [dpdk-dev] [PATCH] igb_uio: use non-threaded ISR

2017-02-24 Thread Su, David W
>-Original Message- >From: Yigit, Ferruh >Sent: Friday, February 24, 2017 9:55 AM >To: Su, David W ; dev@dpdk.org >Subject: Re: [dpdk-dev] [PATCH] igb_uio: use non-threaded ISR > >On 1/20/2017 11:08 PM, David Su wrote: >> This eliminates the overhead of a task switch when an interrupt arriv

Re: [dpdk-dev] rte_sched library performance question

2017-02-24 Thread Zoltan Kiss
On 16/02/17 20:08, Dumitrescu, Cristian wrote: Hi Zoltan, -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Zoltan Kiss Sent: Thursday, February 16, 2017 3:14 PM To: dev@dpdk.org Subject: [dpdk-dev] rte_sched library performance question Hi, I'm experimenting a l

[dpdk-dev] [PATCH 2/2] app/test: add unit test for CRC computation

2017-02-24 Thread Jasvinder Singh
This patch provides a set of unit tests for verifying the functional correctness of 16-bit and 32-bit CRC APIs. Signed-off-by: Jasvinder Singh --- app/test/Makefile | 2 + app/test/test_crc.c | 229 2 files changed, 231 insertions(+) crea

[dpdk-dev] [PATCH 1/2] librte_net: add crc init and compute APIs

2017-02-24 Thread Jasvinder Singh
APIs for initalising and computing the crc (16-bit and 32-bit CRCs) are added. For CRCs calculation, scalar as well as x86 intrinsic(sse4.2) versions are implemented. The scalar version is based on generic Look-Up Table(LUT) algorithm, while x86 intrinsic version uses carry-less multiplication met

[dpdk-dev] [PATCH 0/2] librte_net: add crc computation support

2017-02-24 Thread Jasvinder Singh
In some applications, CRC (Cyclic Redundancy Check) needs to be computed or updated during packet processing operations. This patchset adds software implementation of some common standard CRCs (32-bit Ethernet CRC as per Ethernet/[ISO/IEC 8802-3] and 16-bit CCITT-CRC [ITU-T X.25]). Two versions of

Re: [dpdk-dev] [PATCH] igb_uio: use non-threaded ISR

2017-02-24 Thread Ferruh Yigit
On 1/20/2017 11:08 PM, David Su wrote: > This eliminates the overhead of a task switch when an interrupt arrives. Hi David, Did you test patch with l3fwd-power (or any app that uses Rx interrupts), is there any performance gain? > > Signed-off-by: David Su <...>

[dpdk-dev] [PATCH] proc-info: added collectd-format and host-id options.

2017-02-24 Thread Roman Korynkevych
Extended proc-info application to send DPDK port statistics to STDOUT in the format expected by collectd exec plugin. Added HOST ID option to identify the host DPDK process is running on when multiple instance of DPDK are running in parallel. This is needed for the barometer project in OPNFV. Sign

[dpdk-dev] [PATCH v2 2/2] ethdev: add hierarchical scheduler API

2017-02-24 Thread Cristian Dumitrescu
This patch introduces the generic ethdev API for the hierarchical scheduler capability. Main features: - Exposed as ethdev plugin capability (similar to rte_flow approach) - Capability query API per port, per hierarchy level and per hierarchy node - Scheduling algorithms: Strict Priority (SP), Wei

[dpdk-dev] [PATCH v2 0/2] ethdev: abstraction layer for QoS hierarchical scheduler

2017-02-24 Thread Cristian Dumitrescu
This patch set introduces an ethdev-based abstraction layer for Quality of Service (QoS) hierarchical scheduler. The goal is to provide a simple generic API that is agnostic of the underlying HW, SW or mixed HW-SW implementation. Patch 1 builds on the mechanism introduced by rte_flow in DPDK and g

[dpdk-dev] [PATCH v2 1/2] ethdev: add capability control API

2017-02-24 Thread Cristian Dumitrescu
The rte_flow feature breaks the current monolithic approach for ethdev and introduces the new generic flow API to ethdev using a plugin-like approach. Basically, the rte_flow API is still logically part of ethdev: - It extends the ethdev functionality: rte_flow is a new feature/capability of eth

Re: [dpdk-dev] [PATCH 0/4] New crypto algorithm string parser API

2017-02-24 Thread Trahe, Fiona
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Pablo de Lara > Sent: Thursday, February 23, 2017 12:34 PM > To: Doherty, Declan > Cc: dev@dpdk.org; De Lara Guarch, Pablo > Subject: [dpdk-dev] [PATCH 0/4] New crypto algorithm string parser API > > Last relea

[dpdk-dev] [PATCH] app/crypto-perf: fix uninitialized values for null operations

2017-02-24 Thread Tomasz Kulasek
Some values are uninitialized for "cipher null" and "auth null" operations. It may cause unpredictable results for some crypto pmd drivers, or even segmentation fault. This patch sets values for null operations to zero. Fixes: f8be1786b1b8 ("app/crypto-perf: introduce performance test application

Re: [dpdk-dev] [PATCH] crypto/scheduler: add enqueue and dequeue operations

2017-02-24 Thread Doherty, Declan
On 21/02/2017 11:43 AM, Fan Zhang wrote: This patch adds enqueue and dequeue operations to rte_cryptodev_scheduler_ops structure. This helps improving the performance by reducing unnecessary function calls. Signed-off-by: Fan Zhang --- ... rte_cryptodev_scheduler_create_private_ctx c

Re: [dpdk-dev] decision process to accept new libraries

2017-02-24 Thread Olivier Matz
On Fri, 24 Feb 2017 13:25:46 +, Bruce Richardson wrote: > On Fri, Feb 24, 2017 at 02:17:31PM +0100, Olivier Matz wrote: > > On Fri, 24 Feb 2017 11:33:11 +, Remy Horton > > wrote: > > > On 22/02/2017 19:06, Dumitrescu, Cristian wrote: > > > [..] > > > > This essentially leads to the "o

Re: [dpdk-dev] [PATCH v7 04/17] lib: add new burst oriented distributor structs

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:40AM +, David Hunt wrote: > Signed-off-by: David Hunt > --- > lib/librte_distributor/rte_distributor_private.h | 61 > > 1 file changed, 61 insertions(+) > > diff --git a/lib/librte_distributor/rte_distributor_private.h > b/lib/librte_

Re: [dpdk-dev] [RFC 0/8] mbuf: structure reorganization

2017-02-24 Thread Bruce Richardson
On Fri, Feb 24, 2017 at 03:00:53PM +0100, Olivier Matz wrote: > Hi, > > On Tue, 21 Feb 2017 20:30:57 +, "Ananyev, Konstantin" > wrote: > > > -Original Message- > > > From: jblu...@gmail.com [mailto:jblu...@gmail.com] On Behalf Of Jan > > > Blunck Sent: Tuesday, February 21, 2017 7:18

Re: [dpdk-dev] [PATCH v2 3/5] crypto/null: use ring size function

2017-02-24 Thread Doherty, Declan
On 23/02/2017 4:42 PM, Bruce Richardson wrote: Rather than reading the size directly from the ring structure, use the dedicated function for that purpose. Signed-off-by: Bruce Richardson --- ... Acked-by: Declan Doherty

Re: [dpdk-dev] [PATCH v7 15/17] lib: make v20 header file private

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:51AM +, David Hunt wrote: > Signed-off-by: David Hunt > --- > lib/librte_distributor/Makefile | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/lib/librte_distributor/Makefile b/lib/librte_distributor/Makefile > index 5b599c6..3017398 10

Re: [dpdk-dev] [PATCH v7 07/17] lib: apply symbol versioning to distibutor lib

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:43AM +, David Hunt wrote: > Note: LIBABIVER is also bumped up in the Makefile > > Signed-off-by: David Hunt > --- > lib/librte_distributor/rte_distributor.c | 10 +- > lib/librte_distributor/rte_distributor_v20.c | 10 ++ > lib/li

Re: [dpdk-dev] [PATCH v7 14/17] sample: switch to new distributor API

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:50AM +, David Hunt wrote: > and give distributor it's own thread This change probably deserves a separate patch from switching the API. /Bruce

Re: [dpdk-dev] [PATCH v7 04/17] lib: add new burst oriented distributor structs

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:40AM +, David Hunt wrote: > Signed-off-by: David Hunt > --- > lib/librte_distributor/rte_distributor_private.h | 61 > > 1 file changed, 61 insertions(+) > > diff --git a/lib/librte_distributor/rte_distributor_private.h > b/lib/librte_

Re: [dpdk-dev] [PATCH v7 12/17] example: add extra stats to distributor sample

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:48AM +, David Hunt wrote: > This will allow us to see what's going on at various stages > throughout the sample app, with per-second visibility > > Signed-off-by: David Hunt > --- For example apps, the patch prefix should be "examples/:" check-git-log.sh should c

Re: [dpdk-dev] [PATCH v7 03/17] lib: create rte_distributor_private.h

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:39AM +, David Hunt wrote: > We'll be adding content in here common to both burst and > legacy APIs. > > Signed-off-by: David Hunt > --- Couple of minor nits on the commit text here: * check-git-log.sh doesn't like the title as it's too technical. Suggest using

Re: [dpdk-dev] [PATCH v7 08/17] test: change params to distributor autotest

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:44AM +, David Hunt wrote: > In the next few patches, we'll want to test old and new API, > so here we're allowing different parameters to be passed to > the tests, instead of just a distributor struct. > > Signed-off-by: David Hunt > --- > app/test/test_distribu

Re: [dpdk-dev] [PATCH v7 06/17] lib: add SIMD flow matching to distributor

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:42AM +, David Hunt wrote: > Add an optimised version of the in-flight flow matching algorithm > using SIMD instructions. This should give up to 1.5x over the scalar > versions performance. > > Falls back to scalar version if SSE4.2 not available > > Signed-off-by

Re: [dpdk-dev] [RFC 0/8] mbuf: structure reorganization

2017-02-24 Thread Olivier Matz
On Tue, 21 Feb 2017 22:51:00 +0100, Morten Brørup wrote: > Regarding m->timestamp I have previously argued for keeping it NIC > specific, and not normalizing it. But I have changed my mind: > Normalizing it makes gives the user the ability to transparently swap > out a NIC from one vendor with one

Re: [dpdk-dev] [PATCH v7 05/17] lib: add new distributor code

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:41AM +, David Hunt wrote: > This patch includes public header file which will be used once > we add in the symbol versioning for v20 and v1705 APIs. > > Also includes v1702 private header file, and code for new Now v1705. Looking at the code, the header includes

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/mlx5: fix supported packets types

2017-02-24 Thread Ferruh Yigit
On 2/24/2017 9:16 AM, Nelio Laranjeiro wrote: > Fixes: 0603df73a077 ("net/mlx5: fix Rx packet validation and type") > Cc: sta...@dpdk.org > Signed-off-by: Nelio Laranjeiro > Acked-by: Adrien Mazarguil Applied to dpdk-next-net/master, thanks.

Re: [dpdk-dev] [PATCH v7 02/17] lib: symbol versioning of functions in distributor

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:38AM +, David Hunt wrote: > we will start the symbol versioning by renaming all legacy functions > > Signed-off-by: David Hunt > --- > app/test/test_distributor.c| 104 > +++-- > app/test/test_distributor_perf.c

Re: [dpdk-dev] [PATCH v7 01/17] lib: rename legacy distributor lib files

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:37AM +, David Hunt wrote: > Move files out of the way so that we can replace with new > versions of the distributor libtrary. Files are named in > such a way as to match the symbol versioning that we will > apply for backward ABI compatibility. > > Signed-off-by:

Re: [dpdk-dev] [PATCH v7 0/17] distributor library performance enhancements

2017-02-24 Thread Bruce Richardson
On Tue, Feb 21, 2017 at 03:17:36AM +, David Hunt wrote: > This patch aims to improve the throughput of the distributor library. > > It uses a similar handshake mechanism to the previous version of > the library, in that bits are used to indicate when packets are ready > to be sent to a worker

Re: [dpdk-dev] [RFC 0/8] mbuf: structure reorganization

2017-02-24 Thread Olivier Matz
Hi, On Tue, 21 Feb 2017 20:30:57 +, "Ananyev, Konstantin" wrote: > > -Original Message- > > From: jblu...@gmail.com [mailto:jblu...@gmail.com] On Behalf Of Jan > > Blunck Sent: Tuesday, February 21, 2017 7:18 PM > > To: Ananyev, Konstantin > > Cc: Richardson, Bruce ; Olivier MATZ > >

Re: [dpdk-dev] [PATCH 4/5] net/vhost: remove limit of vhost TX burst size

2017-02-24 Thread Kevin Traynor
On 02/24/2017 01:04 PM, Bruce Richardson wrote: > On Fri, Feb 24, 2017 at 11:08:56AM +, Kevin Traynor wrote: >> On 02/24/2017 08:48 AM, Zhiyong Yang wrote: >>> vhost removes limit of TX burst size(32 pkts) and supports to make >>> an best effort to transmit pkts. >>> >>> Cc: yuanhan@linux.i

Re: [dpdk-dev] decision process to accept new libraries

2017-02-24 Thread Bruce Richardson
On Fri, Feb 24, 2017 at 02:17:31PM +0100, Olivier Matz wrote: > On Fri, 24 Feb 2017 11:33:11 +, Remy Horton > wrote: > > On 22/02/2017 19:06, Dumitrescu, Cristian wrote: > > [..] > > > This essentially leads to the "other" repos becoming second class > > > citizens that can be broken at any ti

Re: [dpdk-dev] decision process to accept new libraries

2017-02-24 Thread Olivier Matz
On Fri, 24 Feb 2017 11:33:11 +, Remy Horton wrote: > On 22/02/2017 19:06, Dumitrescu, Cristian wrote: > [..] > > This essentially leads to the "other" repos becoming second class > > citizens that can be broken at any time without prior notice or the > > right to influence the change. The amou

Re: [dpdk-dev] decision process to accept new libraries

2017-02-24 Thread Bruce Richardson
On Fri, Feb 24, 2017 at 02:07:22PM +0100, Thomas Monjalon wrote: > 2017-02-22 19:06, Dumitrescu, Cristian: > > ... > > > > > The impact of having separate repositories is to reduce the work of a > > > contributor touching many areas in a rework. This cost is transfered > > > to the maintainer of t

Re: [dpdk-dev] [PATCH 1/3] doc: set alignments in NIC overview table

2017-02-24 Thread Thomas Monjalon
2017-02-23 12:51, Mcnamara, John: > On a related note, a while ago we looked at rotating the text of the NICs > to make the table slightly more readable. Me too I've looked at it while ago. > I've attached an example. > > It might be nice to add this as well, if it doesn't interfere with the oth

Re: [dpdk-dev] decision process to accept new libraries

2017-02-24 Thread Thomas Monjalon
2017-02-24 11:33, Remy Horton: > > On 22/02/2017 19:06, Dumitrescu, Cristian wrote: > [..] > > This essentially leads to the "other" repos becoming second class > > citizens that can be broken at any time without prior notice or the > > right to influence the change. The amount of maintenance work

Re: [dpdk-dev] decision process to accept new libraries

2017-02-24 Thread Thomas Monjalon
2017-02-22 19:06, Dumitrescu, Cristian: > ... > > > The impact of having separate repositories is to reduce the work of a > > contributor touching many areas in a rework. This cost is transfered > > to the maintainer of the separate repository impacted by the change > > in the main repository. So

Re: [dpdk-dev] [PATCH 4/5] net/vhost: remove limit of vhost TX burst size

2017-02-24 Thread Bruce Richardson
On Fri, Feb 24, 2017 at 11:08:56AM +, Kevin Traynor wrote: > On 02/24/2017 08:48 AM, Zhiyong Yang wrote: > > vhost removes limit of TX burst size(32 pkts) and supports to make > > an best effort to transmit pkts. > > > > Cc: yuanhan@linux.intel.com > > Cc: maxime.coque...@redhat.com > > >

Re: [dpdk-dev] [PATCH 5/5] net/vhost: remove limit of vhost RX burst size

2017-02-24 Thread Kevin Traynor
On 02/24/2017 08:48 AM, Zhiyong Yang wrote: > vhost removes limit of RX burst size(32 pkts) and supports to make > an best effort to receive pkts. > > Cc: yuanhan@linux.intel.com > Cc: maxime.coque...@redhat.com > > Signed-off-by: Zhiyong Yang > --- > drivers/net/vhost/rte_eth_vhost.c | 23

Re: [dpdk-dev] [PATCH v2] e1000/base: fix multicast setting in VF

2017-02-24 Thread Ferruh Yigit
On 2/24/2017 12:32 AM, Lu, Wenzhuo wrote: > Hi, > >> -Original Message- >> From: Yong Wang [mailto:wang.yon...@zte.com.cn] >> Sent: Tuesday, February 21, 2017 5:33 PM >> To: Lu, Wenzhuo >> Cc: dev@dpdk.org; Yong Wang >> Subject: [PATCH v2] e1000/base: fix multicast setting in VF >> >> In f

Re: [dpdk-dev] decision process to accept new libraries

2017-02-24 Thread Remy Horton
On 22/02/2017 19:06, Dumitrescu, Cristian wrote: [..] This essentially leads to the "other" repos becoming second class citizens that can be broken at any time without prior notice or the right to influence the change. The amount of maintenance work becomes very difficult to quantify (e.g. we al

Re: [dpdk-dev] [PATCH v2] e1000/base: fix multicast setting in VF

2017-02-24 Thread Ferruh Yigit
On 2/21/2017 9:33 AM, Yong Wang wrote: > In function e1000_update_mc_addr_list_vf(), "msgbuf[0]" is used prior > to initialization at "msgbuf[0] |= E1000_VF_SET_MULTICAST_OVERFLOW". > And "msgbuf[0]" is overwritten at "msgbuf[0] = E1000_VF_SET_MULTICAST". > Fix it by moving the second line prior to

Re: [dpdk-dev] [PATCH 4/5] net/vhost: remove limit of vhost TX burst size

2017-02-24 Thread Kevin Traynor
On 02/24/2017 08:48 AM, Zhiyong Yang wrote: > vhost removes limit of TX burst size(32 pkts) and supports to make > an best effort to transmit pkts. > > Cc: yuanhan@linux.intel.com > Cc: maxime.coque...@redhat.com > > Signed-off-by: Zhiyong Yang > --- > drivers/net/vhost/rte_eth_vhost.c | 24

Re: [dpdk-dev] [PATCH] net/ixgbevf: reset hardware when stoppingport

2017-02-24 Thread fengtian Guo
Hi, Wenzhuo Sorry, Now I just notice the email is related with me. > Seems the commit log doesn't match the code change. > This change cannot resolve the described problem. The problem is VF doesn't > handle the PF reset event. > Moving the hw_reset from close to stop doesn't help and not nece

Re: [dpdk-dev] [PATCH] net/ixgbevf: reset hardware whenstoppingport

2017-02-24 Thread fengtian Guo
Hi, Wenzhuo The attachment is test report without product. -- Original -- From: "Lu, Wenzhuo"; Date: Fri, Feb 24, 2017 04:54 PM To: "fengtian Guo"; "Olivier Matz"; "dev@dpdk.org"; "Zhang, Helin"; "Ananyev, Konstantin"; Cc: "sta...@dpdk.org"; "David March

Re: [dpdk-dev] [PATCH] net/ixgbevf: reset hardware whenstoppingport

2017-02-24 Thread Olivier Matz
Hi Fengtian, On Fri, 24 Feb 2017 18:15:27 +0800, "fengtian Guo" wrote: > Hi, Wenzhuo > > The attachment is test report without product. The test report uses is a use-case with 6WINDGate, so it cannot be reproduced by Intel guys. I'll see how to reproduce the same case with testpmd and send an

Re: [dpdk-dev] [PATCHv7 03/47] common/dpaa2: adding qbman driver

2017-02-24 Thread Ferruh Yigit
On 2/22/2017 8:23 AM, Shreyansh Jain wrote: > (Modified the subject to: 'Re: [PATCHv7 03/47] common/dpaa2: adding > qbman driver' from 'Re: Hello Ferruh, Neil,') > > Hello Ferruh, > > On Tuesday 21 February 2017 08:09 PM, Ferruh Yigit wrote: >> On 2/21/2017 1:42 PM, Shreyansh Jain wrote: >>> Tha

Re: [dpdk-dev] [PATCH 1/5] net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size

2017-02-24 Thread Yang, Zhiyong
Hi, Bruce: > -Original Message- > From: Richardson, Bruce > Sent: Friday, February 24, 2017 5:36 PM > To: Yang, Zhiyong > Cc: dev@dpdk.org; Chen, Jing D > Subject: Re: [dpdk-dev] [PATCH 1/5] net/fm10k: remove limit of > fm10k_xmit_pkts_vec burst size > > On Fri, Feb 24, 2017 at 09:32:56

Re: [dpdk-dev] [PATCH 1/5] net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size

2017-02-24 Thread Bruce Richardson
On Fri, Feb 24, 2017 at 09:32:56AM +, Bruce Richardson wrote: > On Fri, Feb 24, 2017 at 04:48:17PM +0800, Zhiyong Yang wrote: > > To add a wrapper function fm10k_xmit_pkts_vec_simple to remove > > the limit of tx burst size. The patch makes fm10k vec function > > an best effort to transmit the

Re: [dpdk-dev] [PATCH 1/5] net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size

2017-02-24 Thread Bruce Richardson
On Fri, Feb 24, 2017 at 04:48:17PM +0800, Zhiyong Yang wrote: > To add a wrapper function fm10k_xmit_pkts_vec_simple to remove > the limit of tx burst size. The patch makes fm10k vec function > an best effort to transmit the pkts in the consistent behavior > like fm10k_xmit_pkts does that. > > Cc:

[dpdk-dev] [PATCH] net/mlx5: fix supported packets types

2017-02-24 Thread Nelio Laranjeiro
Fixes: 0603df73a077 ("net/mlx5: fix Rx packet validation and type") Cc: sta...@dpdk.org Signed-off-by: Nelio Laranjeiro Acked-by: Adrien Mazarguil --- drivers/net/mlx5/mlx5_ethdev.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/mlx5/mlx5_ethdev.c b/driv

Re: [dpdk-dev] [PATCH 0/8] QoS features on i40e - Linux kernel divergence

2017-02-24 Thread Vincent JARDIN
Le 24/02/2017 à 08:23, Lu, Wenzhuo a écrit : It is good to allow setting QoS on device, but it looks like this is a device specific API, not a generic PMD function. I don't think any feature in DPDK should be hardcoded to one device type. Yes, they're private APIs. Normally we want to support ke

Re: [dpdk-dev] [PATCH] net/ixgbevf: reset hardware when stoppingport

2017-02-24 Thread Lu, Wenzhuo
Hi Fengtian, Would you like to let us know how you validate this patch? As I’ve tried just stop -> start VF can make it work again after PF down -> up. Thanks. From: fengtian@6wind.com.cn [mailto:fengtian@6wind.com.cn] On Behalf Of fengtian Guo Sent: Friday, February 24, 2017 4:47 PM T

[dpdk-dev] [PATCH 5/5] net/vhost: remove limit of vhost RX burst size

2017-02-24 Thread Zhiyong Yang
vhost removes limit of RX burst size(32 pkts) and supports to make an best effort to receive pkts. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang --- drivers/net/vhost/rte_eth_vhost.c | 23 +-- 1 file changed, 21 insertions(+), 2 d

[dpdk-dev] [PATCH 1/5] net/fm10k: remove limit of fm10k_xmit_pkts_vec burst size

2017-02-24 Thread Zhiyong Yang
To add a wrapper function fm10k_xmit_pkts_vec_simple to remove the limit of tx burst size. The patch makes fm10k vec function an best effort to transmit the pkts in the consistent behavior like fm10k_xmit_pkts does that. Cc: Jing Chen Signed-off-by: Zhiyong Yang --- drivers/net/fm10k/fm10k_eth

[dpdk-dev] [PATCH 3/5] net/ixgbe: remove limit of ixgbe_xmit_pkts_vec burst size

2017-02-24 Thread Zhiyong Yang
To add a wrapper function ixgbe_xmit_pkts_vec_simple to remove the limit of tx burst size and implement the "make an best effort to transmit the pkts" policy. The patch makes ixgbe vec function work in a consistent behavior like ixgbe_xmit_pkts_simple and ixgbe_xmit _pkts do that. Cc: Helin Zhang

[dpdk-dev] [PATCH 2/5] net/i40e: remove limit of i40e_xmit_pkts_vec burst size

2017-02-24 Thread Zhiyong Yang
To add a wrapper function i40e_xmit_pkts_vec_simple to remove the limit of tx burst size. The patch makes i40e vec function an best effort to transmit the pkts in the consistent behavior like i40e_xmit_pkts_simple and i40e_xmit_pkts do that. Cc: Helin Zhang Cc: Jingjing Wu Signed-off-by: Zhiyon

[dpdk-dev] [PATCH 4/5] net/vhost: remove limit of vhost TX burst size

2017-02-24 Thread Zhiyong Yang
vhost removes limit of TX burst size(32 pkts) and supports to make an best effort to transmit pkts. Cc: yuanhan@linux.intel.com Cc: maxime.coque...@redhat.com Signed-off-by: Zhiyong Yang --- drivers/net/vhost/rte_eth_vhost.c | 24 ++-- 1 file changed, 22 insertions(+), 2

[dpdk-dev] [PATCH 0/5] consistent PMD batching behaviour

2017-02-24 Thread Zhiyong Yang
The rte_eth_tx_burst() function in the file Rte_ethdev.h is invoked to transmit output packets on the output queue for DPDK applications as follows. static inline uint16_t rte_eth_tx_burst(uint8_t port_id, uint16_t queue_id, struct rte_mbuf **tx_pkts, uint16_t nb_pkts); Note: The

Re: [dpdk-dev] [PATCH] net/virtio-user: fix multi-process issue

2017-02-24 Thread Yuanhan Liu
On Thu, Feb 23, 2017 at 01:58:16PM +0200, Ami Sabo wrote: > Secondary process doesn't properly attach to the rte_eth_device > initialized by the primary process. > > Accessing device from secondary process (e.g. via rte_eth_rx_burst), > causes process to crash. because rte_eth_dev_data is not prop