[dpdk-dev] [PATCH] app/testpmd: remove unnecessary cast

2017-10-11 Thread Zhiyong Yang
Signed-off-by: Zhiyong Yang --- app/test-pmd/testpmd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index 69d64026c..4595e664c 100644 --- a/app/test-pmd/testpmd.c +++ b/app/test-pmd/testpmd.c @@ -1321,14 +1321,14 @@ stop_p

Re: [dpdk-dev] [PATCH v10 3/9] linuxapp/eal_pci: get iommu class

2017-10-11 Thread Tan, Jianfeng
> -Original Message- > From: santosh [mailto:santosh.shu...@caviumnetworks.com] > Sent: Wednesday, October 11, 2017 1:38 PM > To: Tan, Jianfeng; olivier.m...@6wind.com; dev@dpdk.org > Cc: tho...@monjalon.net; jerin.ja...@caviumnetworks.com; > hemant.agra...@nxp.com; acon...@redhat.com; >

[dpdk-dev] [PATCH 2/2] test: fix broken build because of prototype mismatch

2017-10-11 Thread Andrew Rybchenko
Fixes: a4b090d0284b ("ethdev: add return value to stats get dev op") Signed-off-by: Andrew Rybchenko --- test/test/virtual_pmd.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/test/test/virtual_pmd.c b/test/test/virtual_pmd.c index 6568cb5..09daf6c 100644 --- a/test/test/

[dpdk-dev] [PATCH 1/2] net/sfc: invert positive errno used internally on return

2017-10-11 Thread Andrew Rybchenko
Fixes: a4b090d0284b ("ethdev: add return value to stats get dev op") Signed-off-by: Andrew Rybchenko --- drivers/net/sfc/sfc_ethdev.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/sfc/sfc_ethdev.c b/drivers/net/sfc/sfc_ethdev.c index 5b089dc..bd09191 100644 --

Re: [dpdk-dev] [PATCH] l3fwd-acl: fix unchecked return value

2017-10-11 Thread Yang, Zhiyong
Hi Kuba, Just a soft reminder. :) Maintainers will make the decision. Ferruh, Thomas, Your opinion? Thanks Zhiyong > -Original Message- > From: Kozak, KubaX > Sent: Wednesday, October 11, 2017 2:58 PM > To: Yang, Zhiyong ; Ananyev, Konstantin > > Cc: dev@dpdk.org; sta...@dpdk.org;

Re: [dpdk-dev] [PATCH v2] net/ixgbe: fix VFIO interrupt mapping in VF

2017-10-11 Thread Dai, Wei
I look through from web browser and not find following mail from Jianwei Ma Hope this can add his test result in the mail list to community. > -Original Message- > From: Ma, Jianwei > Sent: Tuesday, October 10, 2017 4:47 PM > To: Dai, Wei ; Lu, Wenzhuo ; > Ananyev, Konstantin ; Liang, Cu

Re: [dpdk-dev] [PATCH v10 3/9] linuxapp/eal_pci: get iommu class

2017-10-11 Thread santosh
On Wednesday 11 October 2017 12:34 PM, Tan, Jianfeng wrote: > >> -Original Message- >> From: santosh [mailto:santosh.shu...@caviumnetworks.com] >> Sent: Wednesday, October 11, 2017 1:38 PM >> To: Tan, Jianfeng; olivier.m...@6wind.com; dev@dpdk.org >> Cc: tho...@monjalon.net; jerin.ja...@ca

Re: [dpdk-dev] [PATCH] net/ixgbe: fix Rx queue interrupt mapping in VF

2017-10-11 Thread Dai, Wei
I look through from web browser and not find following mail from Jianwei Ma Hope this can add his test result in the mail list to community. > -Original Message- > From: Ma, Jianwei > Sent: Tuesday, October 10, 2017 4:48 PM > To: Dai, Wei ; Lu, Wenzhuo ; > Ananyev, Konstantin > Cc: dev@

Re: [dpdk-dev] [PATCH v2] net/ixgbe: fix VFIO interrupt mapping in VF

2017-10-11 Thread Dai, Wei
Reply for correct mail format. > -Original Message- > From: Dai, Wei > Sent: Wednesday, October 11, 2017 3:07 PM > To: Ma, Jianwei ; Lu, Wenzhuo > ; Ananyev, Konstantin > ; Liang, Cunming > > Cc: dev@dpdk.org; sta...@dpdk.org > Subject: RE: [PATCH v2] net/ixgbe: fix VFIO interrupt mapping

[dpdk-dev] [PATCH] ethdev: fix xstats retrieve by id API

2017-10-11 Thread Lee Daly
From: Lee Fix xstats functions, rte_eth_xstats_get_names_by_id() and rte_eth_xstats_get_by_id(), in current implementation ethdev level reads all xstat values and filters out the ones requested by the application. This behavior doesn't benefit from PMD ops and doesn't provide the benefit the API

[dpdk-dev] [PATCH] net/liquidio: add support for promiscuous mode

2017-10-11 Thread Shijith Thotton
From: Intiyaz Basha Signed-off-by: Intiyaz Basha Signed-off-by: Shijith Thotton --- doc/guides/nics/features/liquidio.ini | 1 + doc/guides/nics/liquidio.rst| 8 ++ drivers/net/liquidio/base/lio_hw_defs.h | 3 +++ drivers/net/liquidio/lio_ethdev.c | 47 ++

Re: [dpdk-dev] [PATCH 04/10] net/i40e: fix compilation with -Og

2017-10-11 Thread Olivier MATZ
On Wed, Oct 11, 2017 at 06:20:35AM +, Wu, Jingjing wrote: > > > > -Original Message- > > From: Yigit, Ferruh > > Sent: Friday, October 6, 2017 7:25 AM > > To: Olivier Matz ; dev@dpdk.org; Wu, Jingjing > > ; Xing, Beilei > > Subject: Re: [dpdk-dev] [PATCH 04/10] net/i40e: fix compilat

