> -Original Message-
> From: Su, Simei
> Sent: Friday, September 15, 2023 10:17 AM
> To: Wu, Jingjing ; Xing, Beilei
> ;
> Zhang, Qi Z
> Cc: dev@dpdk.org; Liu, Mingxia ; Qiao, Wenjing
> ; Su, Simei
> Subject: [PATCH v3 00/17] update idpf base code
>
> This patch set updates idpf bas
Thanks
Acked-by: Hemant Agrawal
> -Original Message-
> From: David Marchand
> Sent: Wednesday, September 13, 2023 7:29 PM
> To: dev@dpdk.org
> Cc: maxime.coque...@redhat.com; sta...@dpdk.org; Nicolas Chautru
> ; Hemant Agrawal ;
> Akhil Goyal
> Subject: [PATCH] app/bbdev: fix link with
Originally, the iavf PMD does not perform special actions when it
receives a PF-to-VF reset event, resulting in vf being offline and
unavailable.
This patch enables vf auto-reset by setting 'watchdog_period' devargs
to true. The iavf PMD will perform an automatic reset to bring the vf
back online
From: Sinan Kaya
Cleanup event registry during shutdown to prevent memory
leaks.
Signed-off-by: Sinan Kaya
---
app/test-pmd/testpmd.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
index c6ad9b18bf..1de7b7cd51 10
Acked-by: Junfeng Guo
Regards,
Junfeng
> -Original Message-
> From: Rushil Gupta
> Sent: Friday, September 15, 2023 04:52
> To: Zhang, Qi Z ; ferruh.yi...@amd.com
> Cc: Guo, Junfeng ; dev@dpdk.org; Rushil Gupta
> ; Joshua Washington
> Subject: [PATCH] net/gve: Refill all descriptors th
https://bugs.dpdk.org/show_bug.cgi?id=1288
Bug ID: 1288
Summary: af_packet do not support multi-process
Product: DPDK
Version: 21.11
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Pri
Remove the unused TX descriptor types and mark them as reserved.
Signed-off-by: Pavan Kumar Linga
Signed-off-by: Simei Su
---
drivers/common/idpf/base/idpf_lan_txrx.h | 132 ++-
1 file changed, 10 insertions(+), 122 deletions(-)
diff --git a/drivers/common/idpf/base/idpf_la
Update README
Signed-off-by: Simei Su
---
drivers/common/idpf/base/README | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/common/idpf/base/README b/drivers/common/idpf/base/README
index 693049c057..ff26f736ec 100644
--- a/drivers/common/idpf/base/README
+++ b/drivers/
In idpf_flex_tx_desc structure, instead of naming the union with 'flex',
use no name union as the union name is not really necessary there. This
reduces the level of indirection in the hotpath.
Signed-off-by: Pavan Kumar Linga
Signed-off-by: Simei Su
---
drivers/common/idpf/base/idpf_lan_txrx.h
Instead of splitting the function name and function type into
multiple lines, use then in a single line.
Signed-off-by: Pavan Kumar Linga
Signed-off-by: Simei Su
---
drivers/common/idpf/base/idpf_controlq.c | 5 ++---
drivers/common/idpf/base/idpf_controlq_setup.c | 5 ++---
2 files chang
Instead of using a custom defined macro for generating a mask,
use the standard GENMASK macro.
Signed-off-by: Pavan Kumar Linga
Signed-off-by: Simei Su
---
drivers/common/idpf/base/idpf_lan_pf_regs.h | 26 ++---
drivers/common/idpf/base/idpf_lan_txrx.h| 116 +---
drivers/co
Refine the macros and definitions by using 'tab' spaces and new
lines wherever necessary. Also refine the comment in
'idpf_ctlq_setup_regs' and remove the TODO comment in idpf_rss_hash
enum as it doesn't make any sense.
Signed-off-by: Pavan Kumar Linga
Signed-off-by: Simei Su
---
drivers/common
Conditional check is not necessary to strip and get status bits
from the descriptor.
Signed-off-by: Pavan Kumar Linga
Signed-off-by: Simei Su
---
drivers/common/idpf/base/idpf_controlq.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/common/idpf/base/idpf_con
Instead of updating directly to 'cq_out' double pointer, use a
local pointer and update only when we return success.
Signed-off-by: Pavan Kumar Linga
Signed-off-by: Simei Su
---
drivers/common/idpf/base/idpf_controlq.c | 43 +---
1 file changed, 23 insertions(+), 20 deletion
As idpf_ctlq_deinit always returns success, make it 'void' instead
of returning only success. This also changes the return type for
idpf_deinit_hw as 'void'.
Based on the upstream comments, explicit __le16 typecasting is not
necessary as CPU_TO_LE16 is already being used.
Signed-off-by: Pavan Kum
Customer has a requirement to use the legacy fixed size, single chunk
structure for ADI creation - one chunk for queue and one chunk for vector.
This is described in detail in customer case
https://issuetracker.google.com/issues/270157802.
On the other hand, upstream code review patch has been pos
Add necessary check for payload and message buffer.
Signed-off-by: Julianx Grajkowski
Signed-off-by: Simei Su
---
.mailmap | 1 +
drivers/common/idpf/base/idpf_common.c | 6 --
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/.mailmap b/.mailmap
i
Instead of using something like a byte offset, we can add a union to the
struct here to enable direct addressing.
Signed-off-by: Alan Brady
Signed-off-by: Simei Su
---
.mailmap | 1 +
drivers/common/idpf/base/idpf_controlq_api.h | 5 +
2 files changed, 6
a) Add maximum ADI count in capabilities message
b) Add PF side ADI index to create_adi message
c) Define another constant to indicate 'Function active' state of ADI
Signed-off-by: Shailendra Bhatnagar
Signed-off-by: Simei Su
---
.mailmap | 1 +
drivers/common/idpf/b
Removing mailbox register offsets as the mapping to device register
offsets are different between CVL and MEV (they are swapped out)
individual drivers will define the offsets based on how registers
are hardware addressed. However the it will begin with VDEV_MBX_START
offset.
Signed-off-by: Madhu
Add miss completion tag to other capabilities list, to indicate support for
detecting a miss completion based on the upper bit of the completion tag.
Signed-off-by: Josh Hay
Signed-off-by: Simei Su
---
.mailmap | 1 +
drivers/common/idpf/base/virtchnl2.h | 4
2
Adding a few PTP capabilities to determine which PTP features are
enabled - legacy cross time, ptm, device clock control, PTP Tx
timestamp with direct registers access, PTP Tx timestamp using
virtchnl messages.
Creating structures and opcodes to support feautres introduced by
capabilities.
Signed
Add support to indicate physical port representor and query its statistics.
Signed-off-by: Zhenning Xiao
Signed-off-by: Jayaprakash Shanmugam
Signed-off-by: Simei Su
---
.mailmap | 2 +
drivers/common/idpf/base/virtchnl2.h | 80 +++-
2 files
This patch set updates idpf base code.
v3:
* Fix coding style issue.
* Modify unexpected error in the update version patch.
v2:
* Add two patches for share code update.
* Add version update.
* Fix coding style issue.
Simei Su (17):
common/idpf/base: enable support for physical port stats
com
Refill all consumed descriptors instead of refilling a set amount of
descriptors everytime. Also, lower the rx-free-thresh to refill more
aggressively. This yields in fewer packet drops for DQO queue format.
Signed-off-by: Rushil Gupta
Reviewed-by: Joshua Washington
---
drivers/net/gve/gve_ethd
Hi all,
As part of the recent rework of the build of our unit test binary
(dpdk-test), the individual autotests were updated in the source code to
specify the test suite they belong to. The replacement of the
registration macro was done automatically using the existing suite lists
in the meson.bui
Add DOCSIS security test cases to the cryptodev scheduler test suite.
Signed-off-by: David Coyle
Signed-off-by: Kevin O'Sullivan
---
app/test/test_cryptodev.c | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
i
Add support to the cryptodev scheduler PMD for the DOCSIS security
protocol. This includes adding the following to the scheduler:
- synchronization of worker's security capabilities
- retrieval of the scheduler's synchronized security capabilities
- retrieval of the security session size i.e. maxim
This patchset adds support to the cryptodev scheduler PMD and unit
tests for the DOCSIS security protocol. Support for other stateless
security protocols can be added in the future as required with relative
ease now that the framework is in place.
> -Original Message-
> From: Richardson, Bruce
> Sent: Thursday, September 14, 2023 4:25 PM
> To: dev@dpdk.org
> Cc: acon...@redhat.com; Richardson, Bruce ;
> Power, Ciara
> Subject: [PATCH] app/test: add telemetry tests to fast test suite
>
> The telemetry data tests, and the telemet
The telemetry data tests, and the telemetry json tests execute very
quickly, so can be added to the fast-test suite.
Signed-off-by: Bruce Richardson
---
app/test/test_telemetry_data.c | 2 +-
app/test/test_telemetry_json.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/ap
Some synchronous tests are not supported for cpu crypto and need to be
skipped. This commit adds in extra skips for these tests.
Fixes: 55ab4a8c4fb5 ("test/crypto: disable wireless cases for CPU crypto API")
Cc: pablo.de.lara.gua...@intel.com
Cc: sta...@dpdk.org
Acked-by: Ciara Power
Signed-off-
SNOW3G and ZUC algorithms require non-zero length IVs.
Fixes: c6c267a00a92 ("test/crypto: add mixed encypted-digest")
Cc: adamx.dybkow...@intel.com
Cc: sta...@dpdk.org
Acked-by: Ciara Power
Signed-off-by: Brian Dooley
--
v8:
Add cc stable
---
app/test/test_cryptodev_mixed_test_vectors.h | 8 ++
AESNI_MB PMD does not support Digest Encrypted. This patch adds a check and
support for this feature.
Acked-by: Ciara Power
Signed-off-by: Brian Dooley
---
v2:
Fixed CHECKPATCH warning
v3:
Add Digest encrypted support to docs
v4:
Add comments and small refactor
v5:
Fix checkpatch warnings
v6:
Ad
This series adds the Digest Encrypted feature to the AESNI_MB PMD.
It also fixes an issue where IV data in SNOW3G and ZUC algorithms
were incorrect and are required to be non-zero length.
v2:
Fixed CHECKPATCH warning
v3:
Add Digest encrypted support to docs
v4:
add comments and refactor
v5:
Fix ch
When called from automated tools, like meson test, it is often useful to
skip tests in a test suite, without having to alter the test build. To
do so, we add support for DPDK_TEST_SKIP environment variable, where one
can provide a comma-separated list of tests. When the test binary is
called to run
Refrain from including Linux-only pci_regs.h header.
Instead, prefer our own definitions from the pci library.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
---
devtools/checkpatches.sh | 8
1 file changed, 8 insertions(+)
diff --git a/devtools/chec
Define some Advanced Error Reporting constants and use them in existing
drivers.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
---
drivers/event/dlb2/pf/dlb2_main.c | 10 +++---
lib/pci/rte_pci.h | 5 +
2 files changed, 8 insertions(+), 7 deletions(-)
diff -
Define some PCI PRI extended feature constants and use them in existing
drivers.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
---
drivers/event/dlb2/pf/dlb2_main.c | 11 ---
lib/pci/rte_pci.h | 5 +
2 files changed, 9 insertions(+), 7 deletions(-)
diff
Define some PCI ACS extended feature constants and use them in existing
drivers.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
---
drivers/event/dlb2/pf/dlb2_main.c | 23 ---
lib/pci/rte_pci.h | 9 +
2 files changed, 17 insertions(+), 15 d
Define some PCI extended capability constants and use them in existing
drivers.
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
Signed-off-by: David Marchand
---
Changes since v2:
- fixed existing SRIOV comment,
---
drivers/event/dlb2/pf/dlb2_main.c | 7 ++-
lib/pci/rte_pci.h
Define some PCI Express constants and use them in existing drivers.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
---
drivers/event/dlb2/pf/dlb2_main.c | 40 ---
drivers/net/bnx2x/bnx2x.c | 16 ++---
drivers/net/bnx
Define some PCI Power Management constants and use them in existing
drivers.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
---
drivers/net/bnx2x/bnx2x.c | 17 +
drivers/net/bnx2x/bnx2x.h | 5 -
lib/pci/rte_pci.h | 6 ++
3 fil
Define some PCI BAR constants and use them in existing drivers.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
---
drivers/bus/pci/linux/pci_vfio.c | 7 +++
lib/pci/rte_pci.h| 4
2 files changed, 7 insertions(+), 4 deletions(-)
diff -
Define some PCI command constants and use them in existing drivers.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
---
drivers/bus/pci/linux/pci_vfio.c | 8
drivers/event/dlb2/pf/dlb2_main.c | 8 +++-
lib/pci/rte_pci.h | 6 --
Define some PCI MSIX constants and use them in existing drivers.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
---
drivers/bus/pci/linux/pci_init.h | 18 --
drivers/bus/pci/linux/pci_vfio.c | 7 ---
drivers/crypto/virtio/virtio_pci.c
Define some PCI capability constants and use them in existing drivers.
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
Signed-off-by: David Marchand
---
Changes since v2:
- adjusted virtio drivers after changes in previous patch,
---
drivers/bus/pci/linux/pci_vfio.c| 2 +-
drivers/cry
Introduce two helpers so that drivers stop reinventing the wheel when it
comes to finding capabilities in a device PCI configuration space.
Use it in existing drivers.
Note:
- base/ drivers code is left untouched, only some wrappers in cxgbe
are touched,
- bnx2x maintained a per device cache of
This is a preparatory step before using new helpers for finding PCI
capabilities.
In the code querying PCI capabilities for checking MSIX availability,
replace direct calls to VFIO fd with the existing helpers for reading
PCI configuration space: this requires setting VFIO fd in the PCI
device obje
Those functions are fine with a const on the device pointer.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
---
drivers/bus/pci/pci_common.c | 4 ++--
drivers/bus/pci/rte_bus_pci.h | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drive
Use existing API to cleanup duplicated code.
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Reviewed-by: Chenbo Xia
---
drivers/bus/pci/linux/pci_uio.c| 32 +--
drivers/bus/pci/linux/pci_vfio.c | 41 ++
drivers/net/hns3/hns3_ethde
Rather than rely on Linux headers to find some PCI(e) standard constants
or reinvent the same PCI capability helper, this series complements the
pci library and the pci bus driver.
PCI drivers can then use OS agnostic macros and helpers.
--
David Marchand
Changes since v2:
- fixed vendor capabi
Hello Chenbo,
On Thu, Sep 7, 2023 at 2:43 PM Xia, Chenbo wrote:
> > Introduce two helpers so that drivers stop reinventing the wheel when it
> > comes to finding capabilities in a device PCI configuration space.
> > Use it in existing drivers.
> >
> > Note:
> > - base/ drivers code is left untouc
On Thu, Sep 14, 2023 at 11:58:52AM +0100, Harry van Haaren wrote:
> This commit adds a unit test for an issue identified
> where ordered history-list entries are not correctly
> cleared when the returned event is of op RELEASE type.
>
> The result of the history-list bug is that a future event
> w
On Thu, Sep 14, 2023 at 11:58:51AM +0100, Harry van Haaren wrote:
> This commit changes the logic in the scheduler to always
> reset reorder-buffer (and QID/FID) entries when writing
> them. This avoids stale ROB/QID/FID data re-use, which
> previously caused ordering issues.
>
> Before this commi
> -Original Message-
> From: Su, Simei
> Sent: Thursday, September 14, 2023 2:37 PM
> To: Wu, Jingjing ; Xing, Beilei
> ;
> Zhang, Qi Z
> Cc: dev@dpdk.org; Wu, Wenjun1 ; Su, Simei
>
> Subject: [PATCH v5] common/idpf: refactor single queue Tx data path
>
> Currently, single queue Tx
This commit adds a unit test for an issue identified
where ordered history-list entries are not correctly
cleared when the returned event is of op RELEASE type.
The result of the history-list bug is that a future event
which re-uses that history-list slot, but has an op type
of FORWARD will incorr
This commit changes the logic in the scheduler to always
reset reorder-buffer (and QID/FID) entries when writing
them. This avoids stale ROB/QID/FID data re-use, which
previously caused ordering issues.
Before this commit, release events left the history-list
in an inconsistent state, and future e
On Thu, Sep 14, 2023 at 10:59 AM 杨佳昊 wrote:
>
> Thank you for your reply.
Please, let's keep this discussion on the ml.
Anybody can help and this thread may help others having issues with ASan.
Let's try to understand if something is wrong in your setup or if
there is a bug in the doc or example
When testing the file prefix handling, and the memory cleanup for the
various prefixes, we don't need to worry about PCI devices. Therefore
skip the device scan on startup. In my testing, this makes the test runs
more resilient.
Signed-off-by: Bruce Richardson
---
app/test/test_eal_flags.c | 20
When examining the IOL testing failures for patch series [1], I observed
that the failures reported were in the eal_flags_file_prefix unit test.
I was able to reproduce this on my system by passing an additional
"--on-pci" flag to the test run, since the log to the test has errors
about device avai
On Thu, Sep 14, 2023 at 11:10:17AM +0100, Van Haaren, Harry wrote:
> > -Original Message-
> > From: Richardson, Bruce
> > Sent: Thursday, September 14, 2023 10:38 AM
> > To: dev@dpdk.org
> > Cc: Van Haaren, Harry ; Richardson, Bruce
> > ; sta...@dpdk.org
> > Subject: [PATCH] event/sw: remo
> -Original Message-
> From: Richardson, Bruce
> Sent: Thursday, September 14, 2023 10:38 AM
> To: dev@dpdk.org
> Cc: Van Haaren, Harry ; Richardson, Bruce
> ; sta...@dpdk.org
> Subject: [PATCH] event/sw: remove obsolete comment
>
> Code that the comment refers to was previously removed,
Recheck-request: iol-unit-arm64-testing, iol-intel-Performance
On Fri, Sep 01, 2023 at 03:23:31PM +0100, Bruce Richardson wrote:
> When the user passes a list of desired libraries to build via the
> "enable_libs" option, the expectation is that those libraries should be
> part of the build. Howeve
On Sat, Sep 02, 2023 at 01:24:18PM -0400, Patrick Robb wrote:
>ARM Ampere server test fails on this patch are lab infra-failures (I
>did some updates on the server yesterday) and they can be ignored.
Can a retest be run on these now, to clear the errors in patchwork, or have
the issued bee
Code that the comment refers to was previously removed, but the
comment remained.
Fixes: dca926ca9faa ("event/sw: use dynamically-sized IQs")
Cc: sta...@dpdk.org
Signed-off-by: Bruce Richardson
---
drivers/event/sw/sw_evdev_scheduler.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/dr
Hello,
On Thu, Sep 14, 2023 at 9:51 AM 杨佳昊 wrote:
>
> Hello
> I am tring to run dpdk example with asan.
> I have compile dpdk and example as directed by link
> https://github.com/DPDK/dpdk/blob/v21.11/doc/guides/prog_guide/asan.rst
> But I always get result as follow:
> =
On 2023-09-13 22:56, Honnappa Nagarahalli wrote:
-Original Message-
From: Mattias Rönnblom
Sent: Wednesday, September 13, 2023 10:48 AM
To: Sevincer, Abdullah ; Stephen Hemminger
; tho...@monjalon.net
Cc: dev@dpdk.org; Tyler Retzlaff
Subject: Re: quick thread in DLB2
On 2023-09-11 1
Hello
I am tring to run dpdk example with asan.
I have compile dpdk and example as directed by link
https://github.com/DPDK/dpdk/blob/v21.11/doc/guides/prog_guide/asan.rst
But I always get result as follow:
=
==20078== ERROR: Address
From: Jiayu Hu
Update the email address for gro and gso libraries.
Signed-off-by: Jiayu Hu
---
MAINTAINERS | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 698608cdb2..8b00ae4711 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1481,12 +1481
Hi,
Could these patches get some review ?
Thanks,
Gowrishankar
> This patch series fixes SM2 algorithm implementation to support digest
> message as input along with plain message as today.
>
> Gowrishankar Muthukrishnan (6):
> crypto/openssl: include SM2 in asymmetric capabilities
> cryptod
71 matches
Mail list logo