Hi,
> -Original Message-
> From: Haifei Luo
> Sent: Monday, February 21, 2022 10:27 AM
> To: Matan Azrad ; Ori Kam ; Slava
> Ovsiienko ; Shahaf Shuler
> Cc: dev@dpdk.org; NBU-Contact-Thomas Monjalon (EXTERNAL)
> ; Raslan Darawsheh
> Subject: [PATCH] net/mlx5: functions refactor for gett
Hi,
> -Original Message-
> From: Michael Baum
> Sent: Monday, February 14, 2022 11:00 AM
> To: dev@dpdk.org
> Cc: Matan Azrad ; Raslan Darawsheh
> ; Slava Ovsiienko
> Subject: [PATCH 0/2] fix shared RxQ creation
>
> Fix error flow in RxQ creation and optimize it.
>
> Michael Baum (2):
'recv()' fills the 'buf', later 'strlcpy()' used to copy from this buffer.
But as coverity warns 'recv()' doesn't guarantee that 'buf' is
null-terminated, but 'strlcpy()' requires it.
Enlarge 'buf' size to 'EAL_UEV_MSG_LEN + 1' and ensure the last one can
be set to 0 when received buffer size is E
Testpmd forwards packets in checksum mode that it needs to calculate
the checksum of each layer's protocol.
When setting the hardware calculates the outer UDP checksum and the
software calculates the outer IP checksum, the dev->tx_pkt_burst in
ice_set_tx_function is set to ice_xmit_pkts_vec_avx2.
> -Original Message-
> From: Shijith Thotton
> Sent: Tuesday, February 22, 2022 12:34 PM
> To: Van Haaren, Harry ; Gujjar, Abhinandan S
> ; Jerin Jacob ;
> Hemant Agrawal ; Nipun Gupta
>
> Cc: Jerin Jacob Kollanukkaran ; dev@dpdk.org
> Subject: RE: [PATCH v5] app/eventdev: add crypto pr
On Wed, Feb 23, 2022 at 12:01 AM wrote:
>
> From: Jerin Jacob
>
> Except for the cn*_rte_flow* files, none of the other
> file has rte in the file name, remove the
> rte to unify the file name across the directory.
>
> Signed-off-by: Jerin Jacob
> Acked-by: Kiran Kumar K
Applied to dpdk-next-n
> v11 - v12:
> - fixed a compile issue
>
> v10:
> - rebase to the lastest for-main
> - fix of build rerror when RTE_LOG_DEBUG enabled
>
> v9:
> - commit messages reword
> - fix of unused function error
>
> v8:
> - rebase to 22.03-rc1
>
> v7:
> - fix of pointer cast compile error in x86
>
> v6:
> This patchset adds following asymmetric crypto algorithms to Intel QuickAssist
> Technology driver:
> - ECDSA
> - ECPM
>
> Code also was refactored to be more readable and easier scalable.
>
> v2:
> - add actual implementation
> v3:
> - rebase against newest changes
>
> Depends-on: patch-10779
> > Usage of unprotected port mask without any configured SA inbound, will
> cause
> > use of uninitialized SA context, so disallow such configuration.
> >
> > Signed-off-by: Volodymyr Fialko
> >
> > ---
> > examples/ipsec-secgw/ipsec-secgw.c | 3 +++
> > 1 file changed, 3 insertions(+)
> >
>
>
Hi Honnappa,
On 23/02/2022 06:13, Honnappa Nagarahalli wrote:
Question for stable maintainers.
DTS WG would like this patch to be merged in 21.11 and 20.11 stable releases
even though it is not a bug fix. This allows for a single DTS version to be
used with both those stable releases and mast
> Added missing step for converting SHA request files to correct
> format. Replaced AES_GCM with GCM to follow the correct
> naming format.
>
> Fixes: 3d0fad56b74 ("examples/fips_validation: add crypto FIPS application")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Jakub Poczatek
> Acked-by: Fan Zha
resolve compilation error caused due to gcc 12 version
error: storing the address of local variable message in *error.message
Fixes: 26b034f78ca7 ("net/cnxk: support to validate meter policy")
Cc: sta...@dpdk.org
Signed-off-by: Rakesh Kudurumalla
Change-Id: Ia4cf18471ae33691c34229ab537f208389e0
Update error prints in data path to RTE_LOG_DP().
Error prints in fast path are not good for performance
as they slow down the application when few bad packets are
received.
Signed-off-by: Nithin Dabilpuram
Acked-by: Akhil Goyal
---
v2:
- Fixed issue with warning in patch 4/4 by checking for se
Fix packet processing to skip after mbuf is freed instead of
touching and Tx'ing it.
Also free vector event buffer in event worker when after processing
there is no pkt to be enqueued to Tx adapter.
Fixes: 86738ebe1e3d ("examples/ipsec-secgw: support event vector")
Cc: scha...@marvell.com
Cc: sta
Add support to enable per port packet pool and also override
vector pool size from command line args. This is useful
on some HW to tune performance based on usecase.
Signed-off-by: Nithin Dabilpuram
Acked-by: Akhil Goyal
---
doc/guides/sample_app_ug/ipsec_secgw.rst | 7 +++
examples/ipsec-secg
resolve compilation error caused due to gcc 12 version
error: storing the address of local variable message in *error.message
Fixes: 26b034f78ca7 ("net/cnxk: support to validate meter policy")
Cc: sta...@dpdk.org
Signed-off-by: Rakesh Kudurumalla
---
v2 : removed changeid
drivers/net/cnxk/cnx
On 2/18/22 7:28 PM, Nithin Kumar Dabilpuram wrote:
On 2/18/22 12:47 AM, Ananyev, Konstantin wrote:
Enable Tx IPv4 checksum offload only when Tx inline crypto is needed.
In other cases such as Tx Inline protocol offload, checksum
computation
is implicitly taken care by HW.
Why is that?
>-Original Message-
>From: Gujjar, Abhinandan S
>Sent: Wednesday, February 23, 2022 2:32 PM
>To: Shijith Thotton ; Van Haaren, Harry
>; Jerin Jacob ; Hemant
>Agrawal ; Nipun Gupta
>Cc: Jerin Jacob Kollanukkaran ; dev@dpdk.org
>Subject: [EXT] RE: [PATCH v5] app/eventdev: add crypto produ
On 2/23/2022 7:17 AM, Morten Brørup wrote:
From: Stephen Hemminger [mailto:step...@networkplumber.org]
Sent: Tuesday, 22 February 2022 17.03
On Tue, 22 Feb 2022 15:13:53 +0100
Morten Brørup wrote:
From: Megha Ajmera [mailto:megha.ajm...@intel.com]
Sent: Tuesday, 22 February 2022 14.19
Maskin
wt., 22 lut 2022 o 23:21 Ferruh Yigit napisał(a):
>
> On 2/22/2022 6:11 PM, Michal Krawczyk wrote:
> > Hi,
> >
> > this set contains new ENA features:
> >
> > * New xstats.
> > * Reconfigurable link status event.
> > * Usage of the optimized memcpy on arm/arm64.
> > * Better MP support.
> > * Reco
Hi Arek,
I agree that these cases may not be particularly useful from cryptographic
standpoint, but how do we say from what values a PMD "must" support? Are you
having some trouble supporting some of these values? Should we consider
extending capabilities to help with this rather than just remo
> -Original Message-
> From: Shijith Thotton
> Sent: Wednesday, February 23, 2022 10:02 AM
> To: Gujjar, Abhinandan S ; Van Haaren, Harry
> ; Jerin Jacob ; Hemant
> Agrawal ; Nipun Gupta
> Cc: Jerin Jacob Kollanukkaran ; dev@dpdk.org
> Subject: RE: [PATCH v5] app/eventdev: add crypto prod
The table remove callback function is trying to destroy the
matchers list associated with table entries without checking
if the list is valid, which causes nullptr dereference.
Fixed by validating the matchers list before destroying it.
Issue can be reproduced with testpmd on Windows, when you run
They are both included in DPDK21.11 according to my alignment with Qi and
Dapeng.
> -Original Message-
> From: Dong, Yao
> Sent: Tuesday, February 22, 2022 8:07 PM
> To: Kevin Traynor
> Cc: Zhang, Qi Z ; Navin Srinivas
> ; Wang, Yixue ; Yang,
> Qiming ; Zhang, Liheng ;
> dev@dpdk.org; s
On 2/23/2022 6:41 AM, Steve Yang wrote:
Cast 1 to type uint64_t to avoid overflow.
CID 375812 (#1 of 1):
Unintentional integer overflow (OVERFLOW_BEFORE_WIDEN)
overflow_before_widen: Potentially overflowing expression 1 << 2 * i + 1
with type int (32 bits, signed) is evaluated using 32-bit arith
Remove redundant debug logs and unify the log format.
v2:
- Fix compile error.
- Merge broken log message lines.
- Use 'RTE_ETHER_ADDR_PRT_FMT' for MAC format.
Jiawen Wu (2):
net/ngbe: fix debug log
net/txgbe: fix debug log
drivers/net/ngbe/base/ngbe_eeprom.c | 22 +--
drivers/net/ngbe/b
Remove 'DEBUGFUNC' due to too many invalid debug log prints, unify the
DEBUG level macros.
Fixes: cc934df178ab ("net/ngbe: add log and error types")
Cc: sta...@dpdk.org
Signed-off-by: Jiawen Wu
---
drivers/net/ngbe/base/ngbe_eeprom.c | 22 +---
drivers/net/ngbe/base/ngbe_hw.c | 152 +
Remove 'DEBUGFUNC' due to too many invalid debug log prints, unify the
DEBUG level macros.
Fixes: 7dc117068a7c ("net/txgbe: support probe and remove")
Cc: sta...@dpdk.org
Signed-off-by: Jiawen Wu
---
drivers/net/txgbe/base/meson.build| 2 +-
drivers/net/txgbe/base/txgbe_eeprom.c | 41 ++-
Hi Anoob,
You are right QAT doesn't like these test vectors - for the reason Arek stated
in the patch.
We are in the same page on updating the capability - we already planned for
next release.
Since you also agree these cases are not useful - can we do both instead
(removing these test vectors
> Subject: [PATCH] net/mlx5: fix destroying empty matchers list
>
> The table remove callback function is trying to destroy the matchers list
> associated with table entries without checking if the list is valid, which
> causes
> nullptr dereference.
> Fixed by validating the matchers list before
Following test cases were added:
- IPv4 TTL
- IPv6 hop limit
- IPv4 L4 checksum in transport mode
v2:
- rebase to the latest for-main
Volodymyr Fialko (2):
test/crypto: add TTL and hop limit decrement cases
test/crypto: add L4 checksum case for transport mode
app/test/test_cryptodev.c
Add test cases to verify TTL and hop limit decrement with lookaside
IPsec offload.
Signed-off-by: Volodymyr Fialko
Acked-by: Anoob Joseph
---
app/test/test_cryptodev.c| 29 +++
app/test/test_cryptodev_security_ipsec.c | 37
app/test/test_
Added test case to verify L4 checksum offload in IPsec transport mode.
Signed-off-by: Volodymyr Fialko
Acked-by: Anoob Joseph
---
app/test/test_cryptodev.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 694b073f4f
+Thomas, you may be interested in this discussion about applications using an
uint64_t bit mask to identify active lcores.
> From: Ferruh Yigit [mailto:ferruh.yi...@intel.com]
> Sent: Wednesday, 23 February 2022 11.03
>
> On 2/23/2022 7:17 AM, Morten Brørup wrote:
> >> From: Stephen Hemminger [m
> Add test cases to verify TTL and hop limit decrement with lookaside
> IPsec offload.
>
> Signed-off-by: Volodymyr Fialko
> Acked-by: Anoob Joseph
Series Applied to dpdk-next-crypto
Thanks.
> Update error prints in data path to RTE_LOG_DP().
> Error prints in fast path are not good for performance
> as they slow down the application when few bad packets are
> received.
>
> Signed-off-by: Nithin Dabilpuram
> Acked-by: Akhil Goyal
> ---
Series Applied to dpdk-next-crypto
Thanks.
> Coverity flags an issue with 32-bit value. If max ethports value is
> configured with a value larger than 32 there will be an issue.
> Coverity issue: 375863 Unintentional integer overflow
>
> Fixes: 387259bd6c67 ("examples/l2fwd-crypto: add sample application")
> Cc: declan.dohe...@intel.com
>
> Coverity issue: 375803, 375813, 375819, 375827, 375831
> The actual maximum number of gather pointers should have been limited
> Fixes: 5ad5060f8f7a ("baseband/acc100: add LDPC processing functions")
>
> Signed-off-by: Nicolas Chautru
Cc: sta...@dpdk.org
Applied to dpdk-next-crypto
Thanks
> -Original Message-
> From: Dooley, Brian
> Sent: Tuesday, February 22, 2022 9:55 AM
> To: dev@dpdk.org
> Cc: Dooley, Brian ; Zhang, Roy Fan
> ; sta...@dpdk.org; Jay Zhou
>
> Subject: [PATCH v2] crypto/virtio: fix out of bounds access bug
>
> Coverity flags an untrusted loop bound. Chec
> -Original Message-
> From: Dooley, Brian
> Sent: Tuesday, February 22, 2022 9:55 AM
> To: dev@dpdk.org
> Cc: Dooley, Brian ; Zhang, Roy Fan
> ; sta...@dpdk.org; Jay Zhou
>
> Subject: [PATCH v2] crypto/virtio: fix out of bounds access bug
>
> Coverity flags an untrusted loop bound. Chec
> > Coverity flags an untrusted loop bound. Check length of session iv.
> >
> > Coverity issue: 375802
> >
> > Fixes: b063e843fa03 ("crypto/virtio: fix IV physical address")
> > Cc: roy.fan.zh...@intel.com
> > Cc: sta...@dpdk.org
> >
> > Signed-off-by: Brian Dooley
> >
> > ---
> Reacking as the
On 2/23/2022 10:42 AM, Morten Brørup wrote:
+Thomas, you may be interested in this discussion about applications using an
uint64_t bit mask to identify active lcores.
From: Ferruh Yigit [mailto:ferruh.yi...@intel.com]
Sent: Wednesday, 23 February 2022 11.03
On 2/23/2022 7:17 AM, Morten Brørup
On 2/23/2022 8:49 AM, Steve Yang wrote:
'recv()' fills the 'buf', later 'strlcpy()' used to copy from this buffer.
But as coverity warns 'recv()' doesn't guarantee that 'buf' is
null-terminated, but 'strlcpy()' requires it.
Enlarge 'buf' size to 'EAL_UEV_MSG_LEN + 1' and ensure the last one can
From: Elena Agostini
To enable the gpudev rte_gpu_mem_cpu_map feature to expose
GPU memory to the CPU, the GPU CUDA driver library needs
the GDRCopy library and driver.
If DPDK is built without GDRCopy, the GPU CUDA driver returns
error if the is invoked rte_gpu_mem_cpu_map.
All the others GPU
On Tue, Feb 22, 2022 at 4:08 PM wrote:
>
> From: Sunil Kumar Kori
>
> CNXK platforms support priority flow control(802.1qbb) to pause
> respective traffic per class on that link.
>
> Adds RoC interface to configure priority flow control on MAC
> block i.e. CGX on cn9k and RPM on cn10k.
>
> Signed
On Wed, Feb 23, 2022 at 3:20 PM Rakesh Kudurumalla
wrote:
>
> resolve compilation error caused due to gcc 12 version
> error: storing the address of local variable message in *error.message
>
> Fixes: 26b034f78ca7 ("net/cnxk: support to validate meter policy")
>
> Cc: sta...@dpdk.org
>
Reported-B
From: wenxuan wu
Apply our reverse iterator macro into testpmd to
avoid testpmd startups with pf and vfs quit without
error heap-free-before-use
wenxuan wu (2):
lib/ethdev: add reverse macro to quit testpmd
app/testpmd:fix testpmd quit failure
app/test-pmd/testpmd.c | 4 ++--
lib/ethde
From: wenxuan wu
There is a heap-free-after-use bug when quit testpmd
with pf and vfs, stop and close ports in reverse order
is a more reasonable approach.
Cc: sta...@dpdk.org
Signed-off-by: wenxuan wu
---
lib/ethdev/rte_ethdev.h | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(
From: wenxuan wu
When testpmd start ed with 1 pf and 2 vfs, testpmd quited while vfs
were still alive would result in failure. Root cause is that pf had
been released already but vfs were still accessing by func
rte_eth_dev_info_get, which would result in heap-free-after-use error.
By quitting
On 2/23/2022 11:32 AM, wenxuanx...@intel.com wrote:
From: wenxuan wu
When testpmd start ed with 1 pf and 2 vfs, testpmd quited while vfs
were still alive would result in failure. Root cause is that pf had
been released already but vfs were still accessing by func
rte_eth_dev_info_get, which wou
On 2/23/2022 9:08 AM, Jerin Jacob wrote:
On Wed, Feb 23, 2022 at 12:01 AM wrote:
From: Jerin Jacob
Except for the cn*_rte_flow* files, none of the other
file has rte in the file name, remove the
rte to unify the file name across the directory.
Signed-off-by: Jerin Jacob
Acked-by: Kiran Kum
Hi,
this set contains new ENA features:
* New xstats.
* Reconfigurable link status event.
* Usage of the optimized memcpy on arm/arm64.
* Better MP support.
* Reconfigurable Tx completion timeout value using devarg.
Beside that, this patchset contains multiple fixes, minor improvements, new
DPDK
The linearization of the mbuf isn't common practice for the PMD, as it
can expose it's capabilities to the upper layer using
rte_eth_dev_info_get().
Moreover, the rte_eth_tx_prepare() function should also verify if the
number of segments inside the mbuf isn't too high.
Because of those 2 circumst
To make sure there is no outstanding mbuf in the reused Tx queue (due to
unproper cleanup, or some invalid logic on Tx path), the assertion was
added on the Tx path.
As it's being compiled out in the release version, it won't affect
the IO path performance.
Signed-off-by: Michal Krawczyk
Reviewe
The enumeration seems to be leftover from porting the Linux driver to
the DPDK. It was used nowhere and refers to the ethtool which is not
present in the DPDK.
Signed-off-by: Michal Krawczyk
Reviewed-by: Artur Rojek
Reviewed-by: Dawid Gorecki
Reviewed-by: Igor Chauskin
Reviewed-by: Shai Brande
Those variables are being set, but never read. As they seem to be
leftover from the old offloads API and don't have any purpose right
now, they are simply being removed.
Signed-off-by: Michal Krawczyk
Reviewed-by: Artur Rojek
Reviewed-by: Dawid Gorecki
Reviewed-by: Igor Chauskin
Reviewed-by: S
* Split 'bad_csum' Rx statistic into 'l3_csum_bad' and 'l4_csum_bad' to
be able to check which checksum was not calculated properly.
* Add l4_csum_good statistic, which shows how many times L4 Rx checksum
was properly offloaded.
Signed-off-by: Michal Krawczyk
Reviewed-by: Dawid Gorecki
Revie
ENA uses AENQ for notification about various events, like LSC, keep
alive etc. By default it was enabling all AENQ that were supported by
both the driver and the device. As a result the LSC was always processed
even if the application turned it off explicitly.
As the DPDK provides application with
Some user applications may not support PMD reset handling. If they will
support timer service it could cause a situation, when information
about the reset trigger is being showed every time the timer service is
being called.
Timer service is now being skipped if the reset was already triggered.
F
To increase likehood that current burst will fit in the HW rings,
perform Tx cleanup before pushing packets to the HW. It may increase
latency a bit for sparse bursts, but the Tx flow now should be more
smooth.
It's also common order in the Tx burst function for other PMDs.
Signed-off-by: Michal
As the default behavior for arm64 is to alias rte_memcpy as memcpy, ENA
cannot redefine memcpy as rte_memcpy as it would cause nested
declaration.
To make it possible to use optimized memcpy in the ena_com layer on Arm,
the driver now redefines memcpy when it is beneficial:
* For arm64 only when
From: Stanislaw Kardach
Due to how the ena_com compatibility layer is written, all AQ commands
triggering functions use stack to save results of AQ and then copy them
to user given function.
Therefore to keep the compatibility layer common, introduce ENA_PROXY
macro. It either calls the wrapped f
From: Stanislaw Kardach
Since statistic gathering is now proxied safely to primary process, it
can be enabled in secondary processes.
Signed-off-by: Stanislaw Kardach
Reviewed-by: Michal Krawczyk
Reviewed-by: Shai Brandes
---
drivers/net/ena/ena_ethdev.c | 3 ---
1 file changed, 3 deletions(
Originally, the ena_com memzone counter was shared by ports, which
caused the memzones to be harder to identify and could potentially
lead to race and because of that the counter had to be atomic.
This atomic counter was global variable and it couldn't work in the
multiprocess implementation.
The
From: Dawid Gorecki
ENA driver did not allow applications to call tx_cleanup. Freeing Tx
mbufs was always done by the driver and it was not possible to manually
request the driver to free mbufs.
Modify ena_tx_cleanup function to accept maximum number of packets to
free and return number of packe
ENA was only supporting retrieval of all the xstats name and wasn't
implementing the eth_xstats_get_names_by_id API.
As this API may be more efficient than retrieving all the names, it
tries to avoid excessive string copying.
Signed-off-by: Michal Krawczyk
Reviewed-by: Dawid Gorecki
Reviewed-by
From: Dawid Gorecki
When triggering the reset, no check was performed to see if the reset
was already triggered. This could result in original reset reason being
overwritten. Add ena_trigger_reset helper function, which checks if the
reset was triggered and only sets the reset reason if the reset
The default missing Tx completion timeout was set to 5 seconds.
In order to provide users with the interface to control this timeout
to adjust it with the application's watchdog, the device argument for
controlling this value was added.
The parameter is called 'miss_txc_to' and can be modified usi
Whenever Tx checksum offload is being used, the meta descriptor content
is taken into consideration. Setting DF field properly in the meta
descriptor may have huge impact on the performance both for the IPv4 and
IPv6 packets.
The requirements for the df field are as below:
* No offload used - valu
From: Dawid Gorecki
Add information about port id, queue id and req_id to error logs in
validate_tx_req_id.
Signed-off-by: Dawid Gorecki
Reviewed-by: Michal Krawczyk
Reviewed-by: Shai Brandes
---
drivers/net/ena/ena_ethdev.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff -
From: Dawid Gorecki
The ena_com_config_dev_mode() performs many calculations related to LLQ
and then performs an admin queue call to configure LLQ in the device.
All of the operations performed by ena_com_config_dev_mode() are
unnecessary if membar hasn't been found. Move the dev_mem_base check
Some HW may invalidly set checksum error bit for the valid L4 checksum.
To avoid drop of the packets in that situation, do not indicate bad
checksum for L4 Rx csum offloads. Instead, set it as unknown, so the
application will re-verify this value.
The statistics counters will still work as previou
This release contains multiple bug fixes and improvements, including
- Removal of the linearization function from the xmit Tx path. The
DPDK assumes checking for the mbuf segments number in the Tx prepare
function.
- Extra logs, statistics, checks...
- Cleanup of the unused variables
cn10k platform can support IP reassembly offload for upto 4 fragments
in the inline IPsec path.
The feature is enabled and tested as per unit test app[1].
The patchset is rebased over [2].
[1]: http://patchwork.dpdk.org/user/todo/dpdk/?series=21716
[2]: http://patchwork.dpdk.org/project/dpdk/list
From: Vidya Sagar Velumuri
When reassembly is enabled by application, set corresponding
flags in SA during creation.
Provide roc API to configure reassembly unit with active and zombie limits
and step size
Signed-off-by: Vidya Sagar Velumuri
---
drivers/common/cnxk/cnxk_security.c | 10 ++
From: Vidya Sagar Velumuri
Add capability and support for inbound reassembly
in cnxk driver.
Register the dynamic field for IPsec reassembly.
Attach the fragments using the dynamic field in case of incomplete
reassembly
Signed-off-by: Vidya Sagar Velumuri
Signed-off-by: Akhil Goyal
Signed-off-
On 2/22/2022 2:31 PM, Ciara Loftus wrote:
When an AF_XDP PMD is created without specifying the 'start_queue', the
default Rx queue associated with the socket will be Rx queue 0. A common
scenario encountered by users new to AF_XDP is that they create the
socket on queue 0 however their interface
On 2/23/2022 10:28 AM, Jiawen Wu wrote:
Remove 'DEBUGFUNC' due to too many invalid debug log prints, unify the
DEBUG level macros.
Fixes: 7dc117068a7c ("net/txgbe: support probe and remove")
Cc:sta...@dpdk.org
Signed-off-by: Jiawen Wu
---
drivers/net/txgbe/base/meson.build| 2 +-
drive
This adds cn10k specific rx xstats of bpf, cpt and
ipsecd counters.
Signed-off-by: Rahul Bhansali
---
drivers/common/cnxk/hw/nix.h | 1 +
drivers/common/cnxk/roc_nix_stats.c | 14 ++
drivers/common/cnxk/roc_nix_xstats.h | 20 +++-
3 files changed, 34 inserti
Hi,
> -Original Message-
> From: Slava Ovsiienko
> Sent: Tuesday, February 22, 2022 12:27 PM
> To: dev@dpdk.org
> Cc: Matan Azrad ; Raslan Darawsheh
>
> Subject: [PATCH v2 0/4] net/mlx5: support send scheduling for ConnectX-7
>
> Since the ConnectX-6DX the send scheduling capability is
Hi Fan,
Is it possible to just skip these for QAT? You can add a driver name check and
skip the specific cases. Later, when we add capability, then this check can be
replaced with capability check.
If we remove the vectors, then a new PMD having the same issues as you reported
may not declare
When trying to dequeue, an OP may fail due to insufficient space for the
OP output, the compressdev API defines out-of-space for OP status.
The driver can detect out-of-space errors and report them to the user.
Check if hw_error_syndrome specifies out-of-space and set the OP
status accordingly.
Al
PMD driver got merged during 22.03 merge window and number in map file
should reflect that.
Fixes: d0b8a4e19131 ("raw/cnxk_gpio: add GPIO driver skeleton")
Reported-by: Ferruh Yigit
Signed-off-by: Tomasz Duszynski
---
drivers/raw/cnxk_gpio/version.map | 2 +-
1 file changed, 1 insertion(+), 1
In cyclecount main_loop function, each iteration it tries to enqueue X ops,
in case Y
---
Acked-by: Matan Azrad ma...@nvidia.com
app/test-compress-perf/comp_perf_test_cyclecount.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/app/test-compress-perf/comp_perf_test_cyclecount.c
An array of the size of total operations needed for the de/compression is
reserved for ops while enqueueing, although only first burst_size entries
of the array are used.
Reduce the size of the array allocated.
Fixes: b68a82425da4 ("app/compress-perf: add performance measurement")
Cc: sta...@dpdk
Recently [1] all the drivers running over the mlx5 device started
sharing some objects.
This raised the need to create documentation for the common driver that
would contain information relevant to all drivers and in particular:
- Tutorial for configuration of the device.
- Instruction for using
Vectorized routines were removed in result of Tx datapath refactoring,
and devarg keys documentation was updated.
However, more updating should have been done. In environment variables
doc, there was explanation according to vectorized Tx which isn't
relevant anymore.
This patch removes this irre
Update links in both mlx4 and mlx5 doc.
Signed-off-by: Michael Baum
Reviewed-by: Raslan Darawsheh
Acked-by: Viacheslav Ovsiienko
---
doc/guides/nics/mlx4.rst | 4 ++--
doc/guides/nics/mlx5.rst | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/doc/guides/nics/mlx4.rst b
Update "BlueField 2" -> "BlueField-2" in mlx5 docs.
Signed-off-by: Michael Baum
Reviewed-by: Raslan Darawsheh
Acked-by: Viacheslav Ovsiienko
---
doc/guides/compressdevs/mlx5.rst | 6 +++---
doc/guides/cryptodevs/mlx5.rst | 6 +++---
doc/guides/nics/mlx5.rst | 2 +-
doc/guides/regexde
Adds new documentation for MLX5 common driver that contains:
- Its features list (doesn't exist for now).
- Its devargs description.
- Device configuration information and tutorial.
- Quick Start Guide for Mellanox OFED/EN.
Move into this doc all shared information from other MLX5 PMD docs and
The "tx_db_nc" devarg forces doorbell register mapping to non-cached
region eliminating the extra write memory barrier. This argument was
used in creating the UAR for Tx and thus affected its performance.
Recently [1] its use has been extended to all UAR creation in all mlx5
drivers, and now its n
23/02/2022 12:20, Ferruh Yigit:
> On 2/23/2022 10:42 AM, Morten Brørup wrote:
> > +Thomas, you may be interested in this discussion about applications using
> > an uint64_t bit mask to identify active lcores.
> >
> >> From: Ferruh Yigit [mailto:ferruh.yi...@intel.com]
> >> Sent: Wednesday, 23 Feb
On 2/23/2022 1:32 PM, Tomasz Duszynski wrote:
PMD driver got merged during 22.03 merge window and number in map file
should reflect that.
Fixes: d0b8a4e19131 ("raw/cnxk_gpio: add GPIO driver skeleton")
Reported-by: Ferruh Yigit
Signed-off-by: Tomasz Duszynski
Acked-by: Ferruh Yigit
On 2/23/2022 12:49 PM, Ferruh Yigit wrote:
On 2/23/2022 10:28 AM, Jiawen Wu wrote:
Remove 'DEBUGFUNC' due to too many invalid debug log prints, unify the
DEBUG level macros.
Fixes: 7dc117068a7c ("net/txgbe: support probe and remove")
Cc:sta...@dpdk.org
Signed-off-by: Jiawen Wu
---
drivers/ne
On 2/23/2022 10:28 AM, Jiawen Wu wrote:
Remove redundant debug logs and unify the log format.
v2:
- Fix compile error.
- Merge broken log message lines.
- Use 'RTE_ETHER_ADDR_PRT_FMT' for MAC format.
Jiawen Wu (2):
net/ngbe: fix debug log
net/txgbe: fix debug log
Series applied to dpdk
Hi Anoob,
Make sense. Will do ASAP.
Regards,
Fan
> -Original Message-
> From: Anoob Joseph
> Sent: Wednesday, February 23, 2022 1:29 PM
> To: Zhang, Roy Fan ; Kusztal, ArkadiuszX
>
> Cc: Akhil Goyal ; Umesh Kartha
> ; Ramkumar Balu ;
> dev@dpdk.org
> Subject: RE: [EXT] [PATCH] test/cry
Hi,
> -Original Message-
> From: Michael Baum
> Sent: Wednesday, February 23, 2022 3:48 PM
> To: dev@dpdk.org
> Cc: Matan Azrad ; Raslan Darawsheh
> ; Slava Ovsiienko
> Subject: [PATCH v2 0/5] Refactor mlx5 guides
>
> Recently [1] all the drivers running over the mlx5 device started sha
Hi Anoob/Fan,
We no more add driver specific checks in test app. Everything comes from
capabilities.
We may defer this patch to next release till we have something in capability or
some other way.
> Hi Anoob,
>
> Make sense. Will do ASAP.
>
> Regards,
> Fan
>
Hi,
> -Original Message-
> From: Shun Hao
> Sent: Tuesday, February 22, 2022 5:07 PM
> To: Matan Azrad ; Slava Ovsiienko
> ; or...@nvidia.com ; tho...@monjalon.net ;
> Xueming(Steven) Li
> Cc: dev@dpdk.org ; Raslan Darawsheh ;
> sta...@dpdk.org
> Subject: [PATCH v1] drivers: fix incorrec
Hi Akhil,
Currently these tests will fail on QAT - is it possible to merge this patch so
that QAT won't fail and we can add them back after the capability check is
added in the next release?
Regards,
Fan
> -Original Message-
> From: Akhil Goyal
> Sent: Wednesday, February 23, 2022 2:3
1 - 100 of 187 matches
Mail list logo