[dpdk-dev] [PATCH] app/crypto-perf: add help option

2017-10-11 Thread Pablo de Lara
Signed-off-by: Pablo de Lara --- app/test-crypto-perf/cperf_options_parsing.c | 49 ++-- 1 file changed, 47 insertions(+), 2 deletions(-) diff --git a/app/test-crypto-perf/cperf_options_parsing.c b/app/test-crypto-perf/cperf_options_parsing.c index 72c515a..ad43e84 10064

Re: [dpdk-dev] [PATCH v10 3/9] linuxapp/eal_pci: get iommu class

2017-10-11 Thread Tan, Jianfeng
> > Then how about we enable this, iova=va, if only all PCI devices are binded > to vfio-pci (iommu mode)? > > Right, same I proposed (I guess) in v2 such that iova bus autodetecting in > case see all device bound > to vfio-pci then autoselect iova=va, in v3 series (I guess) discussion: it was >

Re: [dpdk-dev] [RFC 01/14] ethdev: add link status read/write functions

2017-10-11 Thread Yang, Qiming
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Stephen Hemminger > Sent: Saturday, July 15, 2017 2:30 AM > To: dev@dpdk.org > Cc: Stephen Hemminger ; Stephen Hemminger > > Subject: [dpdk-dev] [RFC 01/14] ethdev: add link status read/write functions > > Many drivers are all doing copy/past

Re: [dpdk-dev] [PATCH v6 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit

2017-10-11 Thread Pei, Yulong
Hi Jasvinder, -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jasvinder Singh Sent: Tuesday, October 10, 2017 3:08 AM To: dev@dpdk.org Cc: Dumitrescu, Cristian ; Wu, Jingjing ; De Lara Guarch, Pablo Subject: [dpdk-dev] [PATCH v6 3/3] app/test-pmd: add CLI for TM

Re: [dpdk-dev] [PATCH] doc: add description of traffic management functions in testpmd

2017-10-11 Thread Pei, Yulong
Hi Jasvinder, -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jasvinder Singh Sent: Wednesday, October 11, 2017 3:18 AM To: dev@dpdk.org Cc: Dumitrescu, Cristian ; Yigit, Ferruh ; Mcnamara, John Subject: [dpdk-dev] [PATCH] doc: add description of traffic manageme

Re: [dpdk-dev] [PATCH v6 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit

2017-10-11 Thread Singh, Jasvinder
> app/test-pmd/cmdline.c| 27 +++ > app/test-pmd/cmdline_tm.c | 526 > ++ > app/test-pmd/cmdline_tm.h | 5 + > 3 files changed, 558 insertions(+) > > diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index > d89583c..c6b35ba 100644 >

Re: [dpdk-dev] [PATCH v10 3/9] linuxapp/eal_pci: get iommu class

2017-10-11 Thread santosh
On Wednesday 11 October 2017 02:01 PM, Tan, Jianfeng wrote: >>> Then how about we enable this, iova=va, if only all PCI devices are binded >> to vfio-pci (iommu mode)? >> >> Right, same I proposed (I guess) in v2 such that iova bus autodetecting in >> case see all device bound >> to vfio-pci then

[dpdk-dev] [PATCH v8 0/2] net/i40e: API to configure queue regions for RSS

2017-10-11 Thread Wei Zhao
The patches mainly finish following functions: 1) Command line for input queue region configure parameters. 2) Set or flush queue region configutation. v2: -fix bug for packet into queue region code. -change code style for DPDK C programing. -change that "flowtype" name to " hw_flowtype " in stru

[dpdk-dev] [PATCH v8 2/2] app/testpmd: add API for configuration of queue region

2017-10-11 Thread Wei Zhao
This patch add a API configuration of queue region in rss. It can parse the parameters of region index, queue number, queue start index, user priority, traffic classes and so on. According to commands from command line, it will call i40e private API and start the process of set or flush queue regio

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

2017-10-11 Thread Wei Zhao
This feature enable queue regions configuration for RSS in PF, so that different traffic classes or different packet classification types can be separated to different queues in different queue regions.This patch can set queue region range, it include queue number in a region and the index of first

Re: [dpdk-dev] [PATCH v2 01/12] lib/rte_security: add security library

2017-10-11 Thread Ananyev, Konstantin
Hi Akhil, > -Original Message- > From: Akhil Goyal [mailto:akhil.go...@nxp.com] > Sent: Tuesday, October 10, 2017 1:17 PM > To: Ananyev, Konstantin ; dev@dpdk.org > Cc: Doherty, Declan ; De Lara Guarch, Pablo > ; hemant.agra...@nxp.com; > Nicolau, Radu ; bor...@mellanox.com; > avia...@me

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

2017-10-11 Thread Wei Zhao
This feature enable queue regions configuration for RSS in PF, so that different traffic classes or different packet classification types can be separated to different queues in different queue regions.This patch can set queue region range, it include queue number in a region and the index of first

[dpdk-dev] [PATCH v8 0/2] net/i40e: API to configure queue regions for RSS

2017-10-11 Thread Wei Zhao
The patches mainly finish following functions: 1) Command line for input queue region configure parameters. 2) Set or flush queue region configutation. v2: -fix bug for packet into queue region code. -change code style for DPDK C programing. -change that "flowtype" name to " hw_flowtype " in stru

[dpdk-dev] [PATCH v8 2/2] app/testpmd: add API for configuration of queue region

2017-10-11 Thread Wei Zhao
This patch add a API configuration of queue region in rss. It can parse the parameters of region index, queue number, queue start index, user priority, traffic classes and so on. According to commands from command line, it will call i40e private API and start the process of set or flush queue regio

