[dpdk-dev] [PATCH v3 0/4] updates for hns3 PMD driver

2020-07-13 Thread Wei Hu (Xavier)
This series are updates for hns3 PMD driver. Lijun Ou (1): net/hns3: fix RSS configuration when empty RSS type Min Hu (Connor) (1): net/hns3: support keeping CRC Wei Hu (Xavier) (2): net/hns3: support copper media type net/hns3: support 200G speed rate drivers/net/hns3/hns3_cmd.h

[dpdk-dev] [PATCH v3 1/4] net/hns3: support copper media type

2020-07-13 Thread Wei Hu (Xavier)
The hns3 network engine is built-in multiple SoCs, such as kunpeng 920, kunpeng 930, etc. The PCI revision id is 0x21 in kunpeng 920, and the PCI revision id is 0x30 in kunpeng 930. Copper media type is supported for hns3 PF device in kunpeng 930 and later SoCs. The configuration operation for PHY

[dpdk-dev] [PATCH v3 4/4] net/hns3: fix RSS configuration when empty RSS type

2020-07-13 Thread Wei Hu (Xavier)
From: Lijun Ou According to the definition of RSS types of action attributes from testpmd, the driver will not disable RSS but instead requests the unspecified "best-effort" settings when upper application call rte_flow_create API function to create flow using empty RSS types. As a result, here

[dpdk-dev] [PATCH v3 3/4] net/hns3: support keeping CRC

2020-07-13 Thread Wei Hu (Xavier)
From: "Min Hu (Connor)" CRC is the end of frame, which occupies 4 bytes. Keeping CRC is a feature of MAC, which will not strip CRC field when receiving frames. The featrue can be enabled using DEV_RX_OFFLOAD_KEEP_CRC offload by upper level application. And the feature is only supported for hns3 P

[dpdk-dev] [PATCH v3 2/4] net/hns3: support 200G speed rate

2020-07-13 Thread Wei Hu (Xavier)
The 200G device has a new device id 0xA228, so adds this device id to pci table for pci driver can probe it. Similar to the network port with other speed, the hns3 PMD driver gets 200G speed information from firmware, and passes them to DPDK framework. Signed-off-by: Wei Hu (Xavier) --- drivers/

Re: [dpdk-dev] [PATCH v4 00/11] rename blacklist/whitelist to exclude/include

2020-07-13 Thread Stephen Hemminger
On Mon, 13 Jul 2020 22:39:08 -0700 Stephen Hemminger wrote: > The terms blacklist and whitelist are often seen as reminders > of the divisions in society. Instead, use more exact terms for > handling of which devices are used in DPDK. > > This is a proposed change for DPDK 20.08 to replace the n

[dpdk-dev] [PATCH v1] vhost: fix missing null dev pointer check

2020-07-13 Thread patrick . fu
From: Patrick Fu This patch adds the check of dev pointer in vhost async enqueue completion poll. If a NULL dev pointer detected, the poll function returns immediately. Coverity issue: 360839 Fixes: cd6760da10 ("vhost: introduce async enqueue for split ring") Signed-off-by: Patrick Fu --- lib

Re: [dpdk-dev] [PATCH] eal/interrupts: add function to allow interruptible epoll

2020-07-13 Thread Stephen Hemminger
On Wed, 24 Jul 2019 10:20:37 -0700 Stephen Hemminger wrote: > The existing definition of rte_epoll_wait retries if interrupted > by a signal. This behavior makes it hard to use rte_epoll_wait > for applications that want to use signals do do things like > exit polling loop and shutdown. > > Sinc

[dpdk-dev] [PATCH v4 10/11] eal: mark old macros for blacklist/whitelist as deprecated

2020-07-13 Thread Stephen Hemminger
Use of these macros in applications should cause a deprecation warning. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- lib/librte_eal/include/rte_bus.h | 9 +++-- lib/librte_eal/include/rte_dev.h | 10 +++--- lib/librte_eal/include/rte_devargs.h | 10 +++--- 3

[dpdk-dev] [PATCH v4 11/11] doc: update examples to new config options

2020-07-13 Thread Stephen Hemminger
Replace -w with -i and -b with -x to reflect new usage. Signed-off-by: Stephen Hemminger --- doc/guides/cryptodevs/qat.rst | 2 +- doc/guides/eventdevs/octeontx2.rst| 20 - doc/guides/linux_gsg/linux_drivers.rst| 4 ++-- doc/guides/mempool/oc

[dpdk-dev] [PATCH v4 06/11] eal: replace pci-whitelist/pci-blacklist options

2020-07-13 Thread Stephen Hemminger
Replace pci-whitelist with pci-include and pci-blacklist with pci-exclude. Allow the old options for now, but print a nag warning since old options are deprecated. The old -b and -w options will be removed in later release. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- lib/librt

[dpdk-dev] [PATCH v4 08/11] app/test: use new allowlist and blocklist

2020-07-13 Thread Stephen Hemminger
Test the renamed pci-blocklist and pci-allowlist arguments. Use new terms in test variable names as well. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- app/test/autotest.py| 16 ++-- app/test/autotest_runner.py | 18 ++--- app/test/test.c | 2

[dpdk-dev] [PATCH v4 09/11] doc: add note about blacklist/whitelist changes

2020-07-13 Thread Stephen Hemminger
The blacklist/whitelist changes to API will not be a breaking change for applications in this release but worth adding a note to encourage migration. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- doc/guides/rel_notes/release_20_08.rst | 6 ++ 1 file changed, 6 insertions(+)

