Re: [dpdk-dev] [PATCH v3 1/2] net/mlx5: replace network to host macros

2017-09-15 Thread Yongseok Koh
> On Sep 14, 2017, at 6:43 AM, Shachar Beiser wrote: > > Signed-off-by: Shachar Beiser > --- [...] > @@ -550,7 +553,8 @@ > max_wqe -= n; > if (tso) { > uint32_t inl = > -

Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` API.

2017-09-15 Thread Pavan Nikhilesh Bhagavatula
On Fri, Sep 15, 2017 at 05:51:36PM +0200, Thomas Monjalon wrote: > 15/09/2017 16:59, Pavan Nikhilesh Bhagavatula: > > On Fri, Sep 15, 2017 at 02:44:57PM +, Van Haaren, Harry wrote: > > > > > > > We could also choose to add this function to rte_service.h ? > > > > > > > > > > Yes that is an opti

[dpdk-dev] [PATCH 2/3] build: add optional arch-specific headers install path

2017-09-15 Thread luca . boccassi
From: Luca Boccassi A subset of the dpdk headers are arch-dependent, but have common names and thus cause a clash in a multiarch installation. For example, rte_config.h is different for each target. Add a "include_subdir_arch" option to allow a user to specify a subdirectory for arch independen

[dpdk-dev] [PATCH 3/3] build: don't hard-code generic/exec-env install path

2017-09-15 Thread luca . boccassi
From: Luca Boccassi Instead of hard-coding the install path of generic and exec-env headers use the includedir option, so that it can be correctly overridden. Signed-off-by: Luca Boccassi --- lib/librte_eal/bsdapp/eal/meson.build | 2 +- lib/librte_eal/common/include/meson.build | 2 +- li

[dpdk-dev] [PATCH 1/3] build: rename pkgconfig to libdpdk.pc

2017-09-15 Thread luca . boccassi
From: Luca Boccassi In Debian and Ubuntu we have been shipping a pkgconfig file for DPDK for more than a year now, and the filename is libdpdk.pc. A few downstream projects, like OVS and Collectd, have adopted the use of libdpdk.pc in their build systems as well. In order to maintain backward com

[dpdk-dev] [PATCH 0/3] meson: compatibility with Debian/Ubuntu

2017-09-15 Thread luca . boccassi
From: Luca Boccassi A couple of small fixes are needed in order to make the meson build compatible with Debian and Ubuntu, to avoid breaking backward compatibility for applications that depend on the packages. Debian and Ubuntu have been shipping a pkg-config file for more than a year, but it is

[dpdk-dev] [PATCH v2 5/6] net/mlx5: add operations for secondary process

2017-09-15 Thread Xueming Li
Add operations that are safe for secondary processes: * (x)stats * device info get * rx/tx descriptor status Signed-off-by: Xueming Li Acked-by: Nelio Laranjeiro --- drivers/net/mlx5/mlx5.c| 16 +++- drivers/net/mlx5/mlx5.h| 1 + drivers/net/mlx5/mlx5_ethdev.c | 11

[dpdk-dev] [PATCH v2 4/6] net/mlx5: remove verbs fork check

2017-09-15 Thread Xueming Li
Verbs API prohibited resources inheritance by default, this further disabled PMD forked secondary process that replis on memory inherited from parent process. This patch removes verbs fork check to enable DPDK forked process. Signed-off-by: Xueming Li Acked-by: Nelio Laranjeiro --- drivers/net

[dpdk-dev] [PATCH v2 6/6] net/mlx5: multi-process document update

2017-09-15 Thread Xueming Li
This patch update the feature list and NIC guide to be multi-process enabled. Signed-off-by: Xueming Li Acked-by: Nelio Laranjeiro --- doc/guides/nics/features/mlx5.ini | 1 + doc/guides/nics/mlx5.rst | 3 +-- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/guides/ni

[dpdk-dev] [PATCH v2 3/6] net/mlx5: allocate verbs object into shared memory

2017-09-15 Thread Xueming Li
PMD uses Verbs object which were not available in the shared memory. This patch modify the location where Verbs objects are allocated (from process memory address space to shared memory address space) and thus allow a secondary process to use those object by mapping this shared memory space its ow

[dpdk-dev] [PATCH v2 2/6] net/mlx5: install a socket to exchange a file descriptor

2017-09-15 Thread Xueming Li
Use a unix socket to get back the communication channel with the Kernel driver from the primary process, this is necessary to remap those pages in the secondary process memory space and thus use the same Tx queues. This is only supported from rdma-core (v15). Signed-off-by: Nelio Laranjeiro Sign

[dpdk-dev] [PATCH v2 1/6] net/mlx5: change eth device reference for secondary process

2017-09-15 Thread Xueming Li
rte_eth_dev created by primary process were not available in secondary process, it was not possible to use the primary process local memory object from a secondary process. This patch modify the reference of primary rte_eth_dev object, use local rte_eth_dev secondary process instead. Signed-off-b

Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` API.

2017-09-15 Thread Thomas Monjalon
15/09/2017 16:59, Pavan Nikhilesh Bhagavatula: > On Fri, Sep 15, 2017 at 02:44:57PM +, Van Haaren, Harry wrote: > > > > > We could also choose to add this function to rte_service.h ? > > > > > > > > Yes that is an option, and OK with me. > > > > > > > > @Pavan what do you think of adding it to

[dpdk-dev] [PATCH v2] examples/l2fwd_fork: fix message pool init

2017-09-15 Thread Xueming Li
Some invalid callback functions are provided to rte_pktmbuf_pool_init() without their associated data causing a segmentation fault when the function tries to use it. In this example, those callbacks are not necessary, they can be safely replaced by NULL pointers. Fixes: 95e8005a56e8 ("examples/l2f

Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` API.

2017-09-15 Thread Pavan Nikhilesh Bhagavatula
On Fri, Sep 15, 2017 at 02:44:57PM +, Van Haaren, Harry wrote: > > > > -Original Message- > > From: Pavan Nikhilesh Bhagavatula [mailto:pbhagavat...@caviumnetworks.com] > > Sent: Friday, September 15, 2017 3:42 PM > > To: Van Haaren, Harry > > Cc: tho...@monjalon.net; dev@dpdk.org > >

Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` API.

2017-09-15 Thread Van Haaren, Harry
> -Original Message- > From: Pavan Nikhilesh Bhagavatula [mailto:pbhagavat...@caviumnetworks.com] > Sent: Friday, September 15, 2017 3:42 PM > To: Van Haaren, Harry > Cc: tho...@monjalon.net; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` > API

Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` API.

2017-09-15 Thread Pavan Nikhilesh Bhagavatula
On Fri, Sep 15, 2017 at 01:57:42PM +, Van Haaren, Harry wrote: > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > Sent: Friday, September 15, 2017 2:53 PM > > To: Pavan Nikhilesh Bhagavatula ; Van > > Haaren, > > Harry > > Cc: dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH] eal: adde

Re: [dpdk-dev] [PATCH 3/4] eventdev: Add eventdev ethernet Rx adapter

2017-09-15 Thread santosh
On Friday 15 September 2017 04:40 PM, Rao, Nikhil wrote: > On 9/15/2017 11:40 AM, santosh wrote: >> Hi Nikhil, >> >> >> On Thursday 14 September 2017 12:23 AM, Nikhil Rao wrote: >>> Add common APIs for configuring packet transfer from ethernet Rx >>> queues to event devices across HW & SW packet t

Re: [dpdk-dev] [PATCH] librte_cfgfile: add section num entries by index

2017-09-15 Thread Thomas Monjalon
04/09/2017 12:58, Bruce Richardson: > On Thu, Apr 27, 2017 at 12:21:40PM +0530, Guduri Prathyusha wrote: > > rte_cfgfile_section_num_entries_by_index() is added to get the number of > > entries of a section when multiple sections of the same name are > > present. > > > > Signed-off-by: Guduri Prat

Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` API.

2017-09-15 Thread Van Haaren, Harry
> From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Friday, September 15, 2017 2:53 PM > To: Pavan Nikhilesh Bhagavatula ; Van Haaren, > Harry > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` > API. > > 28/08/2017 17:09, Pavan Nikhilesh Bha

Re: [dpdk-dev] [PATCH v4 5/5] test/cfgfile: add new unit test

2017-09-15 Thread Thomas Monjalon
04/09/2017 11:30, Bruce Richardson: > This file is a temporary one, so: > a) should be placed in /tmp > b) should be removed at the end of the test. Jacek, do you plan to update this series in 17.11 timeframe?