[dpdk-dev] [PATCH 1/7] eventdev: add API to get service id

2017-10-11 Thread Pavan Nikhilesh
In case of sw event device the scheduling can be done on a service core using the service registered at the time of probe. This patch adds a helper function to get the service id that can be used by the application to assign a lcore for the service to run on. Signed-off-by: Pavan Nikhilesh --- l

[dpdk-dev] [PATCH 2/7] event/sw: extend service capability

2017-10-11 Thread Pavan Nikhilesh
Extend the service capability of the sw event device by exposing service id to the application. The application can use service id to configure service cores to run event scheduling. Signed-off-by: Pavan Nikhilesh --- drivers/event/sw/sw_evdev.c | 9 + 1 file changed, 9 insertions(+) di

[dpdk-dev] [PATCH 0/7] eventdev: remove event schedule API for SW driver

2017-10-11 Thread Pavan Nikhilesh
The software event dev is a centralized software scheduler and needs `rte_event_schedule` to be called repeatedly in order to distribute the events. In most cases, this requires a dedicated lcore to achieve this. With the introduction to the rte_service concept, software eventdev driver can regist

[dpdk-dev] [PATCH 3/7] app/test-eventdev: update app to use service cores

2017-10-11 Thread Pavan Nikhilesh
Use service cores for offloading event scheduling in case of centralized scheduling instead of calling the schedule api directly. This removes the dependency on dedicated scheduler core specified by giving command line option --slcore. Signed-off-by: Pavan Nikhilesh --- app/test-eventdev/evt_com

[dpdk-dev] [PATCH 4/7] test/eventdev: update test to use service core

2017-10-11 Thread Pavan Nikhilesh
Use service core for event scheduling instead of calling the event schedule api directly. Signed-off-by: Pavan Nikhilesh --- test/test/test_eventdev_sw.c | 120 --- 1 file changed, 67 insertions(+), 53 deletions(-) diff --git a/test/test/test_eventdev_sw.

[dpdk-dev] [PATCH 5/7] examples/eventdev: update sample app to use service

2017-10-11 Thread Pavan Nikhilesh
Update the sample app eventdev_pipeline_sw_pmd to use service cores for event scheduling in case of sw eventdev. Signed-off-by: Pavan Nikhilesh --- examples/eventdev_pipeline_sw_pmd/main.c | 51 +--- 1 file changed, 33 insertions(+), 18 deletions(-) diff --git a/exam

[dpdk-dev] [PATCH 7/7] doc: update software event device

2017-10-11 Thread Pavan Nikhilesh
Update software event device documentation to include use of service cores for event distribution. Signed-off-by: Pavan Nikhilesh --- doc/guides/eventdevs/sw.rst | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/doc/guides/eventdevs/sw.rst b/doc/guides/eventdevs/s

[dpdk-dev] [PATCH 6/7] eventdev: remove eventdev schedule API

2017-10-11 Thread Pavan Nikhilesh
remove eventdev schedule api and enforce sw driver to use service core feature for event scheduling. Signed-off-by: Pavan Nikhilesh --- drivers/event/octeontx/ssovf_evdev.c | 1 - drivers/event/skeleton/skeleton_eventdev.c | 2 -- drivers/event/sw/sw_evdev.c| 13 +

[dpdk-dev] [PATCH v7 3/3] app/test-pmd: add CLI for TM nodes and hierarchy commit

2017-10-11 Thread Jasvinder Singh
Add following CLIs in testpmd application for device traffic management; - commands to add TM hierarchy nodes (leaf and nonleaf). - command for runtime update of node weight. - command to commit the TM hierarchy Signed-off-by: Jasvinder Singh Acked-by: Cristian Dumitrescu --- v7 change: - fix th

[dpdk-dev] [PATCH v7 2/3] app/test-pmd: add CLI for shaper and wred profiles

2017-10-11 Thread Jasvinder Singh
Add following CLIs in testpmd application for device traffic management; - commands to add/del shaper profile for TM hieraqrchy nodes. - commands to add/update shared shapers - commands to add/del WRED profiles for TM hiearchy leaf nodes. Signed-off-by: Jasvinder Singh Acked-by: Cristian Dumitres

[dpdk-dev] [PATCH v7 1/3] app/test-pmd: add CLI for TM capability and stats

2017-10-11 Thread Jasvinder Singh
Add following CLIs to testpmd application for device traffic management; - commands to display TM capability information. (per port, per hierarchy level and per hierarchy node) - command to display hiearchy node type - stats collection Signed-off-by: Jasvinder Singh Acked-by: Cristian Dumitresc

Re: [dpdk-dev] [PATCH v1 1/7] ethdev: expose flow API error helper

2017-10-11 Thread Thomas Monjalon
05/10/2017 11:49, Adrien Mazarguil: > rte_flow_error_set() is a convenient helper to initialize error objects. > > Since there is no fundamental reason to prevent applications from using it, > expose it through the public interface after modifying its return value > from positive to negative. This

[dpdk-dev] [PATCH] crypto: use snprintf return value correctly

2017-10-11 Thread Tomasz Duszynski
snprintf return value is the length of was encoded into destination array excluding '\0'. Thus return value equal to the length of the destination array or more means truncation. This commit fixes improper use of the return value. Signed-off-by: Tomasz Duszynski --- drivers/crypto/aesni_gcm/aes

[dpdk-dev] next techboard meeting (13th, October)

2017-10-11 Thread Olivier MATZ
Hi all, The next techboard meeting will happen on IRC #dpdk-board, at 3pm UTC, this friday 13th of october. The agenda is updated here: https://annuel.framapad.org/p/r.0c3cc4d1e011214183872a98f6b5c7db 1) ABI stability: - one LTS per year - 17.11 to be a LTS - every new API has t

