Re: [dpdk-dev] [PATCH] build: fix version map file references in documentation

2020-10-22 Thread David Marchand
On Thu, Oct 22, 2020 at 9:47 AM David Marchand wrote: > > Fixes: 63b3907833d8 ("build: remove library name from version map file name") > > Signed-off-by: David Marchand Acked-by: Ray Kinsella Acked-by: Bruce Richardson Applied, thanks. Ray, I'll let you update the documentation with better e

Re: [dpdk-dev] [EXT] Re: [PATCH 2/2] vfio: fix partial DMA unmapping for VFIO type1

2020-10-22 Thread Nithin Dabilpuram
Ping. On Mon, Oct 19, 2020 at 03:13:15PM +0530, Nithin Dabilpuram wrote: > On Sat, Oct 17, 2020 at 05:14:55PM +0100, Burakov, Anatoly wrote: > > On 16-Oct-20 8:10 AM, Nithin Dabilpuram wrote: > > > On Thu, Oct 15, 2020 at 04:10:31PM +0100, Burakov, Anatoly wrote: > > > > On 15-Oct-20 12:57 PM, Nit

[dpdk-dev] [PATCH 2/3] net/sfc: use compat for 128-bit unsigned integer

2020-10-22 Thread Andrew Rybchenko
Prepare to support ARMv8. Signed-off-by: Andrew Rybchenko Reviewed-by: Andy Moreton --- drivers/net/sfc/sfc_ef10.h| 4 ++-- drivers/net/sfc/sfc_ef10_tx.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/sfc/sfc_ef10.h b/drivers/net/sfc/sfc_ef10.h index 07c3

[dpdk-dev] [PATCH 3/3] net/sfc: support aarch64 architecture

2020-10-22 Thread Andrew Rybchenko
Enable the PMD build on aarch64. Signed-off-by: Andrew Rybchenko Reviewed-by: Andy Moreton --- config/arm/meson.build | 1 - doc/guides/nics/features/sfc_efx.ini | 1 + doc/guides/rel_notes/release_20_11.rst | 1 + drivers/common/sfc_efx/efsys.h | 17 +++

[dpdk-dev] [PATCH 1/3] common/sfc_efx: introduce 128-bit unsigned integer compat

2020-10-22 Thread Andrew Rybchenko
Intel SSE has __m128i, but ARMv8 has __uint128_t. So, add compat efsys_uint128_t to be used in driver source and have either __u128i or __uint128_t behind. Signed-off-by: Andrew Rybchenko Reviewed-by: Andy Moreton --- drivers/common/sfc_efx/base/efx_types.h | 8 drivers/common/sfc_efx

Re: [dpdk-dev] [PATCH v2 2/2] drivers: add headers install helper

2020-10-22 Thread David Marchand
On Thu, Oct 22, 2020 at 10:16 AM Bruce Richardson wrote: > > On Thu, Oct 22, 2020 at 09:55:45AM +0200, David Marchand wrote: > > A lot of drivers export headers, reproduce the same facility than for > > libraries. > > > > Suggested-by: Bruce Richardson > > Signed-off-by: David Marchand > > --- >

Re: [dpdk-dev] [PATCH] bpf: use helper to install headers

2020-10-22 Thread David Marchand
On Thu, Oct 22, 2020 at 10:21 AM Bruce Richardson wrote: > > On Thu, Oct 22, 2020 at 09:50:38AM +0200, David Marchand wrote: > > Libraries can use the headers variable to install headers. > > > > Signed-off-by: David Marchand > > --- > Acked-by: Bruce Richardson Applied. -- David Marchand

Re: [dpdk-dev] [PATCH v5 2/3] eal: return error code when failure

2020-10-22 Thread David Marchand
On Wed, Oct 21, 2020 at 1:19 PM wangyunjian wrote: > > From: Yunjian Wang > > Fix return value, using -EAGAIN instead of 0 when the callback is busy > and using -ENOENT instead of 0 when the callback is not found. > > Fixes: a753e53d517b ("eal: add device event monitor framework") > Cc: sta...@dp

Re: [dpdk-dev] [PATCH v5 3/3] eal: fix a wrong returned value when callback exists

2020-10-22 Thread David Marchand
On Wed, Oct 21, 2020 at 1:20 PM wangyunjian wrote: > > From: Yunjian Wang > > We should return an error value, when the callback is already exist. > > Fixes: a753e53d517b ("eal: add device event monitor framework") > Cc: sta...@dpdk.org Same question as for the previous patch, should we backport

Re: [dpdk-dev] performance degradation with fpic

2020-10-22 Thread Ali Alnubani
Hi Bruce, Sorry for the delayed response. > -Original Message- > From: Bruce Richardson > Sent: Monday, October 19, 2020 4:02 PM > To: Ali Alnubani > Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon > ; Asaf Penso > Subject: Re: [dpdk-dev] performance degradation with fpic > > On Mon, Oct

Re: [dpdk-dev] [PATCH v2] gso: fix free issue of mbuf gso segments attach to

2020-10-22 Thread Ananyev, Konstantin
> > rte_gso_segment decreased refcnt of pkt by one, but > it is wrong if pkt is external mbuf, pkt won't be > freed because of incorrect refcnt, the result is > application can't allocate mbuf from mempool because > mbufs in mempool are run out of. > > One correct way is application should call

Re: [dpdk-dev] [PATCH v2 1/3] build: add aarch64 clang to meson cross-compile