[dpdk-dev] [PATCH v4 04/11] eal: replace usage of blacklist/whitelist in enum

2020-07-13 Thread Stephen Hemminger
The terms blacklist and whitelist are often seen as reminders of the divisions in society. Instead, use more exact terms for handling of which devices are used in DPDK. This patch renames the enum values in the EAL include files. As a backward compatible temporary migration tool, define a replacem

[dpdk-dev] [PATCH v4 05/11] drivers: replace references to blacklist

2020-07-13 Thread Stephen Hemminger
Use the new terminology blocked to describe when devices are excluded from being used. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- drivers/bus/dpaa/dpaa_bus.c| 7 +++ drivers/bus/fslmc/fslmc_bus.c | 9 - drivers/bus/fslmc/fslmc_vfio.c | 8 ---

[dpdk-dev] [PATCH v4 07/11] doc: replace references to blacklist/whitelist

2020-07-13 Thread Stephen Hemminger
The terms blacklist and whitelist are no longer used. Replace them in the documentation. Most of this was automatic replacement, but in a couple of places the language was awkward before and have tried to improve the readabilty. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- doc/

[dpdk-dev] [PATCH v4 01/11] rte_ethdev: change comment to rte_dev_eth_mac_addr_add

2020-07-13 Thread Stephen Hemminger
The comment used the term whitelist and was awkardly written. Replace it with simpler direct description of adding a new address. No code or API changes for this. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- lib/librte_ethdev/rte_ethdev.h | 3 +-- 1 file changed, 1 insertion(+),

[dpdk-dev] [PATCH v4 03/11] check_maintainers: change variable names

2020-07-13 Thread Stephen Hemminger
Change variable names in python script: replace whitelist with include_files and blacklist with exclude_files. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- devtools/check-maintainers.sh | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/devtools/check-ma

[dpdk-dev] [PATCH v4 00/11] rename blacklist/whitelist to exclude/include

2020-07-13 Thread Stephen Hemminger
The terms blacklist and whitelist are often seen as reminders of the divisions in society. Instead, use more exact terms for handling of which devices are used in DPDK. This is a proposed change for DPDK 20.08 to replace the names blacklist and whitelist in API and command lines. The first three

[dpdk-dev] [PATCH v4 02/11] mk: replace reference to blacklist/whitelist

2020-07-13 Thread Stephen Hemminger
Use BLOCKLIST and ALLOWLIST in the make variable names. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- mk/rte.sdktest.mk | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/mk/rte.sdktest.mk b/mk/rte.sdktest.mk index 803018ba3a70..6777e0c0f603 100644

Re: [dpdk-dev] [PATCH v3 00/10] rename blacklist/whitelist to block/allow

2020-07-13 Thread Stephen Hemminger
On Fri, 10 Jul 2020 17:06:11 +0200 David Marchand wrote: > About deprecating existing API/EAL flags in this release, this should > go through the standard deprecation process. > I would go with introducing new options + full compatibility + a > deprecation notice in the 20.08 release. > The actua

Re: [dpdk-dev] [PATCH] common/mlx5: fix link with ibverbs glue dlopen option

2020-07-13 Thread Matan Azrad
From: Thomas Monjalon > In case the ibverbs glue is a separate library to dlopen, the PMD library must > allocate a glue structure to be filled by dlopen. > > The glue management was in mlx5_common.c and moved to > mlx5_common_os.c, but the variable allocation was not removed from the > origina

Re: [dpdk-dev] [PATCH v3 00/10] rename blacklist/whitelist to block/allow

2020-07-13 Thread Stephen Hemminger
On Fri, 10 Jul 2020 17:06:11 +0200 David Marchand wrote: > On Sat, Jun 13, 2020 at 2:01 AM Stephen Hemminger > wrote: > > > > The terms blacklist and whitelist are often seen as reminders > > of the divisions in society. Instead, use more exact terms for > > handling of which devices are used in

[dpdk-dev] [PATCH v1] net/virtio-user: fix uninitialized variable

2020-07-13 Thread Chenbo Xia
This patch fixes an issue that uninitialized has_reply_ack is used for setting message flags. Coverity issue: 360834 Fixes: c60208dd6384 ("net/virtio-user: support reply-ack") Signed-off-by: Chenbo Xia --- drivers/net/virtio/virtio_user/vhost_user.c | 2 +- 1 file changed, 1 insertion(+), 1 del

[dpdk-dev] [PATCH v2] app/testpmd: fix incorrect output format in flow query

2020-07-13 Thread Chenxu Di
This patch fix the error line break in the output format of flow query Fixes: bdb1d61690f7 ("app/testpmd: support RSS config in flow query") Signed-off-by: Chenxu Di --- app/test-pmd/config.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/app/test-pmd/config.c b/app/

Re: [dpdk-dev] [PATCH v3 0/4] introduce global debug flag

2020-07-13 Thread Stephen Hemminger
On Tue, 14 Jul 2020 00:39:43 +0200 Lukasz Wojciechowski wrote: > W dniu 11.07.2020 o 17:11, Thomas Monjalon pisze: > > 09/07/2020 15:48, Lukasz Wojciechowski: > >> This set of patches introduces a global rte_debug flag for dpdk. > >> This will allow easy switch to debug build configuration usin

Re: [dpdk-dev] [PATCH 0/9] python2 deprecation notice