Re: [dpdk-dev] [PATCH] crypto: use snprintf return value correctly

2017-10-11 Thread Bruce Richardson
On Wed, Oct 11, 2017 at 11:25:40AM +0200, Tomasz Duszynski wrote: > snprintf return value is the length of was encoded into destination > array excluding '\0'. Thus return value equal to the length of the > destination array or more means truncation. > > This commit fixes improper use of the retur

Re: [dpdk-dev] [PATCH v1 0/7] Flow API helpers enhancements

2017-10-11 Thread Adrien Mazarguil
On Tue, Oct 10, 2017 at 07:05:30PM +0100, Ferruh Yigit wrote: > On 10/6/2017 9:05 AM, Adrien Mazarguil wrote: > > On Fri, Oct 06, 2017 at 02:13:14AM +0100, Ferruh Yigit wrote: > >> On 10/5/2017 10:49 AM, Adrien Mazarguil wrote: > >>> This series brings enhancements to various rte_flow helpers: > >>

[dpdk-dev] [PATCH] vhost: fix false-positive warning from clang 5

2017-10-11 Thread Bruce Richardson
When compiling with clang extra warning flags, such as used by default with meson, a warning is given in iotlb.c: ../lib/librte_vhost/iotlb.c:318:6: warning: variable 'socket' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized] This is a false positive, as the sock

Re: [dpdk-dev] [PATCH] crypto: use snprintf return value correctly

2017-10-11 Thread De Lara Guarch, Pablo
Hi, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce Richardson > Sent: Wednesday, October 11, 2017 10:58 AM > To: Tomasz Duszynski > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] crypto: use snprintf return value correctly > > On Wed, Oct 11, 2017 a

Re: [dpdk-dev] [PATCH] vhost: fix false-positive warning from clang 5