2020-10-22 Thread Juraj Linkeš
> -Original Message- > From: Jerin Jacob > Sent: Monday, October 5, 2020 4:26 PM > To: Juraj Linkeš > Cc: Thomas Monjalon ; David Marchand > ; Aaron Conole ; > Michael Santana ; dpdk-dev > Subject: Re: [dpdk-dev] [PATCH v2 1/3] build: add aarch64 clang to meson > cross- > compile > >

Re: [dpdk-dev] setting parameters in meson build system

2020-10-22 Thread Bruce Richardson
On Thu, Oct 22, 2020 at 04:43:50PM +0500, Sarosh Arif wrote: > Hello, > > I was using make build system previously, but since it is removed from > dpdk now, I am trying to figure out how to enable/disable certain > things using meson, > > Previously we were able to set some parameters using .conf

Re: [dpdk-dev] performance degradation with fpic

2020-10-22 Thread Bruce Richardson
On Thu, Oct 22, 2020 at 01:17:16PM +, Ali Alnubani wrote: > Hi Bruce, > Sorry for the delayed response. > > > -Original Message- > > From: Bruce Richardson > > Sent: Monday, October 19, 2020 4:02 PM > > To: Ali Alnubani > > Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon > > ; Asaf Pen

[dpdk-dev] [PATCH v2 0/6] add two ports hairpin mode support in mlx5 PMD

2020-10-22 Thread Bing Zhao
This patch set will add the support for hairpin between two ports in mlx5 PMD. v2: * Update the code and reorganize the patch set Bing Zhao (6): net/mlx5: change hairpin queue peer checking net/mlx5: add support for two ports hairpin mode net/mlx5: add support to get hairpin peer ports

[dpdk-dev] [PATCH v2 2/6] net/mlx5: add support for two ports hairpin mode

2020-10-22 Thread Bing Zhao
In order to support hairpin between two ports, mlx5 PMD needs to implement the functions and provide them as the function pointers. The bind and unbind functions are executed per port pairs. All the hairpin queues between the two ports should have the same attributes during queues setup. Different

[dpdk-dev] [PATCH v2 1/6] net/mlx5: change hairpin queue peer checking

2020-10-22 Thread Bing Zhao
In the current implementation of single port mode hairpin, the peer queue should belong to the same port of the current queue. When the two ports hairpin mode is introduced, such checking should be removed to make the hairpin queue setup execute successfully since it is not a valid condition anymor

[dpdk-dev] [PATCH v2 3/6] net/mlx5: add support to get hairpin peer ports

2020-10-22 Thread Bing Zhao
In real-life business, one device could be attached and detached dynamically. The hairpin configuration of this port to/from all the other ports should be enabled and disabled accordingly. The RTE ethdev lib and PMD should provide this ability to get the peer ports list in case that the applicatio

[dpdk-dev] [PATCH v2 5/6] net/mlx5: change hairpin ingress flow validation

2020-10-22 Thread Bing Zhao
In the current implementation of the single port hairpin, there is a implicit splitting process for actions. When inserting a hairpin flow, all the actions will be included with the ingress attribute. The flow engine will check and decide which actions should be moved into the TX flow part, e.g., e

[dpdk-dev] [PATCH v2 4/6] net/mlx5: conditional hairpin auto bind

2020-10-22 Thread Bing Zhao
In single port hairpin mode, after the queues are configured during start up. The binding process will be enabled automatically in the port start phase and the default control flow for egress will be created. When switching to two ports hairpin mode, the auto binding process should be skipped if t

[dpdk-dev] [PATCH v2 6/6] net/mlx5: not split hairpin flow in explicit mode

2020-10-22 Thread Bing Zhao
In the current implementation, the hairpin flow will be split into two flows implicitly if there is some action that only belongs to the TX part. A TX device flow will be inserted by the mlx5 PMD itself. In hairpin between two ports, the explicit TX flow mode will be the only one to be supported.

Re: [dpdk-dev] performance degradation with fpic

2020-10-22 Thread Ali Alnubani
> -Original Message- > From: Bruce Richardson > Sent: Thursday, October 22, 2020 4:58 PM > To: Ali Alnubani > Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon > ; Asaf Penso > Subject: Re: [dpdk-dev] performance degradation with fpic > > On Thu, Oct 22, 2020 at 01:17:16PM +, Ali Alnuba

Re: [dpdk-dev] [PATCH] build: fix version map file references in documentation

2020-10-22 Thread Kinsella, Ray
No worries, happy to. On 22/10/2020 13:11, David Marchand wrote: > On Thu, Oct 22, 2020 at 9:47 AM David Marchand > wrote: >> >> Fixes: 63b3907833d8 ("build: remove library name from version map file name") >> >> Signed-off-by: David Marchand > Acked-by: Ray Kinsella > Acked-by: Bruce Richardso

[dpdk-dev] [PATCH v3 0/5] replace blacklist/whitelist with block/allow

2020-10-22 Thread Stephen Hemminger
This is a revised version of the earlier RFC patch set for changing the blacklist/whitelist terms in DPDK. The first patch is a duplicate from the other patch set about use of master/slave in API. v3 - rebase on main - fix new use of RTE_DEV_BLACKLISTED in fslmc - minor checkpatch report

[dpdk-dev] [PATCH v3 1/5] eal: replace usage of blacklist/whitelist in enum

2020-10-22 Thread Stephen Hemminger
This patch renames the enum values in the EAL include files. As a backward compatible temporary migration tool, define a replacement mapping for old values. The old names relating to blacklist and whitelist are replaced by block list and allow list, but applications may be using the older compatib

