Re: [dpdk-dev] [PATCH v1 4/9] app/procinfo: add code for debug port

2018-10-26 Thread Varghese, Vipin
> > + printf("\t -- RSS len %u key (hex):", > > Why "(hex)" in printf? In the follow up lines, we are disapplying the RSS key used in HEX values. for (k = 0; k < rss_conf.rss_key_len; k++) printf(" %x", rss_conf.rss_key[k]);

Re: [dpdk-dev] [PATCH v2 4/9] app/procinfo: add code for debug port

2018-10-26 Thread Varghese, Vipin
Thanks, I found the correction for the error. I will send the v3. Since this information was missed out in ' 5.10. Checking Compilation' I will send a separate patch trying to update documentation. > -Original Message- > From: Pattan, Reshma > Sent: Thursday, October 25, 2018 9:51 PM > T

Re: [dpdk-dev] [PATCH v2 6/9] app/procinfo: add code for debug crypto

2018-10-26 Thread Varghese, Vipin
Hi, > --- > + struct rte_cryptodev_info dev_info = {0}; > + struct rte_cryptodev_stats stats = {0}; > + > > Memset for initialization as mentioned in other patch. > Yes, I will correct the same as certain compiler flag combination will treat this as incorrect use. >

Re: [dpdk-dev] [PATCH v2 6/9] app/procinfo: add code for debug crypto

2018-10-26 Thread Varghese, Vipin
Hi Reshma, > > +"\t -- id (%u) flags (0x%"PRIx64") socket (%d)\n" > +"\t -- queue pairs (%d)\n", > +rte_cryptodev_name_get(i), > +dev_info.driver_name, dev_info.driver_id, > +dev_info.feature_f

[dpdk-dev] [PATCH v2] bus/pci: fix unexpected resource mapping override

2018-10-26 Thread Qi Zhang
When scanning an already plugged device, the virtual address of mapped PCI resource in rte_pci_device will be overridden with 0, that may cause driver does not work correctly. The fix is not to update any rte_pci_device's field if the being scanned device's driver is already probed. Bugzilla ID: 8

Re: [dpdk-dev] [PATCH 2/2] net/i40e: correct offload not supported mask

2018-10-26 Thread Zhang, Qi Z
> -Original Message- > From: Ye, Xiaolong > Sent: Friday, October 26, 2018 1:33 AM > To: Zhang, Qi Z ; Xing, Beilei > Cc: dev@dpdk.org; Ye, Xiaolong ; Kulasek, TomaszX > > Subject: [PATCH 2/2] net/i40e: correct offload not supported mask > > Just as the name I40E_TX_OFFLOAD_NOTSUP_MAS

Re: [dpdk-dev] [PATCH] net/ixgbe: enable detach from secondary

2018-10-26 Thread Zhang, Qi Z
> -Original Message- > From: Zhang, Qi Z > Sent: Wednesday, October 24, 2018 9:49 PM > To: Lu, Wenzhuo > Cc: Lin, Xueqin ; dev@dpdk.org; Zhang, Qi Z > > Subject: [PATCH] net/ixgbe: enable detach from secondary > > Since we have enabled the hotplug mechanism for multi-process, it's not

Re: [dpdk-dev] [PATCH v9 03/12] telemetry: initial telemetry infrastructure

2018-10-26 Thread Thomas Monjalon
27/10/2018 04:19, Van Haaren, Harry: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 27/10/2018 01:59, Harry van Haaren: > > > --- a/mk/rte.vars.mk > > > +++ b/mk/rte.vars.mk > > > +JANSSON := $(shell pkg-config --exists jansson; echo $$?) > > > +ifneq ($(JANSSON),0) > > > +$(warning Janss

Re: [dpdk-dev] [PATCH v9 03/12] telemetry: initial telemetry infrastructure

2018-10-26 Thread Van Haaren, Harry
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Friday, October 26, 2018 6:56 PM > To: Van Haaren, Harry ; Laatz, Kevin > ; Nicolau, Radu ; Hunt, David > > Cc: dev@dpdk.org; Richardson, Bruce ; > step...@networkplumber.org; gaetan.ri...@6wind.com; shrey

Re: [dpdk-dev] [PATCH v9 03/12] telemetry: initial telemetry infrastructure

2018-10-26 Thread Thomas Monjalon
27/10/2018 01:59, Harry van Haaren: > --- a/mk/rte.vars.mk > +++ b/mk/rte.vars.mk > +JANSSON := $(shell pkg-config --exists jansson; echo $$?) > +ifneq ($(JANSSON),0) > +$(warning Jansson not found, disabling RTE_LIBRTE_TELEMETRY) > +CONFIG_RTE_LIBRTE_TELEMETRY = n > +endif It fails for cross-comp

[dpdk-dev] [PATCH v2] lib/compressdev: Fix logic error on op allocation

2018-10-26 Thread Fiona Trahe
Fixed bad logic in rte_comp_op_alloc() checking return value from rte_comp_op_raw_bulk_alloc(). This could have resulted in a seg-fault in error case. Made rte_comp_ob_bulk_alloc() code consistent with rte_comp_op_alloc(). Fixes: 96086db5a369 ("compressdev: add operation management") Cc: sta...@d

Re: [dpdk-dev] [PATCH] lib/compressdev: Fix logic error on op allocation

2018-10-26 Thread Thomas Monjalon
27/10/2018 02:17, Fiona Trahe: > Fixed bad logic in rte_comp_op_alloc() checking return > value from rte_comp_op_raw_bulk_alloc(). This > could have resulted in a seg-fault in error case. > Made rte_comp_ob_bulk_alloc() code consistent > with rte_comp_op_alloc(). > > Thanks to Sabyasachi Sengupta

[dpdk-dev] [PATCH] lib/compressdev: Fix logic error on op allocation

2018-10-26 Thread Fiona Trahe
Fixed bad logic in rte_comp_op_alloc() checking return value from rte_comp_op_raw_bulk_alloc(). This could have resulted in a seg-fault in error case. Made rte_comp_ob_bulk_alloc() code consistent with rte_comp_op_alloc(). Thanks to Sabyasachi Sengupta for pointing it out. Fixes: 96086db5a369 ("c

Re: [dpdk-dev] [PATCH] common/qat: fix for invalid response from firmware

2018-10-26 Thread Thomas Monjalon
24/10/2018 17:11, Kusztal, ArkadiuszX: > From: Trahe, Fiona > > > > Check that the firmware response has a bit set indicating it's valid before > > dereferencing the rest of the response contents. > > > > Fixes: 0bdd36e12245 ("crypto/qat: make dequeue function generic") > > Cc: sta...@dpdk.org >

[dpdk-dev] [PATCH] lib/compressdev: clarify usage of op structure

2018-10-26 Thread Fiona Trahe
Add note on usage of op structure and when it can be accessed and freed. Fixes: 63f4bfd5328b ("compressdev: add enqueue/dequeue functions") Cc: sta...@dpdk.org Signed-off-by: Fiona Trahe --- lib/librte_compressdev/rte_compressdev.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/lib/

Re: [dpdk-dev] [PATCH v7] compress/qat: enable dynamic huffman encoding

2018-10-26 Thread Thomas Monjalon
26/10/2018 20:18, Fiona Trahe: > Enable dynamic huffman encoding in the QAT comp PMD. > > Signed-off-by: Tomasz Jozwiak > Signed-off-by: Fiona Trahe > Acked-by: Arek Kusztal Applied, thanks

Re: [dpdk-dev] [PATCH v2] buildtools: fix build for some mktemp

2018-10-26 Thread Thomas Monjalon
27/10/2018 01:53, Ferruh Yigit: > On 10/27/2018 12:47 AM, Thomas Monjalon wrote: > > 27/10/2018 02:38, Ferruh Yigit: > >> build error: > >> == Build drivers/net/tap > >> mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: > >> Invalid argument > >> .../buildtools/auto-config-h.sh: lin

[dpdk-dev] [PATCH v9 10/12] doc: add telemetry documentation

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch adds all documentation for telemetry. A description on how to use the Telemetry API with a DPDK application is given in this document. It also adds the MAINTAINERS file entry and a release notes update for telemetry. Signed-off-by: Ciara Power Signed-off-by: Brian

[dpdk-dev] [PATCH v9 09/12] telemetry: add ability to disable selftest

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch adds functionality to enable/disable the selftest. This functionality will be extended in future to make the enabling/disabling more dynamic and remove this 'hardcoded' approach. We are temporarily using this approach due to the design changes (vdev vs eal) made to t

[dpdk-dev] [PATCH v9 06/12] telemetry: add parser for client socket messages

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch adds the parser file. This is used to parse any messages that are received on any of the client sockets. Currently, the unregister functionality works using the parser. Functionality relating to getting statistic values for certain ports will be added in a subsequent

[dpdk-dev] [PATCH v9 12/12] build: add dependency on telemetry to apps in meson

2018-10-26 Thread Harry van Haaren
From: Kevin Laatz This patch adds telemetry as a dependecy to all applications. Without these changes, the --telemetry flag will not be recognised and applications will fail to run if they want to enable telemetry. Signed-off-by: Bruce Richardson Signed-off-by: Kevin Laatz Signed-off-by: Radu

[dpdk-dev] [PATCH v9 11/12] usertools: add client python script for telemetry

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch adds a python script which can be used as a demo client. The script is interactive and will allow the user to register, request statistics, and unregister. To run the script, an argument for the client file path must be passed in: "python telemetry_client.py ". This

[dpdk-dev] [PATCH v9 05/12] telemetry: add client feature and sockets

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch introduces clients to the telemetry API. When a client makes a connection through the initial telemetry socket, they can send a message through the socket to be parsed. Register messages are expected through this socket, to enable clients to register and have a clien

[dpdk-dev] [PATCH v9 08/12] telemetry: format json response when sending stats

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch adds functionality to create a JSON message in order to send it to a client socket. When stats are requested by a client, they are retrieved from the metrics library and encoded in JSON format. Signed-off-by: Ciara Power Signed-off-by: Brian Archbold Signed-off-by

[dpdk-dev] [PATCH v9 07/12] telemetry: update metrics before sending stats

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch adds functionality to update the statistics in the metrics library with values from the ethdev stats. Values need to be updated before they are encoded into a JSON message and sent to the client that requested them. The JSON encoding will be added in a subsequent pat

[dpdk-dev] [PATCH v9 04/12] telemetry: add initial connection socket

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch adds the telemetry UNIX socket. It is used to allow connections from external clients. On the initial connection from a client, ethdev stats are registered in the metrics library, to allow for their retrieval at a later stage. Signed-off-by: Ciara Power Signed-off-

[dpdk-dev] [PATCH v9 03/12] telemetry: initial telemetry infrastructure

2018-10-26 Thread Harry van Haaren
From: Ciara Power This patch adds the infrastructure and initial code for the telemetry library. The telemetry init is registered with eal_init(). We can then check to see if --telemetry was passed as an eal option. If --telemetry was parsed, then we call telemetry init at the end of eal init.

[dpdk-dev] [PATCH v9 00/12] Introduce Telemetry Library

2018-10-26 Thread Harry van Haaren
This patchset introduces a Telemetry library for DPDK Service Assurance. This library provides an easy way to query DPDK Ethdev metrics. The telemetry library provides a method for a service assurance component to retrieve metrics from a DPDK packet forwarding application. Communicating from the s

[dpdk-dev] [PATCH v9 02/12] eal: make get runtime dir function public

2018-10-26 Thread Harry van Haaren
From: Kevin Laatz This patch makes the eal_get_runtime_dir() API public so it can be used from outside EAL. Signed-off-by: Kevin Laatz Acked-by: Harry van Haaren --- lib/librte_eal/bsdapp/eal/eal.c | 2 +- lib/librte_eal/common/eal_filesystem.h | 15 --- lib/librte_eal/c

[dpdk-dev] [PATCH v9 01/12] eal: add option register infrastructure

2018-10-26 Thread Harry van Haaren
From: Kevin Laatz This commit adds infrastructure to EAL that allows an application to register it's init function with EAL. This allows libraries to be initialized at the end of EAL init. This infrastructure allows libraries that depend on EAL to be initialized as part of EAL init, removing cir

Re: [dpdk-dev] [PATCH v2] buildtools: fix build for some mktemp

2018-10-26 Thread Ferruh Yigit
On 10/27/2018 12:47 AM, Thomas Monjalon wrote: > 27/10/2018 02:38, Ferruh Yigit: >> build error: >> == Build drivers/net/tap >> mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: >> Invalid argument >> .../buildtools/auto-config-h.sh: line 86: : No such file or directory >> .../drive

Re: [dpdk-dev] [PATCH v2] buildtools: fix build for some mktemp

2018-10-26 Thread Thomas Monjalon
27/10/2018 02:38, Ferruh Yigit: > build error: > == Build drivers/net/tap > mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: > Invalid argument > .../buildtools/auto-config-h.sh: line 86: : No such file or directory > .../drivers/net/tap/Makefile:55: recipe for target >'tap_aut

[dpdk-dev] [PATCH v2] buildtools: fix build for some mktemp

2018-10-26 Thread Ferruh Yigit
build error: == Build drivers/net/tap mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: Invalid argument .../buildtools/auto-config-h.sh: line 86: : No such file or directory .../drivers/net/tap/Makefile:55: recipe for target 'tap_autoconf.h.new' failed Above error observed on W

Re: [dpdk-dev] [PATCH] buildtools: fix build for some mktemp

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 8:59 PM, Thomas Monjalon wrote: > 26/10/2018 20:31, Ferruh Yigit: >> On 10/26/2018 6:59 PM, Thomas Monjalon wrote: >>> 26/10/2018 20:42, Ferruh Yigit: build error: == Build drivers/net/tap mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: Invalid ar

Re: [dpdk-dev] [PATCH] kni: fix build on Linux < 3.14

2018-10-26 Thread Thomas Monjalon
27/10/2018 00:42, Ferruh Yigit: > On 10/26/2018 10:56 PM, Thomas Monjalon wrote: > > 26/10/2018 23:40, Thomas Monjalon: > >> The atomic functions smp_load_acquire() and smp_store_release() > >> were introduced in Linux 3.14. Older kernels miss the functions: > >> > >> kni_fifo.h:19:2: error: > >>

Re: [dpdk-dev] [PATCH] examples/l3fwd-power: fix build with icc

2018-10-26 Thread Thomas Monjalon
27/10/2018 00:13, Ferruh Yigit: > On 10/26/2018 11:00 PM, Thomas Monjalon wrote: > > main.c(376): error #592: variable "lcore_id" is used before its value is set > > RTE_SET_USED(lcore_id); > > ^ > > > > The variables were voided with RTE_SET_USED without an obvious reason. > > Rem

Re: [dpdk-dev] [PATCH 2/2] net/mlx5: fix detection and error for multiple item layers

2018-10-26 Thread Yongseok Koh
> On Oct 25, 2018, at 1:53 AM, Shahaf Shuler wrote: > > 1. The check for the Eth item was wrong. causing an error with > flow rules like: > > flow create 0 ingress pattern eth / vlan vid is 13 / ipv4 / gre / eth / > vlan vid is 15 / end actions drop / end > > 2. align all error messages. > >

Re: [dpdk-dev] [PATCH 1/2] net/mlx5: fix bit width of flow items

2018-10-26 Thread Yongseok Koh
> On Oct 25, 2018, at 1:53 AM, Shahaf Shuler wrote: > > Apply the changes from commit c744f6b1b969 ("net/mlx5: fix bit width of > item and action flags") in some places that were overlooked. > > Fixes: c744f6b1b969 ("net/mlx5: fix bit width of item and action flags") > Fixes: 23c1d42c7138 ("ne

Re: [dpdk-dev] [PATCH] kni: fix build on Linux < 3.14

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 10:56 PM, Thomas Monjalon wrote: > 26/10/2018 23:40, Thomas Monjalon: >> The atomic functions smp_load_acquire() and smp_store_release() >> were introduced in Linux 3.14. Older kernels miss the functions: >> >> kni_fifo.h:19:2: error: >> implicit declaration of function ‘smp_load

Re: [dpdk-dev] [PATCH v2 5/7] net/mlx5: e-switch VXLAN tunnel devices management

2018-10-26 Thread Yongseok Koh
On Fri, Oct 26, 2018 at 02:35:24AM -0700, Slava Ovsiienko wrote: > > -Original Message- > > From: Yongseok Koh > > Sent: Friday, October 26, 2018 9:26 > > To: Slava Ovsiienko > > Cc: Shahaf Shuler ; dev@dpdk.org > > Subject: Re: [PATCH v2 5/7] net/mlx5: e-switch VXLAN tunnel devices > > ma

Re: [dpdk-dev] [PATCH] examples/l3fwd-power: fix build with icc

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 11:00 PM, Thomas Monjalon wrote: > main.c(376): error #592: variable "lcore_id" is used before its value is set > RTE_SET_USED(lcore_id); > ^ > > The variables were voided with RTE_SET_USED without an obvious reason. > Removing these voidings should avoid the icc erro

Re: [dpdk-dev] [PATCH v2 3/7] net/mlx5: e-switch VXLAN flow translation routine

2018-10-26 Thread Yongseok Koh
On Fri, Oct 26, 2018 at 02:06:53AM -0700, Slava Ovsiienko wrote: > > -Original Message- > > From: Yongseok Koh > > Sent: Friday, October 26, 2018 7:22 > > To: Slava Ovsiienko > > Cc: Shahaf Shuler ; dev@dpdk.org > > Subject: Re: [PATCH v2 3/7] net/mlx5: e-switch VXLAN flow translation > >

[dpdk-dev] [PATCH] examples/l3fwd-power: fix build with icc

2018-10-26 Thread Thomas Monjalon
main.c(376): error #592: variable "lcore_id" is used before its value is set RTE_SET_USED(lcore_id); ^ The variables were voided with RTE_SET_USED without an obvious reason. Removing these voidings should avoid the icc error. Fixes: a137d012 ("examples/l3fwd-power: support traffic

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] test/hash: solve unit test hash compilation error

2018-10-26 Thread Thomas Monjalon
26/10/2018 23:55, Dharmik Thakkar: > > > On Oct 26, 2018, at 4:05 PM, Wang, Yipeng1 wrote: > > > >> -Original Message- > >> From: Thomas Monjalon [mailto:tho...@monjalon.net] > >> Sent: Friday, October 26, 2018 1:25 PM > >> To: Dharmik Thakkar > >> Cc: Richardson, Bruce ; De Lara Guarch

Re: [dpdk-dev] [PATCH v2 2/7] net/mlx5: e-switch VXLAN flow validation routine

2018-10-26 Thread Yongseok Koh
On Fri, Oct 26, 2018 at 01:39:38AM -0700, Slava Ovsiienko wrote: > > -Original Message- > > From: Yongseok Koh > > Sent: Friday, October 26, 2018 6:07 > > To: Slava Ovsiienko > > Cc: Shahaf Shuler ; dev@dpdk.org > > Subject: Re: [PATCH v2 2/7] net/mlx5: e-switch VXLAN flow validation > > r

Re: [dpdk-dev] [PATCH] kni: fix build on Linux < 3.14

2018-10-26 Thread Thomas Monjalon
26/10/2018 23:40, Thomas Monjalon: > The atomic functions smp_load_acquire() and smp_store_release() > were introduced in Linux 3.14. Older kernels miss the functions: > > kni_fifo.h:19:2: error: > implicit declaration of function ‘smp_load_acquire’ > kni_fifo.h:30:2: error: > implicit

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] test/hash: solve unit test hash compilation error

2018-10-26 Thread Dharmik Thakkar
> On Oct 26, 2018, at 4:05 PM, Wang, Yipeng1 wrote: > >> -Original Message- >> From: Thomas Monjalon [mailto:tho...@monjalon.net] >> Sent: Friday, October 26, 2018 1:25 PM >> To: Dharmik Thakkar >> Cc: Richardson, Bruce ; De Lara Guarch, Pablo >> ; dev@dpdk.org; >> honnappa.nagaraha..

Re: [dpdk-dev] [PATCH v3] mlx5: use pkg-config to handle SUSE libmnl

2018-10-26 Thread Yongseok Koh
On Thu, Oct 25, 2018 at 01:29:38PM -0700, Stephen Hemminger wrote: > SUSE decided to install the libmnl include file in a non-standard > place: /usr/include/libmnl/libmnl/libmnl.h > > This was probably a mistake by the SUSE package maintainer, > but hard to get fixed. Workaround the problem by pkg

Re: [dpdk-dev] [PATCH v2] mlx5: Support for rte_eth_rx_queue_count

2018-10-26 Thread Yongseok Koh
On Fri, Oct 26, 2018 at 04:48:29PM +0200, Tom Barbette wrote: > This patch adds support for the rx_queue_count API in mlx5 driver > > Changes in v2: > * Fixed styling issues > * Fix missing return > > Signed-off-by: Tom Barbette > --- Thank you for your contribution! It is good but I have a

[dpdk-dev] [PATCH v3] test/hash: solve unit test hash compilation error

2018-10-26 Thread Dharmik Thakkar
Enable print_key_info() function compilation always. Compilation error message: 'test_hash.c: In function ‘print_key_info’: test_hash.c:90:15: error: cast discards ‘const’ qualifier from pointer target type [-Werror=cast-qual] uint8_t *p = (uint8_t *)key; ^ cc1: all warnings being

[dpdk-dev] [PATCH] kni: fix build on Linux < 3.14

2018-10-26 Thread Thomas Monjalon
The atomic functions smp_load_acquire() and smp_store_release() were introduced in Linux 3.14. Older kernels miss the functions: kni_fifo.h:19:2: error: implicit declaration of function ‘smp_load_acquire’ kni_fifo.h:30:2: error: implicit declaration of function ‘smp_store_release’

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] test/hash: solve unit test hash compilation error

2018-10-26 Thread Dharmik Thakkar
> On Oct 26, 2018, at 3:24 PM, Thomas Monjalon wrote: > > +Cc Yipeng > > 18/09/2018 21:22, Dharmik Thakkar: >> Enable print_key_info() function compilation always. > > Please see my first comment: you need to show the compilation error > in this message. Otherwise, we don't know what you are

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] test/hash: solve unit test hash compilation error