2020-07-13 Thread Stephen Hemminger
On Mon, 13 Jul 2020 10:21:49 +0100 Bruce Richardson wrote: > On Fri, Jul 10, 2020 at 03:33:58PM +0200, Robin Jarry wrote: > > Hi Louise, > > > > 2020-07-10, Louise Kilheeney: > > > This patchset adds deprecation notices to python scripts, > > > warning of the removal of python2 support from th

Re: [dpdk-dev] [PATCH v3 0/4] introduce global debug flag

2020-07-13 Thread Lukasz Wojciechowski
W dniu 13.07.2020 o 11:04, Bruce Richardson pisze: > On Sat, Jul 11, 2020 at 05:11:52PM +0200, Thomas Monjalon wrote: >> 09/07/2020 15:48, Lukasz Wojciechowski: >>> This set of patches introduces a global rte_debug flag for dpdk. >>> This will allow easy switch to debug build configuration using

Re: [dpdk-dev] [PATCH v3 0/4] introduce global debug flag

2020-07-13 Thread Lukasz Wojciechowski
W dniu 11.07.2020 o 17:11, Thomas Monjalon pisze: > 09/07/2020 15:48, Lukasz Wojciechowski: >> This set of patches introduces a global rte_debug flag for dpdk. >> This will allow easy switch to debug build configuration using a single >> flag. In the debug mode a RTE_DEBUG macro is defined to 1 >

Re: [dpdk-dev] [PATCH v6 0/8] fib: implement AVX512 vector lookup

2020-07-13 Thread Stephen Hemminger
On Mon, 13 Jul 2020 12:11:19 +0100 Vladimir Medvedkin wrote: > This patch series implements vectorized lookup using AVX512 for > ipv4 dir24_8 and ipv6 trie algorithms. > Also introduced rte_fib_set_lookup_fn() to change lookup function type. > Added option to select lookup function type in testfi

Re: [dpdk-dev] [PATCH v4 2/2] mempool/ring: add support for new ring sync modes

2020-07-13 Thread Thomas Monjalon
13/07/2020 17:50, Konstantin Ananyev: > Two new sync modes were introduced into rte_ring: > relaxed tail sync (RTS) and head/tail sync (HTS). > This change provides user with ability to select these > modes for ring based mempool via mempool ops API. > > Signed-off-by: Konstantin Ananyev > Acked-

[dpdk-dev] [dpdk-dev v5 3/4] test/crypto: add unit-test for cryptodev direct APIs

2020-07-13 Thread Fan Zhang
This patch adds the QAT test to use cryptodev symmetric crypto direct APIs. Signed-off-by: Fan Zhang --- app/test/test_cryptodev.c | 367 -- app/test/test_cryptodev.h | 6 + app/test/test_cryptodev_blockcipher.c | 50 ++-- 3 files changed, 386 i

[dpdk-dev] [dpdk-dev v5 1/4] cryptodev: add data-path APIs

2020-07-13 Thread Fan Zhang
This patch adds data-path APIs for enqueue and dequeue operations to cryptodev. The APIs support flexible user-define enqueue and dequeue behaviors and operation modes. Signed-off-by: Fan Zhang Signed-off-by: Piotr Bronowski --- lib/librte_cryptodev/rte_crypto_sym.h | 27 +- lib/librte

[dpdk-dev] [dpdk-dev v5 2/4] crypto/qat: add support to direct data-path APIs

2020-07-13 Thread Fan Zhang
This patch add symmetric crypto data-path APIs support to QAT-SYM PMD. Signed-off-by: Fan Zhang --- drivers/common/qat/Makefile| 1 + drivers/common/qat/qat_qp.h| 1 + drivers/crypto/qat/meson.build | 1 + drivers/crypto/qat/qat_sym.h | 3 + drivers/crypto/qat/q

[dpdk-dev] [dpdk-dev v5 4/4] doc: add cryptodev direct APIs guide

2020-07-13 Thread Fan Zhang
This patch updates programmer's guide to demonstrate the usage and limitations of cryptodev symmetric crypto data-path APIs. Signed-off-by: Fan Zhang --- doc/guides/prog_guide/cryptodev_lib.rst | 53 + doc/guides/rel_notes/release_20_08.rst | 8 2 files changed, 61

[dpdk-dev] [dpdk-dev v5 0/4] cryptodev: add symmetric crypto data-path APIs

2020-07-13 Thread Fan Zhang
This patch adds symmetric crypto data-path APIs for Cryptodev. Direct symmetric crypto data-path APIs are a set of APIs that provide more HW friendly enqueue/dequeue data-path functions as an alternative approach to ``rte_cryptodev_enqueue_burst`` and ``rte_cryptodev_dequeue_burst``. The APIs are d

Re: [dpdk-dev] [PATCH v2] mempool/ring: add support for new ring sync modes

2020-07-13 Thread Ananyev, Konstantin
> On Mon, Jul 13, 2020 at 02:46:35PM +, Ananyev, Konstantin wrote: > > Hi Olivier, > > > > > Hi Konstantin, > > > > > > On Fri, Jul 10, 2020 at 03:20:12PM +, Ananyev, Konstantin wrote: > > > > > > > > > > > > > > > > > > Hi Olivier, > > > > > > > > > > > Hi Konstantin, > > > > > > > > > >

Re: [dpdk-dev] [PATCH v4 2/2] doc: announce deprecation of refcnt atomic member