[dpdk-dev] [PATCH v3 3/5] eal: replace pci-whitelist/pci-blacklist options

2020-10-22 Thread Stephen Hemminger
Replace -w / --pci-whitelist with -a / --allow options and --pci-blacklist with --block. The -b short option remains unchanged. Allow the old options for now, but print a nag warning since old options are deprecated. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- lib/librte_eal/c

[dpdk-dev] [PATCH v3 2/5] drivers: replace references to blacklist

2020-10-22 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 Acked-by: Hemant Agrawal --- drivers/bus/dpaa/dpaa_bus.c| 7 +++ drivers/bus/fslmc/fslmc_bus.c | 9 - drivers/bus/fslmc/fsl

[dpdk-dev] [PATCH v3 4/5] app/test: use new allowlist and blocklist

2020-10-22 Thread Stephen Hemminger
Test the renamed blocklist and 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 +- app/

[dpdk-dev] [PATCH v3 5/5] doc: change references to blacklist and whitelist

2020-10-22 Thread Stephen Hemminger
There are two areas where documentation needed update. The first was use of whitelist when describing address filtering. The other is the legacy -w whitelist option for PCI which is used in many examples Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- doc/guides/cryptodevs/dpaa2_s

[dpdk-dev] DPDK Release Status Meeting 22/10/2020

2020-10-22 Thread Ferruh Yigit
Meeting minutes of 22 October 2020 -- Agenda: * Release Dates * -rc1 status * Subtrees * LTS * OvS * Opens Participants: * Arm * Broadcom * Debian/Microsoft * Intel * Marvell * Nvidia * NXP * Red Hat Release Dates - * v20.11 dates * -rc1 is releas

[dpdk-dev] [RFC PATCH 1/3] build: add a dpdk-specific meson log file

2020-10-22 Thread Bruce Richardson
While meson itself writes a log file of all the tests it runs and the output of those, to help debug any build problems and provide more developer information it can be helpful to have a logfile created and controlled by the dpdk meson.build files themselves. Since meson doesn't directly support t

[dpdk-dev] [RFC PATCH 0/3] add custom logging to DPDK meson runs

2020-10-22 Thread Bruce Richardson
While the meson.log file in each build directory contains lots of information, much of it is not of interest to developers, and it can be hard to see the explicitly messages from the dpdk build files among all the rest of the content. Futhermore, while at the end of the a configuration run we print

[dpdk-dev] [RFC PATCH 2/3] build: shorten top-level build file

2020-10-22 Thread Bruce Richardson
Move some basic configuration settings out of the top-level meson.build file to the "config/meson.build" file, so as to keep the top-level file as clear and as short as possible. Signed-off-by: Bruce Richardson --- config/meson.build | 18 ++ meson.build| 17 -

[dpdk-dev] [RFC PATCH 3/3] build: write messages to dpdk build log file

2020-10-22 Thread Bruce Richardson
Add additional info into the DPDK-specific logs by copying or replacing all "message()" calls with writes to the log file, making the build configuration, including lists of enabled/disabled drivers etc. available after the meson run has completed. Signed-off-by: Bruce Richardson --- app/test/me

Re: [dpdk-dev] [PATCH v2 0/2] net/mlx5: add vectorized mprq

2020-10-22 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Alexander Kozyrev > Sent: Wednesday, October 21, 2020 11:30 PM > To: dev@dpdk.org > Cc: Raslan Darawsheh ; Matan Azrad > ; Slava Ovsiienko > Subject: [PATCH v2 0/2] net/mlx5: add vectorized mprq > > The vectorized Rx burst function helps to accelerate th

Re: [dpdk-dev] [PATCH v13 1/7] fib: make lookup function type configurable

2020-10-22 Thread Medvedkin, Vladimir
Hi David, On 22/10/2020 12:52, David Marchand wrote: On Mon, Oct 19, 2020 at 5:05 PM Vladimir Medvedkin wrote: Add type argument to dir24_8_get_lookup_fn() Now it supports 3 different lookup implementations: RTE_FIB_DIR24_8_SCALAR_MACRO RTE_FIB_DIR24_8_SCALAR_INLINE RTE_FIB_DIR24_8_SCAL

Re: [dpdk-dev] [PATCH v2 2/2] lpm: hide internal data