2017-10-11 Thread Maxime Coquelin
On 10/11/2017 11:59 AM, Bruce Richardson wrote: When compiling with clang extra warning flags, such as used by default with meson, a warning is given in iotlb.c: ../lib/librte_vhost/iotlb.c:318:6: warning: variable 'socket' is used uninitialized whenever 'if' condition is false [-Wsometimes-u

[dpdk-dev] [PATCH 0/2] enable 4KB + VFIO-PCI

2017-10-11 Thread Jianfeng Tan
Patch 1: Use VA as IOVA if IOVA mode is enabled. Patch 2: Enable IOVA mode for the PMDs for Intel NICs. How to test: $ (bind nic to vfio-pci) $ testpmd -c 0x3 -n 4 -m 2048 --no-huge -- -i --no-numa Jianfeng Tan (2): eal: honor IOVA mode for no-huge case net: enable IOVA mode for PMDs drive

[dpdk-dev] [PATCH 1/2] eal: honor IOVA mode for no-huge case

2017-10-11 Thread Jianfeng Tan
With the introduction of IOVA mode, the only blocker to run with 4KB pages for NICs binding to vfio-pci, is that RTE_BAD_PHYS_ADDR is not a valid IOVA address. We can refine this by using VA as IOVA if it's IOVA mode. Signed-off-by: Jianfeng Tan --- lib/librte_eal/linuxapp/eal/eal_memory.c | 5

[dpdk-dev] [PATCH 2/2] net: enable IOVA mode for PMDs

2017-10-11 Thread Jianfeng Tan
If we want to enable IOVA mode, introduced by commit 93878cf0255e ("eal: introduce helper API for IOVA mode"), we need PMDs (for PCI devices) to expose this flag. Signed-off-by: Jianfeng Tan --- drivers/net/e1000/em_ethdev.c | 3 ++- drivers/net/e1000/igb_ethdev.c| 5 +++-- drivers/net/f

Re: [dpdk-dev] [PATCH v2 00/26] DPDK PMD for OCTEONTX NW device

2017-10-11 Thread Thomas Monjalon
09/10/2017 04:07, Ferruh Yigit: > On 10/8/2017 1:44 PM, Santosh Shukla wrote: > > v2: > > Includes Minor changes like: > > - Incorporated minor review comment (Suggested y John) > > - Added correct pool handle ops name i.e. from mbuf-pool-ops to > > mbuf-pool-ops-name in the doc. > > - Rebased on

Re: [dpdk-dev] [PATCH] crypto: use snprintf return value correctly

2017-10-11 Thread Tomasz Duszynski
On Wed, Oct 11, 2017 at 10:09:02AM +, De Lara Guarch, Pablo wrote: > Hi, > > > -Original Message- > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce Richardson > > Sent: Wednesday, October 11, 2017 10:58 AM > > To: Tomasz Duszynski > > Cc: dev@dpdk.org > > Subject: Re: [dpd

Re: [dpdk-dev] [PATCH v5 3/5] bus: introduce new log type for bus drivers

2017-10-11 Thread Tan, Jianfeng
On 10/11/2017 2:54 PM, Shreyansh Jain wrote: Hello Jianfeng, On Monday 09 October 2017 04:57 PM, Jianfeng Tan wrote: [...] -#define PMD_DRV_LOG(level, fmt, args...) \ -PMD_DRV_LOG_RAW(level, fmt "\n", ## args) +#define FSLMC_VFIO_LOG(level, fmt, args...) \ +RTE_LOG(level, EAL, "%s(

Re: [dpdk-dev] [PATCH 2/2] net: enable IOVA mode for PMDs

2017-10-11 Thread Burakov, Anatoly
On 11-Oct-17 11:33 AM, Jianfeng Tan wrote: If we want to enable IOVA mode, introduced by commit 93878cf0255e ("eal: introduce helper API for IOVA mode"), we need PMDs (for PCI devices) to expose this flag. Signed-off-by: Jianfeng Tan --- Is this the complete list of drivers which need this fl

Re: [dpdk-dev] [PATCH v2] ethdev: add return value to stats get dev op

2017-10-11 Thread Thomas Monjalon
11/10/2017 04:09, Ferruh Yigit: > On 10/11/2017 3:01 AM, Ferruh Yigit wrote: > > On 10/10/2017 9:20 PM, Matan Azrad wrote: > >> The stats_get dev op API doesn't include return value, so PMD cannot > >> return an error in case of failure at stats getting process time. > >> > >> Since PCI devices can

Re: [dpdk-dev] [PATCH 0/2] enable 4KB + VFIO-PCI

2017-10-11 Thread Burakov, Anatoly
On 11-Oct-17 11:33 AM, Jianfeng Tan wrote: Patch 1: Use VA as IOVA if IOVA mode is enabled. Patch 2: Enable IOVA mode for the PMDs for Intel NICs. How to test: $ (bind nic to vfio-pci) $ testpmd -c 0x3 -n 4 -m 2048 --no-huge -- -i --no-numa Jianfeng Tan (2): eal: honor IOVA mode for no-huge

Re: [dpdk-dev] [PATCH 0/2] enable 4KB + VFIO-PCI

2017-10-11 Thread Thomas Monjalon
11/10/2017 12:47, Burakov, Anatoly: > On 11-Oct-17 11:33 AM, Jianfeng Tan wrote: > > Patch 1: Use VA as IOVA if IOVA mode is enabled. > > Patch 2: Enable IOVA mode for the PMDs for Intel NICs. [...] > > The patchset should probably mention its dependency on IOVA patches from > Santosh. No need b

Re: [dpdk-dev] [PATCH] vhost: fix false-positive warning from clang5

2017-10-11 Thread Thomas Monjalon
11/10/2017 12:12, Maxime Coquelin: > On 10/11/2017 11:59 AM, Bruce Richardson wrote: > > When compiling with clang extra warning flags, such as used by default with > > meson, a warning is given in iotlb.c: > > > > ../lib/librte_vhost/iotlb.c:318:6: warning: variable 'socket' is used > > uninitia

Re: [dpdk-dev] [PATCH 2/2] net: enable IOVA mode for PMDs

2017-10-11 Thread Tan, Jianfeng
On 10/11/2017 6:43 PM, Burakov, Anatoly wrote: On 11-Oct-17 11:33 AM, Jianfeng Tan wrote: If we want to enable IOVA mode, introduced by commit 93878cf0255e ("eal: introduce helper API for IOVA mode"), we need PMDs (for PCI devices) to expose this flag. Signed-off-by: Jianfeng Tan --- Is th

Re: [dpdk-dev] [PATCH v2 00/26] DPDK PMD for OCTEONTX NW device

2017-10-11 Thread Jerin Jacob
-Original Message- > Date: Wed, 11 Oct 2017 12:35:27 +0200 > From: Thomas Monjalon > To: Santosh Shukla > Cc: dev@dpdk.org, Ferruh Yigit , > jerin.ja...@caviumnetworks.com > Subject: Re: [dpdk-dev] [PATCH v2 00/26] DPDK PMD for OCTEONTX NW device > > 09/10/2017 04:07, Ferruh Yigit: > >

Re: [dpdk-dev] [PATCH] vhost: fix false-positive warning from clang5

2017-10-11 Thread Bruce Richardson
On Wed, Oct 11, 2017 at 12:51:59PM +0200, Thomas Monjalon wrote: > 11/10/2017 12:12, Maxime Coquelin: > > On 10/11/2017 11:59 AM, Bruce Richardson wrote: > > > When compiling with clang extra warning flags, such as used by default > > > with > > > meson, a warning is given in iotlb.c: > > > > > >

Re: [dpdk-dev] [PATCH 3/4] event/dpaa2: support event eth adapter

2017-10-11 Thread Nipun Gupta
> -Original Message- > From: Rao, Nikhil [mailto:nikhil@intel.com] > Sent: Wednesday, October 11, 2017 9:15 > To: Nipun Gupta ; dev@dpdk.org > Cc: Hemant Agrawal ; > jerin.ja...@caviumnetworks.com > Subject: Re: [PATCH 3/4] event/dpaa2: support event eth adapter > > > On 10/10/2017

Re: [dpdk-dev] [PATCH 3/4] event/dpaa2: support event eth adapter

2017-10-11 Thread Nipun Gupta
> -Original Message- > From: Rao, Nikhil [mailto:nikhil@intel.com] > Sent: Wednesday, October 11, 2017 9:40 > To: Nipun Gupta ; dev@dpdk.org > Cc: Hemant Agrawal ; > jerin.ja...@caviumnetworks.com > Subject: Re: [PATCH 3/4] event/dpaa2: support event eth adapter > > On 10/10/2017 11:

[dpdk-dev] [PATCH v2] crypto: use snprintf return value correctly

2017-10-11 Thread Tomasz Duszynski
snprintf return value is the length of was encoded into destination array excluding '\0'. Thus return value equal to the length of the destination array or more means truncation. This commit fixes improper use of the return value. Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto ope

Re: [dpdk-dev] [PATCH v5 3/5] bus: introduce new log type for bus drivers

2017-10-11 Thread Shreyansh Jain
On Wednesday 11 October 2017 04:12 PM, Tan, Jianfeng wrote: On 10/11/2017 2:54 PM, Shreyansh Jain wrote: Hello Jianfeng, On Monday 09 October 2017 04:57 PM, Jianfeng Tan wrote: [...] -#define PMD_DRV_LOG(level, fmt, args...) \ -    PMD_DRV_LOG_RAW(level, fmt "\n", ## args) +#define FSLMC_

Re: [dpdk-dev] [PATCH 1/2] eal: honor IOVA mode for no-huge case

2017-10-11 Thread Burakov, Anatoly
On 11-Oct-17 11:33 AM, Jianfeng Tan wrote: With the introduction of IOVA mode, the only blocker to run with 4KB pages for NICs binding to vfio-pci, is that RTE_BAD_PHYS_ADDR is not a valid IOVA address. We can refine this by using VA as IOVA if it's IOVA mode. Signed-off-by: Jianfeng Tan ---

[dpdk-dev] [PATCH v2] vhost: fix false-positive warning from clang 5

2017-10-11 Thread Bruce Richardson
When compiling with clang extra warning flags, such as used by default with meson, a warning is given in iotlb.c: ../lib/librte_vhost/iotlb.c:318:6: warning: variable 'socket' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized] This is a false positive, as the sock

Re: [dpdk-dev] [PATCH v8 0/2] net/i40e: API to configure queue regions for RSS

2017-10-11 Thread Peng, Yuan
Tested-by: Peng Yuan - Tested Branch: dpdk-master - Tested Commit: 7a8889324654c9e39f9e18097ccc74d6ff2588cf - OS: 4.5.5-300.fc24.x86_64 - GCC: gcc (GCC) 5.3.1 20151207 (Red Hat 5.3.1-2) - CPU: Intel(R) Xeon(R) CPU E5-2699 v4 @ 2.20GHz - NIC: Intel Corporation Device Fortville [8086:1572] - Defaul

Re: [dpdk-dev] [PATCH 2/2] net: enable IOVA mode for PMDs

2017-10-11 Thread Burakov, Anatoly
On 11-Oct-17 11:33 AM, Jianfeng Tan wrote: If we want to enable IOVA mode, introduced by commit 93878cf0255e ("eal: introduce helper API for IOVA mode"), we need PMDs (for PCI devices) to expose this flag. Signed-off-by: Jianfeng Tan --- Acked-by: Anatoly Burakov -- Thanks, Anatoly

Re: [dpdk-dev] [PATCH 1/2] eal: honor IOVA mode for no-huge case

2017-10-11 Thread santosh
On Wednesday 11 October 2017 04:03 PM, Jianfeng Tan wrote: > With the introduction of IOVA mode, the only blocker to run > with 4KB pages for NICs binding to vfio-pci, is that > RTE_BAD_PHYS_ADDR is not a valid IOVA address. > > We can refine this by using VA as IOVA if it's IOVA mode. > > Signed-

[dpdk-dev] [PATCH 2/3] net/octeontx: fix 32bit build with driver debug enabled

2017-10-11 Thread Jerin Jacob
squash this to next-next Fixes: 57b01943ae4f ("net/octeontx/base: add remaining PKO operations") Reported-by: Thomas Monjalon Signed-off-by: Jerin Jacob --- drivers/net/octeontx/base/octeontx_pkovf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/octeontx/b

[dpdk-dev] [PATCH 1/3] net/octeontx: fix 32bit build with driver debug enabled

2017-10-11 Thread Jerin Jacob
squash this to next-next Fixes: 7c025f92e9f4 ("net/octeontx/base: add base PKO operations") Reported-by: Thomas Monjalon Signed-off-by: Jerin Jacob --- drivers/net/octeontx/base/octeontx_pkovf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/octeontx/base/oct

[dpdk-dev] [PATCH 3/3] net/octeontx: fix 32bit build with driver debug enabled

2017-10-11 Thread Jerin Jacob
squash this to next-next Fixes: 9198e27bca84 ("net/octeontx: add basic stats support") Reported-by: Thomas Monjalon Signed-off-by: Jerin Jacob --- drivers/net/octeontx/octeontx_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/octeontx/octeontx_ethdev.c

Re: [dpdk-dev] [PATCH 2/2] net: enable IOVA mode for PMDs

2017-10-11 Thread santosh
On Wednesday 11 October 2017 04:03 PM, Jianfeng Tan wrote: > If we want to enable IOVA mode, introduced by > commit 93878cf0255e ("eal: introduce helper API for IOVA mode"), > we need PMDs (for PCI devices) to expose this flag. > > Signed-off-by: Jianfeng Tan > --- Reviewed-by: Santosh Shukla

[dpdk-dev] [PATCH v3] eal: fix resource leak

2017-10-11 Thread Daniel Mrzyglod
Memory allocated in strdup is not free. Coverity issue: 143257 Fixes: d8a2bc71dfc2 ("log: remove app path from syslog id") Cc: tho...@monjalon.net Cc: sta...@dpdk.org Signed-off-by: Daniel Mrzyglod --- v3: * remove strdup because it's basically striped argv[0] v2: * Fix due to compilation error

Re: [dpdk-dev] [PATCH v1 1/7] ethdev: expose flow API error helper

2017-10-11 Thread Adrien Mazarguil
On Wed, Oct 11, 2017 at 11:23:12AM +0200, Thomas Monjalon wrote: > 05/10/2017 11:49, Adrien Mazarguil: > > rte_flow_error_set() is a convenient helper to initialize error objects. > > > > Since there is no fundamental reason to prevent applications from using it, > > expose it through the public i

Re: [dpdk-dev] next techboard meeting (13th, October)

2017-10-11 Thread Wiles, Keith
> On Oct 11, 2017, at 4:36 AM, Olivier MATZ wrote: > > Hi all, > > The next techboard meeting will happen on IRC #dpdk-board, at 3pm UTC, > this friday 13th of october. > > The agenda is updated here: >https://annuel.framapad.org/p/r.0c3cc4d1e011214183872a98f6b5c7db > > > 1) ABI stabili

Re: [dpdk-dev] [dpdk-stable] [PATCH v3] eal: fix resource leak

2017-10-11 Thread Thomas Monjalon
11/10/2017 13:53, Daniel Mrzyglod: > Memory allocated in strdup is not free. > > Coverity issue: 143257 > Fixes: d8a2bc71dfc2 ("log: remove app path from syslog id") > Cc: tho...@monjalon.net > Cc: sta...@dpdk.org > > Signed-off-by: Daniel Mrzyglod Given this recent commit: http://dpdk.

Re: [dpdk-dev] [dpdk-techboard] next techboard meeting (13th, October)

2017-10-11 Thread Thomas Monjalon
11/10/2017 14:41, Wiles, Keith: > > On Oct 11, 2017, at 4:36 AM, Olivier MATZ wrote: > > 1) ABI stability: > >- one LTS per year > >- 17.11 to be a LTS > >- every new API has the "experimental" tag > >- 3 Acks are needed for deprecation and they should come from different > > comp