2018-10-26 Thread Wang, Yipeng1
>-Original Message- >From: Thomas Monjalon [mailto:tho...@monjalon.net] >Sent: Friday, October 26, 2018 1:25 PM >To: Dharmik Thakkar >Cc: Richardson, Bruce ; De Lara Guarch, Pablo >; dev@dpdk.org; >honnappa.nagaraha...@arm.com; Wang, Yipeng1 >Subject: Re: [dpdk-stable] [PATCH v2] test/ha

Re: [dpdk-dev] [dpdk-stable] [PATCH] eal/mp: remove rte_panic and profanity

2018-10-26 Thread Thomas Monjalon
26/10/2018 16:55, Anatoly Burakov: > --- a/lib/librte_eal/common/eal_common_proc.c > +++ b/lib/librte_eal/common/eal_common_proc.c > + /* > + * set the alarm before sending message. there are two possible error > + * scenarios to consider here: > + * > + * - if the alarm set

Re: [dpdk-dev] [PATCH] malloc: fix external heap allocation in no huge mode

2018-10-26 Thread Thomas Monjalon
25/10/2018 13:06, Anatoly Burakov: > When no-huge mode is enabled, we always overwrite the socket ID to be > SOCKET_ID_ANY in rte_malloc, because there is no NUMA awareness in no-huge > mode. However, with external memory support, a socket ID may have other > meaning, and we cannot overwrite the so

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] test/hash: solve unit test hash compilation error

2018-10-26 Thread Thomas Monjalon
+Cc Yipeng 18/09/2018 21:22, Dharmik Thakkar: > Enable print_key_info() function compilation always. Please see my first comment: you need to show the compilation error in this message. Otherwise, we don't know what you are trying to fix. > Fixes: af75078fece36 ("first public release") > Cc: sta

Re: [dpdk-dev] [PATCH v4 0/6] hash: improve multiple places

2018-10-26 Thread Thomas Monjalon
> Yipeng Wang (6): > hash: fix unnecessary pause > test/hash: change multiwriter test to use jhash > test/hash: test more corner cases in unit test > test/hash: add readwrite test for ext table > test/hash: remove hash scaling unit test > test/hash: fix to add read-write test to autotes

Re: [dpdk-dev] [PATCH] buildtools: fix build for some mktemp

2018-10-26 Thread Thomas Monjalon
26/10/2018 20:31, Ferruh Yigit: > On 10/26/2018 6:59 PM, Thomas Monjalon wrote: > > 26/10/2018 20:42, Ferruh Yigit: > >> build error: > >> == Build drivers/net/tap > >> mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: > >> Invalid argument > >> .../buildtools/auto-config-h.sh: line

Re: [dpdk-dev] [PATCH] buildtools: fix build for some mktemp

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 6:59 PM, Thomas Monjalon wrote: > 26/10/2018 20:42, Ferruh Yigit: >> build error: >> == Build drivers/net/tap >> mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: >> Invalid argument >> .../buildtools/auto-config-h.sh: line 86: : No such file or directory >> .../driver

[dpdk-dev] [PATCH v7] compress/qat: enable dynamic huffman encoding

2018-10-26 Thread Fiona Trahe
Enable dynamic huffman encoding in the QAT comp PMD. Signed-off-by: Tomasz Jozwiak Signed-off-by: Fiona Trahe Acked-by: Arek Kusztal --- v7 changes: - replace %ld with %zu for printing size_t to fix compiler issue - replace phys addr with rte_iova_t v6 changes:

Re: [dpdk-dev] [PATCH] buildtools: fix build for some mktemp

2018-10-26 Thread Thomas Monjalon
26/10/2018 20:42, Ferruh Yigit: > build error: > == Build drivers/net/tap > mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: > Invalid argument > .../buildtools/auto-config-h.sh: line 86: : No such file or directory > .../drivers/net/tap/Makefile:55: recipe for target >'tap_aut

Re: [dpdk-dev] [PATCH] test: fix kni test to check kni module dependency

2018-10-26 Thread Thomas Monjalon
14/09/2018 16:40, Ferruh Yigit: > On 9/14/2018 2:57 PM, Naga Suresh Somarowthu wrote: > > Check the prerequisites to run the test > > 1.checked whether the rte_kni module is loaded, if not fail testcase > > 2.checked whether the rte_kni module is loaded with loop back mode, > > if not skip packet

[dpdk-dev] [PATCH] buildtools: fix build for some mktemp

2018-10-26 Thread Ferruh Yigit
build error: == Build drivers/net/tap mktemp: cannot create temp file /tmp/dpdk.auto-config-h.sh.XXX.c: Invalid argument .../buildtools/auto-config-h.sh: line 86: : No such file or directory .../drivers/net/tap/Makefile:55: recipe for target 'tap_autoconf.h.new' failed Above error observed on W

Re: [dpdk-dev] [PATCH v7 0/5] kni: add API to set link status on kernel interface

2018-10-26 Thread Thomas Monjalon
25/10/2018 14:30, Ferruh Yigit: > On 10/24/2018 11:26 PM, Dan Gora wrote: > >> Note that checkpatches.sh compains about patch 1/5, but this appears > >> to be a bug with check-symbol-change or something. If I move the > >> fragment of the patch modifying rte_kni_version.map to the bottom of > >> t

[dpdk-dev] [PATCH v4 5/6] test/hash: remove hash scaling unit test

2018-10-26 Thread Yipeng Wang
The hash scaling unit test is not really needed any more since the multi-writer is supported now inside the library and it is tested by multi-writer unit test. Signed-off-by: Yipeng Wang Acked-by: Bruce Richardson --- test/test/Makefile| 1 - test/test/autotest_data.py| 6 --

[dpdk-dev] [PATCH v4 4/6] test/hash: add readwrite test for ext table

2018-10-26 Thread Yipeng Wang
This commit improves the readwrite test to consider extendable table feature. Signed-off-by: Yipeng Wang Acked-by: Bruce Richardson --- test/test/test_hash_readwrite.c | 42 + 1 file changed, 34 insertions(+), 8 deletions(-) diff --git a/test/test/test_h

[dpdk-dev] [PATCH v4 3/6] test/hash: test more corner cases in unit test

2018-10-26 Thread Yipeng Wang
This commit improves the readwrite unit test to cover more corner cases and reduces the testing time by reducing the total key count. Signed-off-by: Yipeng Wang Acked-by: Bruce Richardson --- test/test/test_hash_readwrite.c | 32 ++-- 1 file changed, 26 insertions(+)

[dpdk-dev] [PATCH v4 2/6] test/hash: change multiwriter test to use jhash

2018-10-26 Thread Yipeng Wang
With sequential key, the test will cover more corner cases with jhash instead of crc hash, since jhash generates more random hash pattern on sequential key. It is useful for functional verification. Signed-off-by: Yipeng Wang --- test/test/test_hash_multiwriter.c | 3 ++- 1 file changed, 2 inser

[dpdk-dev] [PATCH v4 6/6] test/hash: fix to add read-write test to autotest

2018-10-26 Thread Yipeng Wang
Add read-write concurrency test to meson and autotest file. Fixes: 0eb3726ebcf1 ("test/hash: add test for read/write concurrency") Cc: sta...@dpdk.org Signed-off-by: Yipeng Wang --- test/test/autotest_data.py | 6 ++ test/test/meson.build | 1 + 2 files changed, 7 insertions(+) diff -

[dpdk-dev] [PATCH v4 1/6] hash: fix unnecessary pause

2018-10-26 Thread Yipeng Wang
There is a rte_pause in hash table reset function. Since the loop is not a polling loop on shared data structure, the rte_pause is not needed. Fixes: b26473ff8f4a ("hash: add reset function") Cc: sta...@dpdk.org Signed-off-by: Yipeng Wang Acked-by: Bruce Richardson --- lib/librte_hash/rte_cuck

[dpdk-dev] [PATCH v4 0/6] hash: improve multiple places

2018-10-26 Thread Yipeng Wang
This patch set depends on Honnappa's patch set: http://patchwork.dpdk.org/cover/47268/ This patch set fixes/improves a couple of places mostly on unit tests: commit 1: remove unnecessary code in hash library. commit 2: use jhash in multiwriter unit test. commit 3: cover more test case in unit tes

Re: [dpdk-dev] [PATCH v3 1/4] config: use one single config option for C11 memory model

2018-10-26 Thread Thomas Monjalon
10/10/2018 16:48, Ferruh Yigit: > On 10/8/2018 10:11 AM, phil.y...@arm.com wrote: > > Keep only single config option RTE_USE_C11_MEM_MODEL for C11 memory > > model, so all modules can leverage C11 atomic extension by enable this > > option. > > > > Signed-off-by: Phil Yang > > Reviewed-by: Honnap

Re: [dpdk-dev] [PATCH v2] net/i40e: fix Rx instability with vector mode

2018-10-26 Thread Zhang, Qi Z
> -Original Message- > From: Richardson, Bruce > Sent: Friday, October 26, 2018 6:12 AM > To: Xing, Beilei > Cc: Zhang, Qi Z ; dev@dpdk.org; sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2] net/i40e: fix Rx instability with vector > mode > > On Fri, Oct 26, 2018 at 02:33:27PM +0800

Re: [dpdk-dev] [PATCH v2] net/i40e: fix Rx instability with vector mode

2018-10-26 Thread Zhang, Qi Z
> -Original Message- > From: Ananyev, Konstantin > Sent: Friday, October 26, 2018 3:47 AM > To: Xing, Beilei ; Zhang, Qi Z > Cc: dev@dpdk.org; sta...@dpdk.org > Subject: RE: [dpdk-dev] [PATCH v2] net/i40e: fix Rx instability with vector > mode > > > Hi, > > > > > Previously, there is

Re: [dpdk-dev] [dpdk-stable] [PATCH] app/testpmd: fix qinq strip display option

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 3:34 PM, Ferruh Yigit wrote: > On 10/26/2018 1:40 PM, Jerin Jacob wrote: >> Fix a typo on DEV_RX_OFFLOAD_QINQ_STRIP selection. > > I think this is not typo but confusion caused by DEV_RX_OFFLOAD_VLAN_EXTEND > ... > >> >> Fixes: 0074d02fca21 ("app/testpmd: convert to new Rx offloads

Re: [dpdk-dev] [PATCH] doc/testpmd: fix typo in testpmd guide

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 3:32 PM, Ferruh Yigit wrote: > On 10/26/2018 11:27 AM, dev-boun...@dpdk.org wrote: >> The forwarding mode mac_swap should be macswap in testpmd guide. >> >> Signed-off-by: Yong Wang > > Reviewed-by: Ferruh Yigit Applied to dpdk-next-net/master, thanks.

Re: [dpdk-dev] [RFC] ethdev: make offload name API non-experimental

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 3:47 PM, Ferruh Yigit wrote: > On 10/19/2018 6:35 PM, Stephen Hemminger wrote: >> The offload name functions are useful, but since they are >> marked experimental they can not be used by upstream projects. >> For example, VPP duplicates the same table in its code. >> >> Signed-off-by:

Re: [dpdk-dev] [PATCH 2/4] bnx2x: remove profanity

2018-10-26 Thread Burakov, Anatoly
On 25-Jul-18 7:20 PM, Stephen Hemminger wrote: No need for profanity in comments. Signed-off-by: Stephen Hemminger --- drivers/net/bnx2x/elink.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/bnx2x/elink.c b/drivers/net/bnx2x/elink.c index 34a29373af3b..0

[dpdk-dev] [PATCH] eal/mp: remove rte_panic and profanity

2018-10-26 Thread Anatoly Burakov
EAL should not crash when setting alarm fails. Also, remove the profanity in error message. Fixes: daf9bfca717e ("ipc: remove thread for async requests") Cc: sta...@dpdk.org Cc: step...@networkplumber.org Signed-off-by: Stephen Hemminger Signed-off-by: Anatoly Burakov --- lib/librte_eal/commo

[dpdk-dev] [PATCH v2] mlx5: Support for rte_eth_rx_queue_count

2018-10-26 Thread Tom Barbette
This patch adds support for the rx_queue_count API in mlx5 driver Changes in v2: * Fixed styling issues * Fix missing return Signed-off-by: Tom Barbette --- drivers/net/mlx5/mlx5.c | 1 + drivers/net/mlx5/mlx5_rxtx.c | 60 +--- drivers/net/mlx5/

Re: [dpdk-dev] [RFC] ethdev: make offload name API non-experimental

2018-10-26 Thread Ferruh Yigit
On 10/19/2018 6:35 PM, Stephen Hemminger wrote: > The offload name functions are useful, but since they are > marked experimental they can not be used by upstream projects. > For example, VPP duplicates the same table in its code. > > Signed-off-by: Stephen Hemminger Reviewed-by: Ferruh Yigit

Re: [dpdk-dev] [PATCH] net/bonding: avoid making copy of mac address

2018-10-26 Thread Ferruh Yigit
On 10/25/2018 11:04 PM, Chas Williams wrote: > From: Chas Williams > > Calling rte_eth_macaddr_get to get a copy of the MAC address causes > a hot spot according to profiling. We can easily get the current > MAC address by just examining the bonded device. > > Signed-off-by: Chas Williams Appl

Re: [dpdk-dev] DEV_RX_OFFLOAD_VLAN_EXTEND offload

2018-10-26 Thread Jerin Jacob
-Original Message- > Date: Fri, 26 Oct 2018 14:40:42 +0100 > From: Ferruh Yigit > To: Jerin Jacob , "dev@dpdk.org" > > CC: "tho...@monjalon.net" , > "arybche...@solarflare.com" , > "olivier.m...@6wind.com" , "qi.z.zh...@intel.com" > , "beilei.x...@intel.com" , > Wenzhuo Lu , Konstant

Re: [dpdk-dev] [PATCH 0/3] net/ena: upgrade to v1.1.1

2018-10-26 Thread Ferruh Yigit
On 10/25/2018 6:59 PM, Michal Krawczyk wrote: > Hi, > > version upgrade containt two major fixes: > * Memory leak fix on ena_start(), if it was called right after stop() > * Provide application with valid hash, instead of queue ID > > The first issue was introduced in 18.08 when the additional st

Re: [dpdk-dev] [dpdk-stable] [PATCH] app/testpmd: fix qinq strip display option

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 1:40 PM, Jerin Jacob wrote: > Fix a typo on DEV_RX_OFFLOAD_QINQ_STRIP selection. I think this is not typo but confusion caused by DEV_RX_OFFLOAD_VLAN_EXTEND ... > > Fixes: 0074d02fca21 ("app/testpmd: convert to new Rx offloads API") > > Cc: shah...@mellanox.com > Cc: sta...@dpdk.o

Re: [dpdk-dev] [PATCH] doc/testpmd: fix typo in testpmd guide

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 11:27 AM, dev-boun...@dpdk.org wrote: > The forwarding mode mac_swap should be macswap in testpmd guide. > > Signed-off-by: Yong Wang Reviewed-by: Ferruh Yigit

Re: [dpdk-dev] [PATCH v4 0/2] CPU non-blocking delay

2018-10-26 Thread Ferruh Yigit
On 10/25/2018 2:27 PM, Ferruh Yigit wrote: > On 10/10/2018 3:12 PM, Ilya Maximets wrote: >> For meson build without deprecation warnings following >> patch should be applied first: >> http://patches.dpdk.org/patch/46064/ >> >> Version 4: >> * Fixed return value checking for nanosleep(). >>

Re: [dpdk-dev] [PATCH] pci/vfio: fixup rte_intr_callback_unregister() handling

2018-10-26 Thread Burakov, Anatoly
On 24-Oct-18 11:11 AM, Darek Stojaczyk wrote: This function is documented to return the number of unregistered callbacks or negative numbers on error, but pci_vfio checks for ret != 0 to detect failures. Not anymore. Fixes: c115fd000c32 ("vfio: handle hotplug request notifier") Cc: jia@intel

Re: [dpdk-dev] [PATCH] eal: fix rte_mp_request_sync() memleak on device hotplug

2018-10-26 Thread Burakov, Anatoly
On 25-Oct-18 11:46 AM, Darek Stojaczyk wrote: rte_mp_request_sync() says that the caller is responsible for freeing one of its parameters afterwards. EAL didn't do that, causing a memory leak. Fixes: 244d5130719c ("eal: enable hotplug on multi-process") Cc: qi.z.zh...@intel.com Cc: sta...@dpdk.o

Re: [dpdk-dev] [PATCH] vfio: fix read-after-free on getting container fd

2018-10-26 Thread Burakov, Anatoly
On 25-Oct-18 11:49 AM, Darek Stojaczyk wrote: We were reading some memory just after freeing it. Fixes: 83a73c5fef66 ("vfio: use generic multi-process channel") Cc: jianfeng@intel.com Cc: anatoly.bura...@intel.com Cc: sta...@dpdk.org Signed-off-by: Darek Stojaczyk --- lib/librte_eal/linu

Re: [dpdk-dev] [PATCH v2] bus/pci: use device driver name instead of handler type

2018-10-26 Thread Burakov, Anatoly
On 25-Oct-18 11:49 AM, Alejandro Lucero wrote: Invoking the right pci read/write functions is based on interrupt handler type. However, this is not configured for secondary processes precluding to use those functions. This patch fixes the issue using the driver name the device is bound to instea

Re: [dpdk-dev] [PATCH] net/*/base: allow experimental APIs in base of avf and qede