2020-10-22 Thread David Marchand
On Wed, Oct 21, 2020 at 5:02 AM Ruifeng Wang wrote: > diff --git a/lib/librte_lpm/rte_lpm.c b/lib/librte_lpm/rte_lpm.c > index 51a0ae578..88d31df6d 100644 > --- a/lib/librte_lpm/rte_lpm.c > +++ b/lib/librte_lpm/rte_lpm.c > @@ -42,9 +42,17 @@ enum valid_flag { > > /** @internal LPM structure. */ >

Re: [dpdk-dev] [PATCH v3 5/5] doc: change references to blacklist and whitelist

2020-10-22 Thread Wang, Haiyue
> -Original Message- > From: Stephen Hemminger > Sent: Thursday, October 22, 2020 22:39 > To: dev@dpdk.org > Cc: Stephen Hemminger ; Luca Boccassi > ; Akhil Goyal > ; Hemant Agrawal ; Griffin, John > ; > Trahe, Fiona ; Jain, Deepak K > ; Pavan Nikhilesh > ; Jerin Jacob ; Richardson, >

Re: [dpdk-dev] [PATCH v2] gso: fix free issue of mbuf gso segments attach to

2020-10-22 Thread Ananyev, Konstantin
> > > > rte_gso_segment decreased refcnt of pkt by one, but > > it is wrong if pkt is external mbuf, pkt won't be > > freed because of incorrect refcnt, the result is > > application can't allocate mbuf from mempool because > > mbufs in mempool are run out of. > > > > One correct way is applicati

[dpdk-dev] [PATCH 0/5] net/mlx5: add Rx buffer split support

2020-10-22 Thread Viacheslav Ovsiienko
This patch adds to PMD the functionality for the receiving buffer split feasture [1] [1] http://patches.dpdk.org/patch/81154/ Signed-off-by: Viacheslav Ovsiienko --- Viacheslav Ovsiienko (5): net/mlx5: add extended Rx queue setup routine net/mlx5: configure Rx queue to support split net/

[dpdk-dev] [PATCH 1/5] net/mlx5: add extended Rx queue setup routine

2020-10-22 Thread Viacheslav Ovsiienko
The routine to provide Rx queue setup with specifying extended receiving buffer description is added. It allows application to specify desired segment lengths, data position offsets in the buffer and dedicated memory pool for each segment. Signed-off-by: Viacheslav Ovsiienko --- drivers/net/mlx5

[dpdk-dev] [PATCH 4/5] net/mlx5: update Rx datapath to support split

2020-10-22 Thread Viacheslav Ovsiienko
Only the regular rx_burst routine is updated to support split, because the vectorized ones does not support scatter and MPRQ does not support split at all. Signed-off-by: Viacheslav Ovsiienko --- drivers/net/mlx5/mlx5_rxq.c | 11 +-- drivers/net/mlx5/mlx5_rxtx.c | 3 ++- 2 files change

[dpdk-dev] [PATCH 2/5] net/mlx5: configure Rx queue to support split

2020-10-22 Thread Viacheslav Ovsiienko
The scatter-gather elements should be configured accordingly to support the buffer split feature. The application provides the desired settings for the segments at the beginning of the packets and PMD pads the buffer chain (if needed) with attributes of last specified segment to accommodate the pac

[dpdk-dev] [PATCH 3/5] net/mlx5: register multiple pool for Rx queue

2020-10-22 Thread Viacheslav Ovsiienko
The split feature for receiving packets was added to the mlx5 PMD, now Rx queue can receive the data to the buffers belonging to the different pools and the memory of all the involved pool must be registered for DMA operations in order to allow hardware to store the data. Signed-off-by: Viacheslav

[dpdk-dev] [PATCH 5/5] net/mlx5: report Rx segmentation capabilies

2020-10-22 Thread Viacheslav Ovsiienko
Add rte_eth_dev_info->rx_seg_capa parameters: - receiving to multiple pools is supported - buffer offsets are supported - no offset alignment requirement - reports the maximal aamount of segments Signed-off-by: Viacheslav Ovsiienko --- drivers/net/mlx5/mlx5_ethdev.c | 4 drivers/net

[dpdk-dev] [PATCH v5 0/3] support both PIO and MMIO BAR for virtio PMD

2020-10-22 Thread 谢华伟(此时此刻)
From: "huawei.xhw" Legacy virtio-pci only supports PIO BAR resource. As we need to create lots of virtio devices and PIO resource on x86 is very limited, we expose MMIO BAR. Kernel supports both PIO and MMIO BAR for legacy virtio-pci device. We handles different type of BAR in the similar way.

[dpdk-dev] [PATCH v5 1/3] PCI: use PCI standard sysfs entry to get PIO address

2020-10-22 Thread 谢华伟(此时此刻)
From: "huawei.xhw" Previously with igb_uio we get PIO address from igb_uio sysfs entry, with uio_pci_generic, we get PIO address from /proc/ioports. Signed-off-by: huawei.xhw --- drivers/bus/pci/linux/pci.c | 77 - drivers/bus/pci/linux/pci_uio.c | 6

[dpdk-dev] [PATCH v5 3/3] PCI: don't use vfio ioctl call to access PIO resource

2020-10-22 Thread 谢华伟(此时此刻)
From: "huawei.xhw" VFIO should use the same way to map/read/write PORT IO as UIO, for virtio PMD. Signed-off-by: huawei.xhw --- drivers/bus/pci/linux/pci.c | 8 drivers/bus/pci/linux/pci_uio.c | 4 +++- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/drivers/bus/pci

[dpdk-dev] [PATCH v5 2/3] PCI: support MMIO in rte_pci_ioport_map/unap/read/write

2020-10-22 Thread 谢华伟(此时此刻)
From: "huawei.xhw" If IO BAR, we get PIO address. If MMIO BAR, we get mapped virtual address. We distinguish PIO and MMIO by their address like how kernel does. ioread/write8/16/32 is provided to access PIO/MMIO. BTW, for virtio on arch other than x86, BAR flag indicates PIO but is mapped. Signe

[dpdk-dev] [PATCH v2] net/mlx5: implement tunnel offload API

2020-10-22 Thread Gregory Etelson
Tunnel Offload API provides hardware independent, unified model to offload tunneled traffic. Key model elements are: - apply matches to both outer and inner packet headers during entire offload procedure; - restore outer header of partially offloaded packet; - model is implemented as a set of

[dpdk-dev] [PATCH 2/2] test/crypto: disable wireless tests for symmetric API

2020-10-22 Thread Pablo de Lara
SNOW3G, KASUMI and ZUC algorithms have been added to aesni_mb PMD in this release. These algorithms are not supported for the crypto symmetric API, so the crypto tests for these algorithms are disabled. Fixes: ae8e085c608d ("crypto/aesni_mb: support KASUMI F8/F9") Fixes: 6c42e0cf4d12 ("crypto/aesn

[dpdk-dev] [PATCH 1/2] crypto/aesni_mb: support Chacha-Poly in symmetric API

2020-10-22 Thread Pablo de Lara
Add support for Chacha20-Poly1305 in the crypto symmetric API. Signed-off-by: Pablo de Lara --- drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 21 +++-- 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c b/drivers/crypto/aesni

Re: [dpdk-dev] [PATCH v6 1/2] app/testpmd: fix max rx packet length for VLAN packets

2020-10-22 Thread Ferruh Yigit
On 10/22/2020 9:48 AM, SteveX Yang wrote: When the max rx packet length is smaller than the sum of mtu size and ether overhead size, it should be enlarged, otherwise the VLAN packets will be dropped. Fixes: 35b2d13fd6fd ("net: add rte prefix to ether defines") Signed-off-by: SteveX Yang ---

Re: [dpdk-dev] [PATCH v6 2/2] librte_ethdev: fix MTU size exceeds max rx packet length

2020-10-22 Thread Ferruh Yigit
On 10/22/2020 9:48 AM, SteveX Yang wrote: If max rx packet length is smaller then MTU + Ether overhead, that will drop all MTU size packets. Update the MTU size according to the max rx packet and Ether overhead. Fixes: 59d0ecdbf0e1 ("ethdev: MTU accessors") Signed-off-by: SteveX Yang --- li

Re: [dpdk-dev] [PATCH V3 1/1] net/mlx5: fix IPv6 next proto validation

2020-10-22 Thread Ferruh Yigit
On 10/22/2020 11:01 AM, Raslan Darawsheh wrote: Hi, -Original Message- From: Eli Britstein Sent: Thursday, October 22, 2020 11:26 AM To: dev@dpdk.org Cc: Dekel Peled ; Ori Kam ; Matan Azrad ; Raslan Darawsheh ; Slava Ovsiienko ; Shahaf Shuler ; Asaf Penso ; Eli Britstein Subject: [PAT

Re: [dpdk-dev] [PATCH v6 2/2] librte_ethdev: fix MTU size exceeds max rx packet length

2020-10-22 Thread Ananyev, Konstantin
> > Update the MTU size according to the max rx packet and Ether overhead. > > Fixes: 59d0ecdbf0e1 ("ethdev: MTU accessors") > > Signed-off-by: SteveX Yang > --- > lib/librte_ethdev/rte_ethdev.c | 14 ++ > 1 file changed, 14 insertions(+) > > diff --git a/lib/librte_ethdev/rte_e

Re: [dpdk-dev] [PATCH] eal: create a runtime directory on windows

2020-10-22 Thread Ranjit Menon
On 10/21/2020 1:26 PM, Pallavi Kadam wrote: Added eal_create_runtime_dir() function in order to run any application as a user that does not have administrator access. Currently, since there is no runtime directory set, the code tries to create a file in C:\ which is only writable with administr

Re: [dpdk-dev] [PATCH v6 1/4] rcu: build on Windows

2020-10-22 Thread Kadam, Pallavi
On 10/21/2020 3:50 PM, Dharmik Thakkar wrote: Build the lib for Windows. Signed-off-by: Dharmik Thakkar Tested-by: Dmitry Kozlyuk --- Builds successfully on Windows. Tested-by: Pallavi Kadam

Re: [dpdk-dev] [PATCH] eal: create a runtime directory on windows

2020-10-22 Thread Dmitry Kozlyuk
> /* Detect if we are a primary or a secondary process */ > enum rte_proc_type_t > eal_proc_type_detect(void) > @@ -181,6 +231,13 @@ eal_parse_args(int argc, char **argv) > } > } > > + /* create runtime data directory */ > + if (internal_conf->no_shconf == 0 && > +

[dpdk-dev] [PATCH 2/2] net/netvsc: introduce driver parameter to control the use of external mbuf on receiving data

2020-10-22 Thread Long Li
From: Long Li When receiving packets, netvsp puts data in a buffer mapped through UIO. Depending on packet size, netvsc may attach the buffer as an external mbuf. This is not a problem if this mbuf is consumed in the application, and the application can correctly read data out of an external mbuf

[dpdk-dev] [PATCH 1/2] net/netvsc: allow setting rx and tx copy break

2020-10-22 Thread Long Li
From: Stephen Hemminger The values for Rx and Tx copy break should be tunable rather than hard coded constants. The rx_copybreak sets the threshold where the driver uses an external mbuf to avoid having to copy data. Setting 0 for copybreak will cause driver to always create an external mbuf. Se

Re: [dpdk-dev] [PATCH v2] eal: fix dereference after null check

2020-10-22 Thread David Marchand
On Thu, Oct 15, 2020 at 10:43 AM wangyunjian wrote: > > From: Yunjian Wang > > This patch fixes (dereference after null check) coverity issue. > For this reason, we should add null check at the beginning of the > function and return error directly if the 'intr_handle' is null. > > Coverity issue:

Re: [dpdk-dev] [PATCH] efd: fix tailq entry leak in error path

2020-10-22 Thread David Marchand
On Tue, Sep 29, 2020 at 5:32 AM wangyunjian wrote: > > From: Yunjian Wang > > In rte_efd_create() allocated memory for tailq entry, we should > free it when error happens, otherwise it will lead to memory leak. > > Fixes: 56b6ef874f80 ("efd: new Elastic Flow Distributor library") > Cc: sta...@dpd

Re: [dpdk-dev] [PATCH] hash: use x86 common flag for jhash

2020-10-22 Thread David Marchand
On Thu, Oct 22, 2020 at 10:23 AM Bruce Richardson wrote: > On Thu, Oct 22, 2020 at 09:59:59AM +0200, David Marchand wrote: > > jhash has been forgotten when factorising the x86 arch check. > > > > Fixes: dbf17d44f375 ("hash: use common x86 flag") > > > > Signed-off-by: David Marchand > Acked-by:

[dpdk-dev] [Bug 563] mlx5 silently fails to create an ethernet rte_flow rule

2020-10-22 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=563 Bug ID: 563 Summary: mlx5 silently fails to create an ethernet rte_flow rule Product: DPDK Version: 20.11 Hardware: x86 OS: Linux Status: UNCONFIRMED

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

2020-10-22 Thread Stephen Hemminger
This patch renames the enum values in the EAL include files. As a backward compatible temporary migration tool, define a replacement mapping for old values. The old names relating to blacklist and whitelist are replaced by block list and allow list, but applications may be using the older compatib

[dpdk-dev] [PATCH v4 0/5] replace blacklist/whitelist with block/allow

2020-10-22 Thread Stephen Hemminger
This is a revised version of the earlier RFC patch set for changing the blacklist/whitelist terms in DPDK. The first patch is a duplicate from the other patch set about use of master/slave in API. Note: This may cause some warnings in existing programs in the CI tests using the -w flag. It also

Re: [dpdk-dev] [PATCH v7 12/14] doc: remove references to make from contributing guide

2020-10-22 Thread Thomas Monjalon
21/10/2020 10:17, Ciara Power: > -* ``CONFIG_RTE_ARCH`` is a string that contains the name of the architecture. > -* ``CONFIG_RTE_ARCH_I686``, ``CONFIG_RTE_ARCH_X86_64``, > ``CONFIG_RTE_ARCH_X86_64_32`` or ``CONFIG_RTE_ARCH_PPC_64`` are defined only > if we are building for those architectures. >

[dpdk-dev] [PATCH v4 2/5] drivers: replace references to blacklist

2020-10-22 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 Acked-by: Hemant Agrawal --- drivers/bus/dpaa/dpaa_bus.c| 7 +++ drivers/bus/fslmc/fslmc_bus.c | 9 - drivers/bus/fslmc/fsl

[dpdk-dev] [PATCH v4 3/5] eal: replace pci-whitelist/pci-blacklist options

2020-10-22 Thread Stephen Hemminger
Replace -w / --pci-whitelist with -a / --allow options and --pci-blacklist with --block. The -b short option remains unchanged. Allow the old options for now, but print a nag warning since old options are deprecated. Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- lib/librte_eal/c

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

2020-10-22 Thread Stephen Hemminger
Test the renamed blocklist and 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 +- app/

[dpdk-dev] [PATCH v4 5/5] doc: change references to blacklist and whitelist

2020-10-22 Thread Stephen Hemminger
There are two areas where documentation needed update. The first was use of whitelist when describing address filtering. The other is the legacy -w whitelist option for PCI which is used in many examples Signed-off-by: Stephen Hemminger Acked-by: Luca Boccassi --- doc/guides/cryptodevs/dpaa2_s

Re: [dpdk-dev] [PATCH v7 00/14] remove make support in DPDK

2020-10-22 Thread Thomas Monjalon
21/10/2020 10:17, Ciara Power: > 147 files changed, 729 insertions(+), 2065 deletions(-) Huge patchset! After few adjustments (including new driver and remaining CONFIG_RTE), Applied, thanks for the great effort.

Re: [dpdk-dev] [PATCH] doc: update build guide for armv8 crypto PMD

2020-10-22 Thread Thomas Monjalon
14/10/2020 11:24, Ruifeng Wang: > Removed references to Makefile and added guide about > building by using meson. > > Also added the command to create virtual device. > > Suggested-by: Akhil Goyal > Signed-off-by: Ruifeng Wang Applied, thanks

Re: [dpdk-dev] [PATCH] doc: update build guide for armv8 crypto PMD

2020-10-22 Thread Thomas Monjalon
14/10/2020 11:24, Ruifeng Wang: > +* Build DPDK with meson option > ``-Darmv8_crypto_dir=``: > + > + meson -Darmv8_crypto_dir= build Please, could you add pkg-config support to this lib? I would like to remove with option from DPDK, and use the standard variable PKG_CONFIG_PATH instead. Than

Re: [dpdk-dev] [PATCH] doc: update build guide for armv8 crypto PMD

2020-10-22 Thread Honnappa Nagarahalli
+ Dharmik > -Original Message- > From: Thomas Monjalon > Sent: Thursday, October 22, 2020 4:18 PM > To: Ruifeng Wang > Cc: dev@dpdk.org; akhil.go...@nxp.com; Honnappa Nagarahalli > ; nd > Subject: Re: [dpdk-dev] [PATCH] doc: update build guide for armv8 crypto > PMD > > 14/10/2020 11:2

[dpdk-dev] [PATCH 0/5] cleanup comments and logs about config options

2020-10-22 Thread Thomas Monjalon
Below patches are cleaning traces of CONFIG_RTE_ after make removal, except one occurence in app/test/test_cryptodev.c (left as exercise). PS: In reality I don't know what must be done for this case about QAT. Thomas Monjalon (5): eal: remove comment about old partition option mem: fix config

[dpdk-dev] [PATCH 2/5] mem: fix config name in error logs

2020-10-22 Thread Thomas Monjalon
When introducing the new option CONFIG_RTE_MAX_MEM_MB_PER_TYPE, some logs were referencing a wrong name: CONFIG_RTE_MAX_MEM_PER_TYPE. Fixes: 66cc45e293ed ("mem: replace memseg with memseg lists") Cc: sta...@dpdk.org Signed-off-by: Thomas Monjalon --- lib/librte_eal/freebsd/eal_memory.c | 2 +-

[dpdk-dev] [PATCH 1/5] eal: remove comment about old partition option

2020-10-22 Thread Thomas Monjalon
The main initialization function (rte_eal_init) has documentation about a feature from another era: memory partition. Curiously, this lost treasure is found only now, suggesting there may be other interesting things to discover in the doc. To all aspiring Indiana Jones: the hunt is open! Signed-o

[dpdk-dev] [PATCH 3/5] lib: remove config prefix used with make

2020-10-22 Thread Thomas Monjalon
The config options CONFIG_RTE_* are simple RTE_* defines with meson. Now that make support is dropped, update the names in logs and comments. Signed-off-by: Thomas Monjalon --- lib/librte_eal/arm/include/rte_atomic_32.h | 2 +- lib/librte_eal/arm/include/rte_atomic_64.h | 2 +- lib/librte_ea

[dpdk-dev] [PATCH 4/5] drivers: remove config prefix used with make

2020-10-22 Thread Thomas Monjalon
The config options CONFIG_RTE_* are simple RTE_* defines with meson. Now that make support is dropped, update the names in log and comments. Signed-off-by: Thomas Monjalon --- drivers/net/bnxt/bnxt_ethdev.c | 2 +- drivers/net/ixgbe/ixgbe_rxtx.c | 2 +- drivers/net/sfc/sfc_ef10_essb_rx.c

[dpdk-dev] [PATCH 5/5] doc: remove references to make from known issues

2020-10-22 Thread Thomas Monjalon
The config options CONFIG_RTE_* are simple RTE_* defines with meson. Now that make support is dropped, update the HPET config reference. The comment about the AVX512 config option is not relevant anymore. Signed-off-by: Thomas Monjalon --- doc/guides/rel_notes/known_issues.rst | 8 ++-- 1 f

[dpdk-dev] [PATCH] net/bnxt: fix incorrect boolean operator usage

2020-10-22 Thread Lance Richardson
Use boolean AND operator instead of bitwise operator. Coverity issue: 323488 Fixes: b42c15c83e88 ("net/bnxt: support trusted VF") Signed-off-by: Lance Richardson Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dri

[dpdk-dev] [PATCH] net/bnxt: use shorter SIMD initializers

2020-10-22 Thread Lance Richardson
Make SIMD initialization code less verbose by using appropriate intrinsics when all lanes of a vector are initialized to the same value. Signed-off-by: Lance Richardson Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_rxtx_vec_neon.c | 58 +++ drivers/net/bnxt/bnxt_r

[dpdk-dev] [PATCH] net/bnxt: update PMD supported features

2020-10-22 Thread Lance Richardson
Mark "BSD nic_uio", "Usage doc", and "Perf doc" as supported for the bnxt PMD. Signed-off-by: Lance Richardson Reviewed-by: Ajit Kumar Khaparde --- doc/guides/nics/features/bnxt.ini | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/guides/nics/features/bnxt.ini b/doc/guides/nics/featu

Re: [dpdk-dev] [PATCH v2] gso: fix free issue of mbuf gso segments attach to

2020-10-22 Thread Hu, Jiayu
> -Original Message- > From: Ananyev, Konstantin > Sent: Thursday, October 22, 2020 11:34 PM > To: yang_y...@163.com; dev@dpdk.org > Cc: Hu, Jiayu ; techbo...@dpdk.org; > tho...@monjalon.net; yangy...@inspur.com > Subject: RE: [PATCH v2] gso: fix free issue of mbuf gso segments attach t

Re: [dpdk-dev] [PATCH v2 1/2] net/ixgbe: fix unchecked return value

2020-10-22 Thread Zhang, Qi Z
> -Original Message- > From: dev On Behalf Of Wang, Haiyue > Sent: Wednesday, October 21, 2020 1:06 AM > To: Walsh, Conor ; Guo, Jia ; > Yigit, Ferruh ; Awal, Mohammad Abdul > ; Doherty, Declan > ; sta...@dpdk.org > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/2] net/ixgbe: fi

Re: [dpdk-dev] [PATCH v2 2/2] net/ixgbe: remove rte_panic

2020-10-22 Thread Zhang, Qi Z
> -Original Message- > From: dev On Behalf Of Wang, Haiyue > Sent: Wednesday, October 21, 2020 1:06 AM > To: Walsh, Conor ; Guo, Jia ; > Yigit, Ferruh ; Awal, Mohammad Abdul > ; Doherty, Declan > ; sta...@dpdk.org > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 2/2] net/ixgbe: re