Re: [dpdk-dev] [PATCH v2] vhost: fix false-positive warning from clang 5

2017-10-11 Thread Thomas Monjalon
11/10/2017 13:28, Bruce Richardson: > When compiling with clang extra warning flags, such as used by default with > meson, a warning is given in iotlb.c: > > ../lib/librte_vhost/iotlb.c:318:6: warning: variable 'socket' is used > uninitialized whenever 'if' condition is false [-Wsometimes-uniniti

Re: [dpdk-dev] [dpdk-techboard] next techboard meeting (13th, October)

2017-10-11 Thread Wiles, Keith
> On Oct 11, 2017, at 7:48 AM, Thomas Monjalon wrote: > > 11/10/2017 14:41, Wiles, Keith: >>> On Oct 11, 2017, at 4:36 AM, Olivier MATZ wrote: >>> 1) ABI stability: >>> - one LTS per year >>> - 17.11 to be a LTS >>> - every new API has the "experimental" tag >>> - 3 Acks are needed for

Re: [dpdk-dev] [PATCH] eal: change init macro as exec environment specific

2017-10-11 Thread Thomas Monjalon
07/08/2017 14:04, Jerin Jacob: > baremetal execution environments may have a different > method to enable RTE_INIT instead of using compiler > constructor scheme. Move RTE_INIT* definition under > exec-env to support different execution environments. > > Signed-off-by: Jerin Jacob > --- > app/te