2020-07-13 Thread Phil Yang
Ruifeng Wang writes: > > Subject: [PATCH v4 2/2] doc: announce deprecation of refcnt atomic > member > > > > refcnt_atomic member in structures rte_mbuf and > > rte_mbuf_ext_shared_info will be deprecated in 20.11 release. > The plan should be deprecate it in 20.08 release and remove it in 20.1

[dpdk-dev] [PATCH v4 2/2] mempool/ring: add support for new ring sync modes

2020-07-13 Thread Konstantin Ananyev
Two new sync modes were introduced into rte_ring: relaxed tail sync (RTS) and head/tail sync (HTS). This change provides user with ability to select these modes for ring based mempool via mempool ops API. Signed-off-by: Konstantin Ananyev Acked-by: Gage Eads --- doc/guides/mempool/ring.rst

[dpdk-dev] [PATCH v4 1/2] doc: add ring based mempool guide

2020-07-13 Thread Konstantin Ananyev
Add documentation for rte_ring mempool driver. Signed-off-by: Konstantin Ananyev --- doc/guides/mempool/index.rst | 1 + doc/guides/mempool/ring.rst | 34 ++ 2 files changed, 35 insertions(+) create mode 100644 doc/guides/mempool/ring.rst diff --git a/doc/guid

[dpdk-dev] [PATCH v4 0/2] mempool/ring: add support for new ring sync modes

2020-07-13 Thread Konstantin Ananyev
v4: add new section into mempool derivers guide (Thomas Monjalon comments) v3: silently ignore F_SP_PUT/F_SC_GET (Olivier Matz comments) v2: update Release Notes (Gage Eads comments) Konstantin Ananyev (2): doc: add ring based mempool guide mempool/ring: add support for new ring sync m

Re: [dpdk-dev] [PATCH] lib: fix build for missing RCU library dependency

2020-07-13 Thread Ruifeng Wang
> -Original Message- > From: Ferruh Yigit > Sent: Monday, July 13, 2020 9:51 PM > To: Honnappa Nagarahalli ; Ruifeng Wang > ; Vladimir Medvedkin > ; Ray Kinsella > Cc: dev@dpdk.org; Ferruh Yigit > Subject: [PATCH] lib: fix build for missing RCU library dependency > > 'librte_rcu' is n

[dpdk-dev] [PATCH] common/mlx5: fix link with ibverbs glue dlopen option

2020-07-13 Thread Thomas Monjalon
In case the ibverbs glue is a separate library to dlopen, the PMD library must allocate a glue structure to be filled by dlopen. The glue management was in mlx5_common.c and moved to mlx5_common_os.c, but the variable allocation was not removed from the original file. The consequence was a link fa

Re: [dpdk-dev] [PATCH v2 1/2] power: fix power management env detection

2020-07-13 Thread Bruce Richardson
On Mon, Jul 13, 2020 at 03:54:27PM +0100, Anatoly Burakov wrote: > Anything coming from sysfs has a newline at the end. Cut it off before > comparing the strings. > > Fixes: 20ab67608a39 ("power: add environment capability probing") > > Signed-off-by: Anatoly Burakov > Acked-by: David Hunt > --

Re: [dpdk-dev] [PATCH v2 1/1] common/qat: support GEN2 QAT device 200xx

2020-07-13 Thread Trahe, Fiona
Hi Adam, > -Original Message- > From: Dybkowski, AdamX > Sent: Monday, July 13, 2020 12:10 PM > To: Trahe, Fiona ; akhil.go...@nxp.com; dev@dpdk.org > Cc: Dybkowski, AdamX > Subject: [PATCH v2 1/1] common/qat: support GEN2 QAT device 200xx > > This adds pci detection and documentation

Re: [dpdk-dev] [PATCH] test/ring_perf: fix error statistics in bulk enq/dequeue

2020-07-13 Thread Olivier Matz
On Mon, Jun 08, 2020 at 01:49:38PM +0800, Feifei Wang wrote: > In size 32 bulk ring enq/dequeue performance test, the "Total count" > statistics is incorrect. For example, running the test on lcore 25 and > lcore 26, the output is as follows: > > The test command: > $sudo ./arm64-armv8a-linuxapp-g

[dpdk-dev] [PATCH v2 7/7] net/qede: add VF FLR support

2020-07-13 Thread Manish Chopra
This patch adds required bit to handle VF FLR indication from Management FW (MFW) of the device With that VFs were able to load in VM (VF attached as PCI passthrough to the guest VM) followed by FLR successfully Updated the docs/guides with the feature support Signed-off-by: Manish Chopra Signe

[dpdk-dev] [PATCH v2 6/7] net/qede: initialize VF MAC and link

2020-07-13 Thread Manish Chopra
This patch configures VFs with random mac if no MAC is provided by the PF/bulletin. This also adds required bulletin APIs by PF-PMD driver to communicate LINK properties/changes to the VFs through bulletin update mechanism. With these changes, VF-PMD instance is able to run fastpath over PF-PMD dr

[dpdk-dev] [PATCH v2 3/7] net/qede: define PCI config space specific osals

2020-07-13 Thread Manish Chopra
This patch defines various PCI config space access APIs in order to read and find IOV specific PCI capabilities. With these definitions implemented, it enables the base driver to do SR-IOV specific initialization and HW specific configuration required from PF-PMD driver instance. Signed-off-by: M

[dpdk-dev] [PATCH v2 5/7] net/qede: add infrastructure support for VF load