Re: [dpdk-dev] [PATCH 4/5] drivers: remove config prefix used with make

2020-10-22 Thread Wang, Haiyue
> -Original Message- > From: Thomas Monjalon > Sent: Friday, October 23, 2020 06:05 > To: dev@dpdk.org > Cc: Yigit, Ferruh ; david.march...@redhat.com; > Richardson, Bruce > ; Burakov, Anatoly ; > olivier.m...@6wind.com; > Power, Ciara ; akhil.go...@nxp.com; Trahe, Fiona > ; > Griffin,

[dpdk-dev] [PATCH v3 02/11] net/bnxt: changes to support two table scopes

2020-10-22 Thread Ajit Khaparde
From: Jay Ding New API to map a PARIF to an EEM table scope (set of Rx and Tx EEM base addresses). It uses HWRM_TF_GLOBAL_CFG_SET HWRM to configure. PARIF is handler to a partition of the physical port. Adjustments to tf_global_cfg_set() to reduce overhead and nominal name clarification. Signed-

[dpdk-dev] [PATCH v3 00/11] bnxt fixes and enhancements to TRUFLOW support

2020-10-22 Thread Ajit Khaparde
This patchset has support for VXLAN decap full offload, SR device support in tf_core and a few bug fixes. v1->v2: - Squashed some patches. - Updated commit logs. - Fixed a compilation issue reported during clang builds. v2->v3: - Added the SPDX header for cfa_resource_types.h in patch [01/11]