Re: [dpdk-dev] [PATCH v2] ethdev: add return value to stats get dev op

2017-10-11 Thread Andrew Rybchenko
On 10/11/2017 01:46 PM, Thomas Monjalon wrote: 11/10/2017 04:09, Ferruh Yigit: On 10/11/2017 3:01 AM, Ferruh Yigit wrote: On 10/10/2017 9:20 PM, Matan Azrad wrote: The stats_get dev op API doesn't include return value, so PMD cannot return an error in case of failure at stats getting process t

Re: [dpdk-dev] [PATCH v3] eal: add counter size for efd clean

2017-10-11 Thread Thomas Monjalon
Hi, 24/08/2017 04:10, Jingjing Wu: > } else if (intr_handle->type == RTE_INTR_HANDLE_VDEV) { > - /* do nothing, and let vdev driver to initialize this struct > */ > + /* only check, initialization would be done in vdev driver.*/ > + if (intr_handl

Re: [dpdk-dev] [PATCH v2] ethdev: add return value to stats get dev op

2017-10-11 Thread Thomas Monjalon
11/10/2017 15:09, Andrew Rybchenko: > On 10/11/2017 01:46 PM, Thomas Monjalon wrote: > > 11/10/2017 04:09, Ferruh Yigit: > >> On 10/11/2017 3:01 AM, Ferruh Yigit wrote: > >>> On 10/10/2017 9:20 PM, Matan Azrad wrote: > The stats_get dev op API doesn't include return value, so PMD cannot >

[dpdk-dev] DPDK Meet-up in Boston area

2017-10-11 Thread Dave Neary
Hi everyone, We are hosting another DPDK meet-up in the Boston area next week, Wednesday October 18th - this time in the Red Hat offices in Westford, MA. You can find more information about the meet-up, including (when I put them up) our speaker line-up. https://www.meetup.com/BostonSoftwareNe

Re: [dpdk-dev] [dpdk-users] VLAN tags always stripped on i40evf [VMware SR-IOV]

2017-10-11 Thread Iain Barker
On Tuesday, October 10, 2017 9:49 AM (EST), Iain Barker wrote: > I have a problem trying to get VLAN tagged frames to be received at the > i40evf PMD. With more debugging enabled, I can see that this seems to be a compatibility problem between DPDK and i40evf related to VLAN hardware stripping

[dpdk-dev] [PATCH] devtools: rename build dependency of mlx drivers

2017-10-11 Thread Thomas Monjalon
The Mellanox drivers were requiring MOFED at compilation time. It is now possible to use the upstream rdma-core package. So the dependency option is renamed in the build tool. Fixes: 43e9d9794cde ("net/mlx5: support upstream rdma-core") Signed-off-by: Thomas Monjalon --- devtools/test-build.sh

Re: [dpdk-dev] [PATCH v4 02/16] net/mrvl: add mrvl net pmd driver skeleton

2017-10-11 Thread Thomas Monjalon
Hi, 09/10/2017 17:00, Tomasz Duszynski: > # > +# Compile Marvell PMD driver > +# > +CONFIG_RTE_LIBRTE_MRVL_PMD=n > +CONFIG_RTE_LIBRTE_MRVL_DEBUG=n > +CONFIG_RTE_MRVL_MUSDK_DMA_MEMSIZE=41943040 We are trying to remove build-time configurations and behave like a true library. The DEBUG config opt

Re: [dpdk-dev] [PATCH] examples/vhost_scsi: fix buffer not terminated

2017-10-11 Thread Jastrzebski, MichalX K
> -Original Message- > From: Jastrzebski, MichalX K > Sent: Monday, October 2, 2017 3:53 PM > To: Jastrzebski, MichalX K ; > y...@fridaylinux.org; maxime.coque...@redhat.com > Cc: dev@dpdk.org; Jain, Deepak K ; Piasecki, > JacekX ; Liu, Changpeng > ; sta...@dpdk.org > Subject: RE: [dpdk-dev

Re: [dpdk-dev] [PATCH] examples/performance-thread: fix out-of-bounds read

2017-10-11 Thread Jastrzebski, MichalX K
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Slawomir > Mrozowicz > Sent: Wednesday, September 20, 2017 10:06 AM > To: Mcnamara, John > Cc: dev@dpdk.org; Mrozowicz, SlawomirX > ; ian.be...@intel.com; sta...@dpdk.org > Subject: [dpdk-dev] [PATCH] examples/perf

Re: [dpdk-dev] [PATCH] examples/performance-thread: fix out-of-bounds read

2017-10-11 Thread Jastrzebski, MichalX K
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jastrzebski, > MichalX K > Sent: Monday, October 2, 2017 4:08 PM > To: Mrozowicz, SlawomirX ; Mcnamara, > John > Cc: dev@dpdk.org; Mrozowicz, SlawomirX > ; ian.be...@intel.com; sta...@dpdk.org > Subject: Re: [dpdk-

Re: [dpdk-dev] [PATCH] examples/performance-thread: fix out-of-bounds read

2017-10-11 Thread Jastrzebski, MichalX K
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Slawomir > Mrozowicz > Sent: Wednesday, September 20, 2017 10:20 AM > To: Mcnamara, John > Cc: dev@dpdk.org; Mrozowicz, SlawomirX > ; ian.be...@intel.com; sta...@dpdk.org > Subject: [dpdk-dev] [PATCH] examples/perf

Re: [dpdk-dev] [PATCH] examples/performance-thread: fix out-of-bounds read

2017-10-11 Thread Jastrzebski, MichalX K
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Slawomir > Mrozowicz > Sent: Wednesday, September 20, 2017 10:21 AM > To: Mcnamara, John > Cc: dev@dpdk.org; Mrozowicz, SlawomirX > ; ian.be...@intel.com; sta...@dpdk.org > Subject: [dpdk-dev] [PATCH] examples/perf

Re: [dpdk-dev] [PATCH v2 00/14] Move PCI away from the EAL

2017-10-11 Thread Doherty, Declan
On 18/09/2017 10:31 AM, Gaetan Rivet wrote: Hi all, Here is a new version of the PCI bus move out of the EAL. The EAL PCI implementation is divided in two parts: - librte_pci: library offering helpers to handle PCI objects - librte_bus_pci: bus driver for PCI devices This allows other l

Re: [dpdk-dev] [PATCH V2 0/5] table: add key mask for hash tables

2017-10-11 Thread Dumitrescu, Cristian
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Cristian Dumitrescu > Sent: Tuesday, October 10, 2017 12:19 PM > To: dev@dpdk.org > Subject: [dpdk-dev] [PATCH V2 0/5] table: add key mask for hash tables > > Main changes: > > 1. The key_mask parameter is added

Re: [dpdk-dev] [PATCH v4 16/16] doc: add mrvl net pmd documentation

2017-10-11 Thread Thomas Monjalon
09/10/2017 17:00, Tomasz Duszynski: > +- MUSDK (Marvell User-Space SDK) sources available > + `here > `__. > + > +MUSDK is a light-weight library that provides direct access to Marvell's > +PPv2 (Packet Pr

Re: [dpdk-dev] [PATCH v2 00/14] Move PCI away from the EAL

2017-10-11 Thread Gaëtan Rivet
On Wed, Oct 11, 2017 at 03:19:31PM +0100, Doherty, Declan wrote: > On 18/09/2017 10:31 AM, Gaetan Rivet wrote: > >Hi all, > > > >Here is a new version of the PCI bus move out of the EAL. > > > >The EAL PCI implementation is divided in two parts: > > > > - librte_pci: library offering helpers to h

Re: [dpdk-dev] [PATCH] eal: change init macro as exec environment specific

2017-10-11 Thread Jerin Jacob
-Original Message- > Date: Wed, 11 Oct 2017 15:02:51 +0200 > From: Thomas Monjalon > To: Jerin Jacob > Cc: dev@dpdk.org, bruce.richard...@intel.com > Subject: Re: [dpdk-dev] [PATCH] eal: change init macro as exec environment > specific > > 07/08/2017 14:04, Jerin Jacob: > > baremetal ex

Re: [dpdk-dev] [PATCH] net/e1000: i350 VLAN tags in loopback packets are big endian

2017-10-11 Thread Roger B. Melton
Hi Wenzhou, On 10/10/17 9:32 PM, Lu, Wenzhuo wrote: Hi Roger, -Original Message- From: Roger B Melton [mailto:rmel...@cisco.com] Sent: Saturday, October 7, 2017 7:34 AM To: Lu, Wenzhuo Cc: dev@dpdk.org; Roger B Melton Subject: [PATCH] net/e1000: i350 VLAN tags in loopback packets are

[dpdk-dev] [PATCH v1 00/29] net/mlx4: restore PMD functionality

2017-10-11 Thread Adrien Mazarguil
This series restores all the control path functionality removed in prior series "net/mlx4: trim and refactor entire PMD", including: - Promiscuous mode. - All multicast mode. - MAC address configuration. - Support for multiple simultaneous MAC addresses. - Reception of broadcast and user-defined m

[dpdk-dev] [PATCH v1 01/29] ethdev: expose flow API error helper

2017-10-11 Thread Adrien Mazarguil
rte_flow_error_set() is a convenient helper to initialize error objects. Since there is no fundamental reason to prevent applications from using it, expose it through the public interface after modifying its return value from positive to negative. This is done for consistency with the rest of the

  1   2   3   >