2018-10-26 Thread Ferruh Yigit
On 10/4/2018 2:37 PM, Bruce Richardson wrote: > On Thu, Oct 04, 2018 at 04:18:35PM +0300, Ilya Maximets wrote: >> This functionality was missed while adding new drivers to >> the meson build. >> >> Fixes: 1d75caf81474 ("net/avf: add meson support") >> Fixes: 30d3d0168301 ("net/qede: add in meson bu

Re: [dpdk-dev] [PATCH] power: fix traffic aware compilation errors

2018-10-26 Thread Liang, Ma
Hi Dave, Many thanks! Regards Liang On 26 Oct 13:38, David Hunt wrote: > 1. %ld to PRId64 for 32-bit builds > 2. Fix dependency on librte_timer > > Fixes: 450f0791312c ("power: add traffic pattern aware power control") > > Signed-off-by: David Hunt > Tested-by: Ferruh Yigit > --- > lib/Mak

[dpdk-dev] [PATCH] mlx5: Support for rte_eth_rx_queue_count

2018-10-26 Thread Tom Barbette
This patch adds support for the rx_queue_count API in mlx5 driver Signed-off-by: Tom Barbette --- drivers/net/mlx5/mlx5.c | 1 + drivers/net/mlx5/mlx5_rxtx.c | 62 +--- drivers/net/mlx5/mlx5_rxtx.h | 1 + 3 files changed, 54 insertions(+), 10 deleti

Re: [dpdk-dev] DEV_RX_OFFLOAD_VLAN_EXTEND offload

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 11:56 AM, Jerin Jacob wrote: > > Does anyone know the expectation of DEV_RX_OFFLOAD_VLAN_EXTEND > offload? Does not look like it is documented. > > Looks like it is very specific to Intel controllers, Based on 82599 HRM, > it is following, not sure what is the real expectation from

Re: [dpdk-dev] [PATCH v12 1/5] lib/librte_power: traffic pattern aware power control

2018-10-26 Thread Liang, Ma
Hi Thomas, Many thanks, I will check carefully for meson build and doc next time. Regards Liang On 26 Oct 01:55, Thomas Monjalon wrote: > 19/10/2018 13:07, Liang Ma: > > The proposed solution focuses on how many times empty polls are executed. > > The less the number of empty polls, means

[dpdk-dev] [PATCH v2] usertools: add DPDK quick-start python script

2018-10-26 Thread David Hunt
Add a script meant to be linked off the DPDK website from the quick start page to allow user to quickly download DPDK, configure, build DPDK and run TestPMD. It uses ``dialog`` as a user interface. This script performs the following: * Gets the latest DPDK release * Gets the necessary dependencies

Re: [dpdk-dev] [PATCH v2] kni: fix build for Linux kernel 4.19

2018-10-26 Thread Thomas Monjalon
24/10/2018 13:10, Ferruh Yigit: > The build error observed with Linux kernel 4.19 when KNI ethtool > support enabled (CONFIG_RTE_KNI_KMOD_ETHTOOL=y) > > .../build/build/kernel/linux/kni/kni_ethtool.c:193:3: >error: ‘struct ethtool_ops’ has no member named ‘get_settings’; > .get_settings = k

Re: [dpdk-dev] [PATCH v2 6/9] app/procinfo: add code for debug crypto

2018-10-26 Thread Pattan, Reshma
Hi, -Original Message- From: Varghese, Vipin + "\t -- id (%u) flags (0x%"PRIx64") socket (%d)\n" + "\t -- queue pairs (%d)\n", + rte_cryptodev_name_get(i), + dev_info.driver_name, dev_info.driver_id, +

Re: [dpdk-dev] [PATCH v2 6/9] app/procinfo: add code for debug crypto

2018-10-26 Thread Pattan, Reshma
Hi -Original Message- From: Varghese, Vipin --- + struct rte_cryptodev_info dev_info = {0}; + struct rte_cryptodev_stats stats = {0}; + Memset for initialization as mentioned in other patch. + +#define DSP_CRYPTO_FLAG(x) do { \ +printf(" - feature flags\n")

  1   2   >