[dpdk-dev] [PATCH v3 01/11] net/bnxt: add stingray support to core layer

2020-10-22 Thread Ajit Khaparde
From: Peter Spreadborough - Moved P4 specific code under the P4 directory - Added P45 skeleton code for SR to build on - Add SR support in TRUFLOW core layer. The TRUFLOW core or the tf-core is a shim layer which communicates with the CFA block in the hardware. Signed-off-by: Peter Spreadborough

[dpdk-dev] [PATCH v3 03/11] net/bnxt: add table scope to PF Mapping

2020-10-22 Thread Ajit Khaparde
From: Farah Smith Add table scope to PF Mapping for SR and Wh+ devices. Legacy devices require PF set of base addresses for EEM operation. A table scope id is a logical construct and is mapped to the PF associated with the communications channel used. In the case of a VF, the parent PF is used.

[dpdk-dev] [PATCH v3 05/11] net/bnxt: fix infinite loop in flow query count

2020-10-22 Thread Ajit Khaparde
From: Somnath Kotur `nxt_resource_idx` could be zero in some cases which is invalid and should be part of the while loop condition. Also synchronize access to the flow db using the fdb_lock Fixes: 306c2d28e247 ("net/bnxt: support count action in flow query") Signed-off-by: Somnath Kotur Review