2020-07-13 Thread Manish Chopra
This patch adds necessary infrastructure support (required to handle messages from VF and sending ramrod on behalf of VF's configuration request from alarm handler context) to start/load the VF-PMD driver instance on top of PF-PMD driver instance. Signed-off-by: Manish Chopra Signed-off-by: Igor

[dpdk-dev] [PATCH v2 4/7] net/qede: configure VFs on hardware

2020-07-13 Thread Manish Chopra
Based on number of VFs enabled at PCI, PF-PMD driver instance enables/configures those VFs from hardware perspective, such that in later patches they could get required HW access to communicate with PFs for slowpath configuration and run the fastpath themsleves. This patch also add two new qede IO

[dpdk-dev] [PATCH v2 1/7] lib/librte_pci: add rte_pci_regs.h

2020-07-13 Thread Manish Chopra
This is merely copy of latest linux/pci_regs.h in order to avoid dependency of dpdk on user headers. Signed-off-by: Manish Chopra Signed-off-by: Igor Russkikh --- drivers/bus/pci/linux/pci_uio.c |2 +- drivers/bus/pci/linux/pci_vfio.c|2 +- drivers/net/bnx2x/bnx2x.h |

[dpdk-dev] [PATCH v2 2/7] drivers: add generic API to find PCI extended cap

2020-07-13 Thread Manish Chopra
By adding generic API, this patch removes individual functions/defines implemented by drivers to find PCI extended capability. Signed-off-by: Manish Chopra Signed-off-by: Igor Russkikh --- drivers/bus/pci/pci_common.c | 41 + drivers/bus/pci/rte_bus_pci.h

[dpdk-dev] [PATCH v2 0/7] qede: SR-IOV PF driver support

2020-07-13 Thread Manish Chopra
Hi, This series adds SR-IOV PF pmd driver support to have VF pmd driver work over PF pmd driver instances in order to run the adapter completely under DPDK environment for one of the use cases like ovs-dpdk. This is very initial bring-up with following testing covered - * Enable/Disable SR-IOV V

Re: [dpdk-dev] [RFC] - Offloading tunnel ports

2020-07-13 Thread William Tu
On Sun, Jul 12, 2020 at 9:52 PM Oz Shlomo wrote: > > Hi William, > > On 7/12/2020 7:34 PM, William Tu wrote: > > Hi Oz, > > snip > >> > >> Openvswitch, for example, uses two flows: > >> 1. classification flow - setting the virtual port representing the tunnel > >> type > >> For example: match on

[dpdk-dev] [v3 1/3] examples/vm_power_manager: Make branch ratio threshold per core

2020-07-13 Thread Rory Sexton
This modification allows for the branch ratio threshold to be set per core rather than system wide. This gives greater flexibility to the branch ration monitoring allowing it to manage different workloads with different characteristics on the same system. Signed-off-by: Rory Sexton Reviewed-by: D

[dpdk-dev] [v3 3/3] doc: update vm_power_manager cmdline options in doc

2020-07-13 Thread Rory Sexton
Updating the vm_power_manager docs to reflect changes made in previous patches in set. Signed-off-by: Rory Sexton Reviewed-by: David Hunt --- doc/guides/sample_app_ug/vm_power_management.rst | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/doc/guides/sample_a

[dpdk-dev] [v3 2/3] examples/vm_power_manager: Allowing power managing of idle cores

2020-07-13 Thread Rory Sexton
This change is required to allow the branch ratio algorithm to power manage cores with no workload running on them. This is useful both when idle cores don't use C-states and for a number of hyperthreading scenarios. Signed-off-by: Rory Sexton Reviewed-by: David Hunt --- examples/vm_power_manag

Re: [dpdk-dev] [PATCH v2] mempool/ring: add support for new ring sync modes

2020-07-13 Thread Olivier Matz
On Mon, Jul 13, 2020 at 02:46:35PM +, Ananyev, Konstantin wrote: > Hi Olivier, > > > Hi Konstantin, > > > > On Fri, Jul 10, 2020 at 03:20:12PM +, Ananyev, Konstantin wrote: > > > > > > > > > > > > > > Hi Olivier, > > > > > > > > > Hi Konstantin, > > > > > > > > > > On Thu, Jul 09, 2020 at

[dpdk-dev] [PATCH v2 2/2] l3fwd-power: fix updating performance lcore parameters

2020-07-13 Thread Anatoly Burakov
When perf-config option is specified, we are calling into the power library even though it may not necessarily be enabled. It is questionable whether perf-config option is even applicable to non-power library modes, but for now, fix it just by avoiding calling into the power library if it wasn't in

[dpdk-dev] [PATCH v2 1/2] power: fix power management env detection

2020-07-13 Thread Anatoly Burakov
Anything coming from sysfs has a newline at the end. Cut it off before comparing the strings. Fixes: 20ab67608a39 ("power: add environment capability probing") Signed-off-by: Anatoly Burakov Acked-by: David Hunt --- lib/librte_power/power_common.c | 8 1 file changed, 8 insertions(+)

Re: [dpdk-dev] [PATCH] mempool: fix memory allocation in memzones during retry.

2020-07-13 Thread Burakov, Anatoly
On 13-Jul-20 12:29 PM, Andrew Rybchenko wrote: On 7/13/20 2:17 PM, Burakov, Anatoly wrote: On 13-Jul-20 4:40 AM, Zhike Wang wrote: If allocation is successful on the first attempt, typically there is no problem since we allocated everything required and we'll terminate the loop (if memory chunk

Re: [dpdk-dev] [PATCH v2] mempool/ring: add support for new ring sync modes

2020-07-13 Thread Ananyev, Konstantin
Hi Olivier, > Hi Konstantin, > > On Fri, Jul 10, 2020 at 03:20:12PM +, Ananyev, Konstantin wrote: > > > > > > > > > > Hi Olivier, > > > > > > > Hi Konstantin, > > > > > > > > On Thu, Jul 09, 2020 at 05:55:30PM +, Ananyev, Konstantin wrote: > > > > > Hi Olivier, > > > > > > > > > > > Hi Ko

[dpdk-dev] [PATCH] doc: update pdump documentation

2020-07-13 Thread Reshma Pattan
Update the pdump library programmers guide and Howto doc with the use of multi process channel replacing socket based communication. Signed-off-by: Reshma Pattan --- doc/guides/howto/packet_capture_framework.rst | 16 - doc/guides/prog_guide/pdump_lib.rst | 49 +++--

Re: [dpdk-dev] [PATCH v3 1/3] net: fix s_addr redefinition in Windows

2020-07-13 Thread Olivier Matz
Hi, On Wed, Jul 08, 2020 at 11:25:23AM +0300, Fady Bader wrote: > s_addr in Windows is defined in windows.h so its Is it windows.h, or is it winsock.h or winsock2.h instead? > undefined in order to be defined as part of rte_ether_hdr. > > Signed-off-by: Fady Bader > --- > lib/librte_net/rte_e

Re: [dpdk-dev] [PATCH v3 2/3] net: replace htons with rte_cpu_to_be_16

2020-07-13 Thread Olivier Matz
On Wed, Jul 08, 2020 at 11:25:24AM +0300, Fady Bader wrote: > htons wasn't defined in Windows for the minGW compiler. > htons was replaced with rte_cpu_to_be_16 in order to compile > under Windows. > > Signed-off-by: Fady Bader > --- > lib/librte_net/rte_arp.c | 11 +++ > 1 file changed,

[dpdk-dev] DPDK vDPA application on Mellanox ConnectX 6Dx

2020-07-13 Thread William Tu
Hi, We are setting up a testbed using mellanox connectX 6Dx MCX621102AN-ADAT (2x25G) and are playing with vDPA features. We setup two machines (traffic gen and KVM) and connect them back-to-back with two connectX 6Dx. At this moment we don't know which component might be misconfigured (qemu, vdp

[dpdk-dev] [PATCH v4 2/2] ethdev: add common stats for telemetry

2020-07-13 Thread Ciara Power
The ethdev library now registers a telemetry command for common ethdev statistics. An example usage is shown below: Connecting to /var/run/dpdk/rte/dpdk_telemetry.v2 {"version": "DPDK 20.08.0-rc1", "pid": 14119, "max_output_len": 16384} --> /ethdev/stats,0 {"/ethdev/stats": {"ipackets": 0, "opack

[dpdk-dev] [PATCH v4 0/2] add basic ethdev stats with data object recursion

2020-07-13 Thread Ciara Power
v4: Added missing param description to Doxygen comment. v3: - Modified commit logs. - Changed function names to reference container. - Modified Doxygen comments to reference container. v2: - Added support for arrays to have container values. - Added support for int and string arrays withi

[dpdk-dev] [PATCH v4 1/2] telemetry: support array values in data objects

2020-07-13 Thread Ciara Power
Arrays of type uint64_t/int/string can now be included within an array or dict. One level of embedded containers is supported. This is necessary to allow for instances such as the ethdev queue stats to be reported as a list of uint64_t values, rather than having multiple dict entries with one uint6

[dpdk-dev] [PATCH v2] net: do fragmented headers check in non-debug build as well

2020-07-13 Thread Andrew Rybchenko
Pseudo-header checksum calculation requires contiguous headers. There is no any formal requirements on data location and mbuf structure which could be used by the application. Since commit dfc6b2fd8da3 ("mbuf: remove Intel offload checks from generic API") fragmented headers checks are done insi

Re: [dpdk-dev] [PATCH] net: do fragmented headers check in non-debug build as well

2020-07-13 Thread Andrew Rybchenko
Hi Olivier, On 7/13/20 4:35 PM, Olivier Matz wrote: > Hi Andrew, > > On Wed, May 27, 2020 at 03:32:56PM +0100, Andrew Rybchenko wrote: >> Pseudo-header checksum calculation requires contiguous headers. >> There is no any formal requirements on data location and mbuf >> structure which could be use

Re: [dpdk-dev] [PATCH v3] mempool/ring: add support for new ring sync modes

2020-07-13 Thread Thomas Monjalon
13/07/2020 14:58, Ananyev, Konstantin: > > 10/07/2020 18:21, Konstantin Ananyev: > > > Two new sync modes were introduced into rte_ring: > > > relaxed tail sync (RTS) and head/tail sync (HTS). > > > This change provides user with ability to select these > > > modes for ring based mempool via mempoo

Re: [dpdk-dev] [DPDK] net/ice/base: adjust code order and cleanup coding style

2020-07-13 Thread Ferruh Yigit
On 7/13/2020 12:05 PM, Junfeng Guo wrote: > Previous logic in removing RSS for GTPU was added in incorrect lines. > The adjustment in this patch will correct the code order so that RSS > for GTPU can be removed successfully. > > We also delete the redundant print statement and cleanup the coding >

[dpdk-dev] [PATCH] lib: fix build for missing RCU library dependency

2020-07-13 Thread Ferruh Yigit
'librte_rcu' is now dependency to 'librte_lpm' library, this dependency should be reflected to build system. Fixes: 8a9f8564e9f9 ("lpm: implement RCU rule reclamation") Signed-off-by: Ferruh Yigit --- Cc: ruifeng.w...@arm.com --- lib/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

[dpdk-dev] [PATCH 07/10] net/bnxt: add support for identifier search and ref count

2020-07-13 Thread Somnath Kotur
From: Kishore Padmanabha Add support to search for identifiers and increase the reference count for identifiers that are already allocated. Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/ulp_mapper.c | 118 ++

[dpdk-dev] [PATCH 09/10] net/bnxt: increase the number of egress flow entries

2020-07-13 Thread Somnath Kotur
From: Kishore Padmanabha Increase the number of egress flow entries. Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Reviewed-by: Ajit Kumar Khaparde Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[dpdk-dev] [PATCH 06/10] net/bnxt: ignore ipv4 TOS mask

2020-07-13 Thread Somnath Kotur
From: Kishore Padmanabha This is a work around for the OVS setting offload rules that are passing ipv4 tos mask as wild card and currently we do not support. Signed-off-by: Kishore Padmanabha Reviewed-by: Somnath Kotur Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/ulp_rte_parser.c

[dpdk-dev] [PATCH 10/10] net/bnxt: add support for decrement TTL action

2020-07-13 Thread Somnath Kotur
From: Kishore Padmanabha Added support for decrement TTL action. Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/ulp_rte_parser.c | 30 +++ drivers/net/bnxt/tf_ulp/ulp_rte_parser.h | 5 +

[dpdk-dev] [PATCH 03/10] net/bnxt: check index range in bulk get

2020-07-13 Thread Somnath Kotur
From: Jay Ding In tf_tbl_bulk_get, check if the indexes are in the range of reserved tbl id instead of checking the allocation of each id. Signed-off-by: Jay Ding Reviewed-by: Somnath Kotur Reviewed-by: Randy Schacher Reviewed-by: Ajit Kumar Khaparde Signed-off-by: Somnath Kotur --- driver

[dpdk-dev] [PATCH 01/10] net/bnxt: add option to delay EEM sysmem mapping

2020-07-13 Thread Somnath Kotur
From: Peter Spreadborough - The mapping of kernel pages for EEM sysmem operation takes a significant amount of time. This change give the build option to delay the sysmem mapping until the first write to EEM Signed-off-by: Peter Spreadborough Reviewed-by: Randy Schacher Signed-off-by: Somn

[dpdk-dev] [PATCH 08/10] net/bnxt: consider VLAN fields for template match criteria

2020-07-13 Thread Somnath Kotur
From: Kishore Padmanabha The vlan mask fields were not setting the field bitmap causing the template match process to ignore vlan fields. This change fixes this bug. Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/ulp_map

[dpdk-dev] [PATCH 05/10] net/bnxt: add support to extract data from the ulp blob

2020-07-13 Thread Somnath Kotur
From: Kishore Padmanabha Extended the ulp blob to extract data from the blob for a given offset and length. The support is added only for little endian format. Signed-off-by: Kishore Padmanabha Reviewed-by: Randy Schacher Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/ulp_utils.c |

[dpdk-dev] [PATCH 04/10] net/bnxt: add changes to support 2 table scopes

2020-07-13 Thread Somnath Kotur
From: Farah Smith Need to remap the table scope ids allocated from HCAPI RM from high to low value because for legacy devices a table scope is a set of base addresses. The PCIe addresses must map to a PCIe PF which exists in the hardware. Signed-off-by: Farah Smith Reviewed-by: Somnath Kotur

[dpdk-dev] [PATCH v3 00/10] bnxt patches

2020-07-13 Thread Somnath Kotur
Minor enhancments added to the TF-ULP/Core layers Farah Smith (1): net/bnxt: add changes to support 2 table scopes Jay Ding (2): net/bnxt: implement TF Identifier search net/bnxt: check index range in bulk get Kishore Padmanabha (6): net/bnxt: add support to extract data from the ulp blo

[dpdk-dev] [PATCH 02/10] net/bnxt: implement TF Identifier search

2020-07-13 Thread Somnath Kotur
From: Jay Ding Implement shadow copy DB to hold reference count for each ID in each identifier type. Implement identifier search functionality. Signed-off-by: Jay Ding Reviewed-by: Randy Schacher Signed-off-by: Somnath Kotur --- drivers/net/bnxt/meson.build| 3 +- drive

Re: [dpdk-dev] [PATCH] net: do fragmented headers check in non-debug build as well

2020-07-13 Thread Olivier Matz
Hi Andrew, On Wed, May 27, 2020 at 03:32:56PM +0100, Andrew Rybchenko wrote: > Pseudo-header checksum calculation requires contiguous headers. > There is no any formal requirements on data location and mbuf > structure which could be used by the application. > > Make corresponding check to be don

Re: [dpdk-dev] [PATCH v2] mempool/ring: add support for new ring sync modes

2020-07-13 Thread Olivier Matz
Hi Konstantin, On Fri, Jul 10, 2020 at 03:20:12PM +, Ananyev, Konstantin wrote: > > > > > > Hi Olivier, > > > > > Hi Konstantin, > > > > > > On Thu, Jul 09, 2020 at 05:55:30PM +, Ananyev, Konstantin wrote: > > > > Hi Olivier, > > > > > > > > > Hi Konstantin, > > > > > > > > > > On Mon,

Re: [dpdk-dev] [PATCH 20.11] raw/ioat: added a flag to control copying handle parameters

2020-07-13 Thread Bruce Richardson
On Mon, Jul 13, 2020 at 07:15:19AM +, Cheng Jiang wrote: > Added a flag which controls whether rte_ioat_enqueue_copy > and rte_ioat_completed_copies function should process > handle parameters to improve the performance when handle > parameters are not necessary to use. This is targeting > 20.1

[dpdk-dev] [PATCH V3 1/2] net/i40e: i40e FDIR update rate optimization data structures

2020-07-13 Thread chenmin . sun
From: Chenmin Sun This patch introduces the new data structures for i40e fdir update rate optimization. Signed-off-by: Chenmin Sun --- drivers/net/i40e/i40e_ethdev.h | 72 -- 1 file changed, 61 insertions(+), 11 deletions(-) diff --git a/drivers/net/i40e/i40e_e

Re: [dpdk-dev] [PATCH] examples/packet_ordering: Use rte_exit in case of invalid EAL or application arguments

2020-07-13 Thread Sarosh Arif
Did you get the time to review it? On Tue, Jun 23, 2020 at 2:22 PM Pattan, Reshma wrote: > > > > > *From:* Sarosh Arif > *Sent:* Tuesday, June 23, 2020 10:19 AM > *To:* dev@dpdk.org; Pattan, Reshma > *Subject:* Re: [PATCH] examples/packet_ordering: Use rte_exit in case of > invalid EAL or appl

[dpdk-dev] [PATCH V3 2/2] net/i40e: i40e FDIR update rate optimization

2020-07-13 Thread chenmin . sun
From: Chenmin Sun This patch optimized the fdir update rate for i40e PF, by tracking whether the fdir rule being inserted into the guaranteed space or shared space. For the flows that are inserted to the guaranteed space, we assume that the insertion will always succeed as the hardware only repor

[dpdk-dev] [PATCH V3 0/2] i40e FDIR update rate optimization

2020-07-13 Thread chenmin . sun
From: Chenmin Sun i40e FDIR update rate optimization [PATCH v3 1/2] net/i40e: introduce new data structures [PATCH v3 2/2] net/i40e: i40e FDIR update rate optimization v3: * Split the patch into two * Renamed some data structures and added some descriptions --- v2: * Refine commit message and c

Re: [dpdk-dev] [PATCH 2/2] ethdev: tunnel offload model

2020-07-13 Thread Gregory Etelson
> -Original Message- > From: Thomas Monjalon > Sent: Monday, July 13, 2020 11:21 > To: Gregory Etelson > Cc: Kinsella, Ray ; dev@dpdk.org; Matan Azrad > ; Raslan Darawsheh ; Ori > Kam ; John McNamara ; > Marko Kovacevic ; Neil Horman > ; Ferruh Yigit ; Andrew > Rybchenko ; Ajit Khaparde >

[dpdk-dev] [PATCH v3 1/2] app/proc-info: enhance mempool to print ops name

2020-07-13 Thread Hemant Agrawal
Enhance the mempool details to also print the ops index and name Signed-off-by: Hemant Agrawal --- These patches are built over Stephen's series: https://patches.dpdk.org/cover/69876/ But they can be applied independently as well. app/proc-info/main.c | 5 + 1 file changed, 5 insertions(+)

[dpdk-dev] [PATCH v3 2/2] app/proc-info: add crypto security context info

2020-07-13 Thread Hemant Agrawal
This patch adds the crypto based security context info. Also improve the flag printing to SECURITY OFFLOAD from INLINE. Signed-off-by: Hemant Agrawal --- app/proc-info/main.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/app/proc-info/main.c b/app/pro

[dpdk-dev] [PATCH] drivers/common/octeontx2: use generic memory management

2020-07-13 Thread Sarosh Arif
Use generic memory management calls instead of Unix memory management calls for mempool. Signed-off-by: Sarosh Arif --- drivers/common/octeontx2/otx2_dev.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/common/octeontx2/otx2_dev.c b/drivers/common/octeontx2/

Re: [dpdk-dev] [PATCH v4 09/27] examples/ip_pipeline: replace references to master_lcore

2020-07-13 Thread Burakov, Anatoly
On 01-Jul-20 9:23 PM, Stephen Hemminger wrote: Use initial lcore instead of primary lcore Signed-off-by: Stephen Hemminger --- Acked-by: Anatoly Burakov -- Thanks, Anatoly

Re: [dpdk-dev] [PATCH v4 08/27] examples/ethtool-app: replace references to slave with worker

2020-07-13 Thread Burakov, Anatoly
On 01-Jul-20 9:23 PM, Stephen Hemminger wrote: Conforms to new API naming conventions. Signed-off-by: Stephen Hemminger --- Acked-by: Anatoly Burakov -- Thanks, Anatoly

Re: [dpdk-dev] [PATCH v6 3/4] qat: use WC store to update queue tail registers

2020-07-13 Thread Bruce Richardson
On Mon, Jul 13, 2020 at 01:52:24PM +0100, Trahe, Fiona wrote: > > > > -Original Message- > > From: Bruce Richardson > > Sent: Monday, July 13, 2020 1:45 PM > > To: Nicolau, Radu > > Cc: dev@dpdk.org; Xing, Beilei ; Guo, Jia > > ; Ananyev, Konstantin > > ; jerinjac...@gmail.com; > > da

  1   2   3   >