Re: [dpdk-dev] [PATCH] eal: added new `rte_lcore_is_service_lcore` API.

2017-09-15 Thread Thomas Monjalon
28/08/2017 17:09, Pavan Nikhilesh Bhagavatula: > On Mon, Aug 28, 2017 at 01:49:37PM +, Van Haaren, Harry wrote: > > From: Pavan Nikhilesh Bhagavatula [mailto:pbhagavat...@caviumnetworks.com] > > > On Mon, Aug 28, 2017 at 10:59:51AM +, Van Haaren, Harry wrote: > > > > From: Pavan Nikhilesh [

[dpdk-dev] [PATCH] doc: add note on hardware support deprecation

2017-09-15 Thread Bruce Richardson
Following agreement at the DPDK Technical Board meeting [1], the policy that hardware support deprecation should be treated as though it were an ABI break needs to be documented in the contributors guide. [1] http://dpdk.org/ml/archives/dev/2017-September/074613.html Signed-off-by: Bruce Richards

Re: [dpdk-dev] [PATCH v2] service: fix service lcore stop function

2017-09-15 Thread Thomas Monjalon
> > lcore_states store the state of the lcore. Fixing the invalid > > dereference of lcore_states with service number > > > > Unit test case service_lcore_start_stop fails with the above fix. > > Service core was stopped without stopping the service. > > > > This commit fixes the test by adding n

Re: [dpdk-dev] [PATCH v2 3/3] crypto/qat: enable TX tail writes coalescing

2017-09-15 Thread Trahe, Fiona
> -Original Message- > From: Burakov, Anatoly > Sent: Tuesday, September 12, 2017 10:31 AM > To: dev@dpdk.org > Cc: Trahe, Fiona ; Griffin, John > ; Jain, Deepak K > ; De Lara Guarch, Pablo > > Subject: [PATCH v2 3/3] crypto/qat: enable TX tail writes coalescing > > Don't write CSR ta

Re: [dpdk-dev] [PATCH 09/11] net/ixgbe: enable inline ipsec

2017-09-15 Thread Doherty, Declan
On 15/09/2017 5:48 AM, Hemant Agrawal wrote: Hi Radu, On 9/14/2017 1:56 PM, Akhil Goyal wrote: +static const struct rte_security_capability ixgbe_security_capabilities[] = { +    { /* IPsec Inline Crypto AH Transport Egress */ +    .action = RTE_SECURITY_ACTION_TYPE_INLINE_CRYPTO, +  

Re: [dpdk-dev] [PATCH v2 11/30] crypto/dpaa2_sec: update MC to 10.3.x

2017-09-15 Thread Hemant Agrawal
On 9/15/2017 6:07 PM, Ferruh Yigit wrote: On 9/8/2017 9:45 AM, Hemant Agrawal wrote: From: Shreyansh Jain Signed-off-by: Shreyansh Jain --- drivers/crypto/dpaa2_sec/mc/dpseci.c | 676 +-- drivers/crypto/dpaa2_sec/mc/fsl_dpseci.h | 782 -

Re: [dpdk-dev] [PATCH v2] eventdev: clarify usage of forward and release ops

2017-09-15 Thread Jerin Jacob
-Original Message- > Date: Mon, 11 Sep 2017 13:59:01 +0530 > From: Jerin Jacob > To: Tim McDaniel > CC: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2] eventdev: clarify usage of forward and > release ops > User-Agent: Mutt/1.9.0 (2017-09-02) > > -Original Message- > > Date: W

Re: [dpdk-dev] [PATCH] eventdev: ease single-link queue config requirements

2017-09-15 Thread Jerin Jacob
-Original Message- > Date: Mon, 4 Sep 2017 11:39:13 +0530 > From: Jerin Jacob > To: Gage Eads > Cc: dev@dpdk.org, nipun.gu...@nxp.com, hemant.agra...@nxp.com, > harry.van.haa...@intel.com, bruce.richard...@intel.com > Subject: Re: [PATCH] eventdev: ease single-link queue config requireme

Re: [dpdk-dev] [PATCH v2 11/30] crypto/dpaa2_sec: update MC to 10.3.x

2017-09-15 Thread Ferruh Yigit
On 9/8/2017 9:45 AM, Hemant Agrawal wrote: > From: Shreyansh Jain > > Signed-off-by: Shreyansh Jain > --- > drivers/crypto/dpaa2_sec/mc/dpseci.c | 676 +-- > drivers/crypto/dpaa2_sec/mc/fsl_dpseci.h | 782 > --- > drivers/crypto/dpaa2_sec

Re: [dpdk-dev] [PATCH v4 0/4] eventdev: add attribute based get APIs

2017-09-15 Thread Jerin Jacob
-Original Message- > Date: Thu, 14 Sep 2017 17:08:59 +0100 > From: Harry van Haaren > To: dev@dpdk.org > CC: jerin.ja...@caviumnetworks.com, Harry van Haaren > > Subject: [PATCH v4 0/4] eventdev: add attribute based get APIs > X-Mailer: git-send-email 2.7.4 > > This patchset refactors t

Re: [dpdk-dev] [PATCH] crypto/ccp: Add support for AMD CCP Crypto poll mode driver

2017-09-15 Thread Kumar, Ravi1
>> -Original Message- >> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Kumar, Ravi1 >> Sent: Wednesday, August 30, 2017 7:25 AM >> To: dev@dpdk.org >> Subject: Re: [dpdk-dev] [PATCH] crypto/ccp: Add support for AMD CCP >> Crypto poll mode driver >> >> Hi, >> >> Please do not cons

Re: [dpdk-dev] [PATCH v2 05/30] bus/fslmc: enhance the QBMAN CENA mode

2017-09-15 Thread Hemant Agrawal
On 9/15/2017 5:56 PM, Ferruh Yigit wrote: On 9/8/2017 9:45 AM, Hemant Agrawal wrote: From: Haiying Wang Signed-off-by: Haiying Wang Signed-off-by: Hemant Agrawal --- drivers/bus/fslmc/qbman/qbman_sys.h | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/bus/fsl

Re: [dpdk-dev] [PATCH 2/4] net/axgbe : Add PMD support for AMD XGBE Ethernet

2017-09-15 Thread Kumar, Ravi1
>On 9/13/2017 6:47 PM, Ravi Kumar wrote: >> Signed-off-by: Ravi Kumar ><...> > >> +# >> +# Export include files >> +# >> +SYMLINK-y-include += axgbe_ethdev.h >> +SYMLINK-y-include += axgbe_rxtx.h >> +SYMLINK-y-include += axgbe_common.h >> +SYMLINK-y-include += axgbe_phy.h > >Are these need to be p

Re: [dpdk-dev] [PATCH v2 05/30] bus/fslmc: enhance the QBMAN CENA mode

2017-09-15 Thread Ferruh Yigit
On 9/8/2017 9:45 AM, Hemant Agrawal wrote: > From: Haiying Wang > > Signed-off-by: Haiying Wang > Signed-off-by: Hemant Agrawal > --- > drivers/bus/fslmc/qbman/qbman_sys.h | 7 +-- > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/drivers/bus/fslmc/qbman/qbman_sys.h > b/d

Re: [dpdk-dev] [PATCH 3/4] doc: Add documents for AMD XGBE Ethernet

2017-09-15 Thread Kumar, Ravi1
>On 9/14/2017 7:32 PM, Kumar, Ravi1 wrote: >>> Hi Ravi, >>> ' >>> On 9/13/2017 11:17 PM, Ravi Kumar wrote: Signed-off-by: Ravi Kumar --- doc/guides/nics/axgbe.rst | 214 + doc/guides/nics/features/axgbe.ini | 18 do

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/i40e: fix packet count for PF

2017-09-15 Thread Ferruh Yigit
On 8/20/2017 9:05 PM, Qi Zhang wrote: > Previously, for PF statistics we use VSI register for packet count > but use port's register for packet bytes, that cause inconsistent > situation of PF statistics when some VF is active, since it will > cover VF's packet bytes but not packet count. > The pa

[dpdk-dev] [PATCH v2] net/liquidio: fix uninitialized variable

2017-09-15 Thread Yong Wang
In func lio_dev_link_update(), "link.link_autoneg" is used in func call lio_dev_atomic_write_link_status(), but is uninitialized. v2: * Modify the initial value of 'link.link_autoneg' from 'ETH_LINK_FIXED' to 'ETH_LINK_AUTONEG'. Signed-off-by: Yong Wang --- drivers/net/liquidio/lio_ethdev.c |

Re: [dpdk-dev] [PATCH v2 2/3] crypto/qat: enable RX head writes coalescing

2017-09-15 Thread Trahe, Fiona
> -Original Message- > From: Burakov, Anatoly > Sent: Tuesday, September 12, 2017 10:31 AM > To: dev@dpdk.org > Cc: Trahe, Fiona ; Griffin, John > ; Jain, Deepak K > ; De Lara Guarch, Pablo > > Subject: [PATCH v2 2/3] crypto/qat: enable RX head writes coalescing > > Don't write CSR he

Re: [dpdk-dev] [PATCH v2 00/15] service: API improvements and updates

2017-09-15 Thread Thomas Monjalon
> Harry van Haaren (15): > service: rework probe and get name to use ids > service: rework lcore to service map functions > service: rework register to return service id > service: rework service start stop to runstate > service: rework service stats functions > service: rework unregist

[dpdk-dev] [PATCH] buildtools: add null point check for calloc

2017-09-15 Thread Yong Wang
In func locate_pmd_entries(), pointer 'new' returned from call to func 'calloc' may be NULL. It is dereferenced without null point check. Signed-off-by: Yong Wang --- buildtools/pmdinfogen/pmdinfogen.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/buildtools/pmdinfoge

Re: [dpdk-dev] [PATCH v2 24/30] bus/dpaa2: add support for hw extra stats API

2017-09-15 Thread Hemant Agrawal
On 9/15/2017 5:08 PM, Ferruh Yigit wrote: On 9/8/2017 9:45 AM, Hemant Agrawal wrote: Signed-off-by: Hemant Agrawal --- drivers/bus/dpaa/base/fman/fman_hw.c | 30 ++ drivers/bus/dpaa/include/fsl_fman.h | 3 +++ drivers/bus/dpaa/rte_bus_dpaa_version.map |

Re: [dpdk-dev] [PATCH v2 24/30] bus/dpaa2: add support for hw extra stats API

2017-09-15 Thread Ferruh Yigit
On 9/8/2017 9:45 AM, Hemant Agrawal wrote: > Signed-off-by: Hemant Agrawal > --- > drivers/bus/dpaa/base/fman/fman_hw.c | 30 ++ > drivers/bus/dpaa/include/fsl_fman.h | 3 +++ > drivers/bus/dpaa/rte_bus_dpaa_version.map | 1 + This patch, also next one, up

Re: [dpdk-dev] [PATCH v2 1/3] crypto/qat: remove atomics

2017-09-15 Thread Trahe, Fiona
> -Original Message- > From: Burakov, Anatoly > Sent: Tuesday, September 12, 2017 10:31 AM > To: dev@dpdk.org > Cc: Trahe, Fiona ; Griffin, John > ; Jain, Deepak K > ; De Lara Guarch, Pablo > > Subject: [PATCH v2 1/3] crypto/qat: remove atomics > > Replacing atomics in the QAT driver

Re: [dpdk-dev] [PATCH] net/liquidio: fix uninitialized variable

2017-09-15 Thread Shijith Thotton
On Thu, Sep 14, 2017 at 10:32:59PM -0400, Yong Wang wrote: > In func lio_dev_link_update(), "link.link_autoneg" is used in func call > lio_dev_atomic_write_link_status(), but is uninitialized. > > Signed-off-by: Yong Wang > --- > drivers/net/liquidio/lio_ethdev.c | 1 + > 1 file changed, 1 inser

Re: [dpdk-dev] [PATCH v2 1/2] net/sfc: free mbufs in bulks on EF10 native Tx datapath reap

2017-09-15 Thread Ferruh Yigit
On 9/13/2017 10:33 AM, Andrew Rybchenko wrote: > From: Ivan Malov > > Signed-off-by: Ivan Malov > Signed-off-by: Andrew Rybchenko Series applied to dpdk-next-net/master, thanks.

Re: [dpdk-dev] [PATCH 3/4] eventdev: Add eventdev ethernet Rx adapter

2017-09-15 Thread Rao, Nikhil
On 9/15/2017 11:40 AM, santosh wrote: Hi Nikhil, On Thursday 14 September 2017 12:23 AM, Nikhil Rao wrote: Add common APIs for configuring packet transfer from ethernet Rx queues to event devices across HW & SW packet transfer mechanisms. A detailed description of the adapter is contained in t

Re: [dpdk-dev] [PATCH 1/2] net/i40e: queue region set and flush

2017-09-15 Thread Ferruh Yigit
On 9/6/2017 10:11 AM, Ferruh Yigit wrote: > On 9/1/2017 3:38 AM, Zhao1, Wei wrote: >> HI, Ferruh >> >>> -Original Message- >>> From: Yigit, Ferruh >>> Sent: Friday, September 1, 2017 12:18 AM >>> To: Zhao1, Wei ; dev@dpdk.org >>> Subject: Re: [dpdk-dev] [PATCH 1/2] net/i40e: queue region s

Re: [dpdk-dev] [dpdk-stable] [PATCH v5 1/4] net/mlx5: fix num seg assumption on vPMD

2017-09-15 Thread Ferruh Yigit
On 9/14/2017 11:50 AM, Shahaf Shuler wrote: > vPMD Tx function assumes that after the scatter of the > multi-segment packets the next packet will be a single segment packet. > > This is not current as the function can return due to lack of resources > without sending all of the multi-segment mbufs

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/i40e: fix packet count for PF

2017-09-15 Thread Ferruh Yigit
On 9/15/2017 11:33 AM, Zhang, Qi Z wrote: > Sorry for late reply > >> -Original Message- >> From: Yigit, Ferruh >> Sent: Monday, August 21, 2017 8:43 PM >> To: Zhang, Qi Z ; Wu, Jingjing >> Cc: dev@dpdk.org; sta...@dpdk.org >> Subject: Re: [dpdk-stable] [PATCH] net/i40e: fix packet count

Re: [dpdk-dev] [PATCH] eal: cleanup strerror function

2017-09-15 Thread Van Haaren, Harry
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce Richardson > Sent: Thursday, September 7, 2017 2:10 PM > To: dev@dpdk.org > Cc: Richardson, Bruce > Subject: [dpdk-dev] [PATCH] eal: cleanup strerror function > > When compiled on Ubuntu with extra warnings enabled, the rte_strerror() >

Re: [dpdk-dev] [PATCH] cmdline: fix warning for unused return value

2017-09-15 Thread Van Haaren, Harry
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce Richardson > Sent: Thursday, September 7, 2017 2:09 PM > To: dev@dpdk.org > Cc: Richardson, Bruce > Subject: [dpdk-dev] [PATCH] cmdline: fix warning for unused return value > > When DPDK is compiled on Ubuntu with extra warnings turned

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/i40e: fix packet count for PF

2017-09-15 Thread Zhang, Qi Z
Sorry for late reply > -Original Message- > From: Yigit, Ferruh > Sent: Monday, August 21, 2017 8:43 PM > To: Zhang, Qi Z ; Wu, Jingjing > Cc: dev@dpdk.org; sta...@dpdk.org > Subject: Re: [dpdk-stable] [PATCH] net/i40e: fix packet count for PF > > On 8/20/2017 9:05 PM, Qi Zhang wrote: >

[dpdk-dev] [PATCH v2 3/3] app/crypto-perf: fix packet length check

2017-09-15 Thread Pablo de Lara
When using DES-CBC, packet size has to be multiple of 8 bytes, but if a list of packets is provided. the check was not correct. Fixes: fc4600fb2520 ("app/crypto-perf: add extra option checks") Cc: sta...@dpdk.org Signed-off-by: Pablo de Lara --- app/test-crypto-perf/cperf_options_parsing.c | 18

[dpdk-dev] [PATCH v2 2/3] crypto/openssl: add support for DES-CBC

2017-09-15 Thread Pablo de Lara
Signed-off-by: Pablo de Lara --- drivers/crypto/openssl/rte_openssl_pmd.c | 27 +++ drivers/crypto/openssl/rte_openssl_pmd_ops.c | 20 test/test/test_cryptodev.c | 21 + test/test/test_cryptodev_des_test_vector

[dpdk-dev] [PATCH v2 1/3] crypto/aesni_mb: add DES support

2017-09-15 Thread Pablo de Lara
The Multi-buffer library now supports DES-CBC and DES-DOCSISBPI algorithms, so this commit extends adds support for them in the PMD. Signed-off-by: Pablo de Lara Acked-by: Fan Zhang --- doc/guides/cryptodevs/aesni_mb.rst | 2 + doc/guides/cryptodevs/features/aesni_mb.ini| 3 ++

[dpdk-dev] [PATCH v2 0/3] Extend DES support

2017-09-15 Thread Pablo de Lara
This patchset extends the support of DES algorithms to the AESNI MB and OpenSSL PMDs. For AESNI MB PMD, the following algorithms are added: - DES-CBC - DES-DOCSISBPI For OpenSSL PMD, DES-CBC is added, as DES-DOCSISBPI was already supported. Changes in v2: - Added support for DES-CBC in OpenSSL PM

Re: [dpdk-dev] vf init issue with patch igb_uio: issue FLR during open and release of device file

2017-09-15 Thread Shijith Thotton
On Fri, Sep 15, 2017 at 09:18:00AM +, Yang, Qiming wrote: > > -Original Message- > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > Sent: Friday, September 15, 2017 4:43 PM > > To: Yang, Qiming > > Cc: dev@dpdk.org; Tan, Jianfeng ; Thotton, Shijith > > ; Hu, Xuekun ; Yigit, > >

Re: [dpdk-dev] vf init issue with patch igb_uio: issue FLR during open and release of device file

2017-09-15 Thread Ferruh Yigit
On 9/15/2017 10:18 AM, Yang, Qiming wrote: >> -Original Message- >> From: Thomas Monjalon [mailto:tho...@monjalon.net] >> Sent: Friday, September 15, 2017 4:43 PM >> To: Yang, Qiming >> Cc: dev@dpdk.org; Tan, Jianfeng ; Thotton, Shijith >> ; Hu, Xuekun ; Yigit, >> Ferruh ; Patil, Harish ;

Re: [dpdk-dev] [dpdk-stable] [PATCH] devtools: fix version search with git < 2.7.0

2017-09-15 Thread Nélio Laranjeiro
On Tue, Sep 05, 2017 at 12:05:32AM +0200, Thomas Monjalon wrote: > The script git-log-fixes.sh (used in check-git-log.sh) looks > for git tags to find the version where a bug is introduced. > > In DPDK 17.08, the script has been fixed to ignore tags from > non current branch. > It was using the op

Re: [dpdk-dev] vf init issue with patch igb_uio: issue FLR during open and release of device file

2017-09-15 Thread Yang, Qiming
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Friday, September 15, 2017 4:43 PM > To: Yang, Qiming > Cc: dev@dpdk.org; Tan, Jianfeng ; Thotton, Shijith > ; Hu, Xuekun ; Yigit, > Ferruh ; Patil, Harish ; > Gregory Etelson ; sta...@dpdk.org > Subject: Re:

Re: [dpdk-dev] vf init issue with patch igb_uio: issue FLR during open and release of device file

2017-09-15 Thread Thomas Monjalon
Hi 15/09/2017 10:04, Yang, Qiming: > Hi, Thomas > We meet a urgent issue because of Shijith's patch. > When use DPDK version 17.08, DPDK VF driver and kernel PF driver, i40e, ixgbe > and qede(Harish said) VF all meet initial failure. This patch has been integrated in DPDK 17.08-rc2 (2 months ag

Re: [dpdk-dev] [dpdk-stable] [PATCH] net/mlx5: fix calculating TSO inline size

2017-09-15 Thread Ferruh Yigit
On 9/13/2017 8:26 AM, Nélio Laranjeiro wrote: > On Wed, Sep 13, 2017 at 05:05:14AM +, Shahaf Shuler wrote: >> Tuesday, September 12, 2017 9:34 PM, Nélio Laranjeiro: On Sep 12, 2017, at 12:24 AM, Nélio Laranjeiro >> Is not it dangerous to assume inl will always be 4 bytes long? Why >>>

Re: [dpdk-dev] [PATCH v3 2/5] gso: add TCP/IPv4 GSO support

2017-09-15 Thread Hu, Jiayu
> -Original Message- > From: Ananyev, Konstantin > Sent: Friday, September 15, 2017 4:17 PM > To: Hu, Jiayu ; Kavanagh, Mark B > > Cc: dev@dpdk.org; Tan, Jianfeng > Subject: RE: [PATCH v3 2/5] gso: add TCP/IPv4 GSO support > > > > > -Original Message- > > From: Ananyev, Konst

Re: [dpdk-dev] [PATCH v3 2/5] gso: add TCP/IPv4 GSO support

2017-09-15 Thread Ananyev, Konstantin
> -Original Message- > From: Ananyev, Konstantin > Sent: Friday, September 15, 2017 9:16 AM > To: Hu, Jiayu ; Kavanagh, Mark B > > Cc: dev@dpdk.org; Tan, Jianfeng > Subject: RE: [PATCH v3 2/5] gso: add TCP/IPv4 GSO support > > Hi Jiayu, > > > -Original Message- > > From: Hu,

Re: [dpdk-dev] [PATCH v3 2/5] gso: add TCP/IPv4 GSO support

2017-09-15 Thread Ananyev, Konstantin
Hi Jiayu, > -Original Message- > From: Hu, Jiayu > Sent: Friday, September 15, 2017 8:55 AM > To: Ananyev, Konstantin ; Kavanagh, Mark B > > Cc: dev@dpdk.org; Tan, Jianfeng > Subject: RE: [PATCH v3 2/5] gso: add TCP/IPv4 GSO support > > Hi Konstantin, > > > -Original Message-

Re: [dpdk-dev] [PATCH v5 4/4] net/mlx5: enforce Tx num of segments limitation

2017-09-15 Thread Nélio Laranjeiro
On Thu, Sep 14, 2017 at 01:50:39PM +0300, Shahaf Shuler wrote: > Mellanox NICs has a limitation on the number of mbuf segments a multi > segment mbuf can have. The max number depends on the Tx offloads requested. > > The current code not enforce such limitation, which might cause > malformed work

Re: [dpdk-dev] vf init issue with patch igb_uio: issue FLR during open and release of device file

2017-09-15 Thread Yang, Qiming
Hi, Thomas We meet a urgent issue because of Shijith's patch. When use DPDK version 17.08, DPDK VF driver and kernel PF driver, i40e, ixgbe and qede(Harish said) VF all meet initial failure. This issue only occurred when use 17.08 igb_uio, vfio-pci and 17.05 igb_uio all works well. Considering t

Re: [dpdk-dev] [PATCH v3 2/5] gso: add TCP/IPv4 GSO support

2017-09-15 Thread Hu, Jiayu
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Friday, September 15, 2017 2:39 AM > To: Kavanagh, Mark B ; Hu, Jiayu > > Cc: dev@dpdk.org; Tan, Jianfeng > Subject: RE: [PATCH v3 2/5] gso: add TCP/IPv4 GSO support > > > > > -Original Message- > > From:

Re: [dpdk-dev] [PATCH v2] net/tap: fix flow and port commands

2017-09-15 Thread Pascal Mazon
Hi, The patch looks good globally, but I have a few comments/requests, inline. Best regards, Pascal On 14/09/2017 17:07, Ophir Munk wrote: > This commit fixes two bugs related to tap devices. The first bug occurs > when executing in testpmd the following flow rule assuming tap device has > 4 rx