Re: [dpdk-dev] [PATCH 1/3] ethdev: remove API to config L2 tunnel EtherType

2020-10-22 Thread Wang, Haiyue
> -Original Message- > From: Andrew Rybchenko > Sent: Thursday, October 22, 2020 18:06 > To: Lu, Wenzhuo ; Xing, Beilei ; > Iremonger, Bernard > ; Ray Kinsella ; Neil Horman > ; Guo, > Jia ; Wang, Haiyue ; Thomas > Monjalon ; > Yigit, Ferruh ; Andrew Rybchenko > > Cc: dev@dpdk.org > S

[dpdk-dev] [PATCH v3 04/11] net/bnxt: update ULP resource counts

2020-10-22 Thread Ajit Khaparde
From: Peter Spreadborough - Update the SR ULP requested resource counts to reflect those available after AFM resources are accounted for. - Add build option to select either 2 or 4 slot EM entries. The default is 4 slot entries. Signed-off-by: Peter Spreadborough Signed-off-by: Farah Smith Rev

[dpdk-dev] [PATCH v3 08/11] net/bnxt: add mapper support for wildcard TCAM

2020-10-22 Thread Ajit Khaparde
From: Kishore Padmanabha Added support for the key and mask fields encoding for the wildcard tcam entry. Signed-off-by: Kishore Padmanabha Reviewed-by: Mike Baucom Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/ulp_mapper.c | 47 -- drivers/net/bnxt/tf_ulp/ulp_utils.c | 125

[dpdk-dev] [PATCH v3 06/11] net/bnxt: add support for flow counter accumulation

2020-10-22 Thread Ajit Khaparde
From: Kishore Padmanabha Add support for flow counter accumulation. In case of hierarchical flows, involving parent and child flows, the child flow counters are aggregated to get the parent flow counter information. Signed-off-by: Kishore Padmanabha Reviewed-by: Shahaji Bhosle Reviewed-by: Aji

[dpdk-dev] [PATCH v3 07/11] net/bnxt: change HWRM command to create reps

2020-10-22 Thread Ajit Khaparde
From: Somnath Kotur Use cfa pair alloc for configuring reps. Instead of cfa_vfr_alloc for Wh+ and cfa_pair_alloc for Stingray, converge to cfa_pair_alloc/free for both devices. Set the command request structure bits accordingly. As part of this, remove the old cfa_vfr_alloc cmd definitions as FW

[dpdk-dev] [PATCH v3 11/11] net/bnxt: add VXLAN decap offload support

2020-10-22 Thread Ajit Khaparde
From: Venkat Duvvuru VXLAN decap offload can happen in stages. The offload request may not come as a single flow request rather may come as two flow offload requests F1 & F2. This patch is adding support for this two stage offload design. The match criteria for F1 is O_DMAC, O_SMAC, O_DST_IP, O_U

<    1   2   3   >