(uint16_t * uint16_t) promoted to uint64_t has a sign extension
problem reported by Coverity. Cast one arg to uint64_t first
to eliminate the sign extension.
Coverity issue: 426422
Coverity issue: 426427
Fixes: 2c1662bb53ca ("crypto/ionic: add adminq command")
Signed-off-by: An
rypto/ionic: support sessions")
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto_main.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/crypto/ionic/ionic_crypto_main.c
b/drivers/crypto/ionic/ionic_crypto_main.c
index d4810e3617..9960dc3a6d 10
;)
CC: sta...@dpdk.org
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_rxtx.c | 16 +---
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/drivers/net/ionic/ionic_rxtx.c b/drivers/net/ionic/ionic_rxtx.c
index 923f517661..339b20f113 100644
--- a/drivers/net/ionic/ionic_rx
If no progress has been made within the timeout, post a dummy operation
using session 0. This will restart the queue in the rare case that a
doorbell is lost inside the device.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 16
drivers/crypto/ionic
This defines the stats handlers and exposes them to the stack.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 8
drivers/crypto/ionic/ionic_crypto_main.c | 48
drivers/crypto/ionic/ionic_crypto_ops.c | 33 +++-
3 files
This defines the main crypto operation enqueue and dequeue handlers.
Signed-off-by: Andrew Boyer
---
doc/guides/cryptodevs/features/ionic.ini | 8 +
doc/guides/cryptodevs/ionic.rst | 12 +
drivers/crypto/ionic/ionic_crypto.h | 17 ++
drivers/crypto/ionic/ionic_crypto_caps.c
This defines the session object and related commands.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 33 ++
drivers/crypto/ionic/ionic_crypto_main.c | 143 +++
drivers/crypto/ionic/ionic_crypto_ops.c | 123 +++
3 files
This defines the device (register-based) commands. They are used for
device identification, setup, and teardown.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 24 ++
drivers/crypto/ionic/ionic_crypto_cmds.c | 348 +++
drivers/crypto/ionic
This defines the adminq used for control path commands. The adminq is
faster and more flexible than the device command interface.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 115 +
drivers/crypto/ionic/ionic_crypto_cmds.c | 302
This exposes the supported capabilities to the stack.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 3 ++
drivers/crypto/ionic/ionic_crypto_caps.c | 25 +
drivers/crypto/ionic/ionic_crypto_main.c | 8 +++
drivers/crypto/ionic/ionic_crypto_ops.c | 66
skeleton features list and guide.
Hook the new PMD up to the build.
Update MAINTAINERS.
Update release notes.
Signed-off-by: Andrew Boyer
---
MAINTAINERS | 7 ++
doc/guides/cryptodevs/features/ionic.ini | 32 +
doc/guides/cryptodevs/index.rst | 1 +
doc
These definitions are shared between the PMD and firmware.
Teach the device to read the FW version and check for liveness.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 53 ++
drivers/crypto/ionic/ionic_crypto_if.h | 1021 ++
drivers/crypto
- Do some very minor whitespace touchups
V3:
- Address review comments on logging and documentation
Andrew Boyer (9):
crypto/ionic: introduce AMD Pensando driver
crypto/ionic: add the firmware interface definition file
crypto/ionic: add device commands
crypto/ionic: add adminq command
If no progress has been made within the timeout, post a dummy operation
using session 0. This will restart the queue in the rare case that a
doorbell is lost inside the device.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 16
drivers/crypto/ionic
This defines the session object and related commands.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 33 ++
drivers/crypto/ionic/ionic_crypto_main.c | 143 +++
drivers/crypto/ionic/ionic_crypto_ops.c | 123 +++
3 files
This defines the stats handlers and exposes them to the stack.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 8
drivers/crypto/ionic/ionic_crypto_main.c | 48
drivers/crypto/ionic/ionic_crypto_ops.c | 33 +++-
3 files
This defines the main crypto operation enqueue and dequeue handlers.
Signed-off-by: Andrew Boyer
---
doc/guides/cryptodevs/features/ionic.ini | 8 +
doc/guides/cryptodevs/ionic.rst | 11 +
drivers/crypto/ionic/ionic_crypto.h | 17 ++
drivers/crypto/ionic/ionic_crypto_caps.c
This exposes the supported capabilities to the stack.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 3 ++
drivers/crypto/ionic/ionic_crypto_caps.c | 25 +
drivers/crypto/ionic/ionic_crypto_main.c | 8 +++
drivers/crypto/ionic/ionic_crypto_ops.c | 66
This defines the adminq used for control path commands. The adminq is
faster and more flexible than the device command interface.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 115 +
drivers/crypto/ionic/ionic_crypto_cmds.c | 302
This defines the device (register-based) commands. They are used for
device identification, setup, and teardown.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 24 ++
drivers/crypto/ionic/ionic_crypto_cmds.c | 348 +++
drivers/crypto/ionic
These definitions are shared between the PMD and firmware.
Teach the device to read the FW version and check for liveness.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 53 ++
drivers/crypto/ionic/ionic_crypto_if.h | 1021 ++
drivers/crypto
skeleton features list and guide.
Hook the new PMD up to the build.
Update MAINTAINERS.
Signed-off-by: Andrew Boyer
---
MAINTAINERS | 7 ++
doc/guides/cryptodevs/features/ionic.ini | 32 +
doc/guides/cryptodevs/index.rst | 1 +
doc/guides/cryptodevs
- Do some very minor whitespace touchups
Andrew Boyer (9):
crypto/ionic: introduce AMD Pensando ionic crypto driver
crypto/ionic: add the firmware interface definition file
crypto/ionic: add device commands
crypto/ionic: add adminq command support
crypto/ionic: add capabilities and basic
Switch to the new RTE_LOG_LINE_PREFIX logging macro. While here, fix up
some trailing-newline issues reported by the new macro.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev_pci.c | 7 +++
drivers/net/ionic/ionic_lif.c | 8
drivers/net/ionic/ionic_logs.h| 14
This defines the main datapath and reports the device
capabilities to the stack.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 62 +++
drivers/crypto/ionic/ionic_crypto_caps.c | 55 ++
drivers/crypto/ionic/ionic_crypto_main.c | 417 +++-
drivers/crypto
Add a features list and guide for the ionic crypto PMD.
Hook the new PMD up to the build.
Signed-off-by: Andrew Boyer
---
MAINTAINERS | 7 +
doc/guides/cryptodevs/features/ionic.ini | 40
doc/guides/cryptodevs/index.rst | 1
This defines the main device object routines and the vdev
support code. The vdev code uses the common library.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 89
drivers/crypto/ionic/ionic_crypto_main.c | 538 +++
drivers/crypto/ionic
This adds support for cryptodevs to the common ionic library.
Signed-off-by: Andrew Boyer
---
drivers/common/ionic/ionic_common.h | 2 ++
drivers/common/ionic/ionic_common_uio.c | 48 -
drivers/common/ionic/version.map| 1 +
3 files changed, 50 insertions
This defines the handlers used for device (register-based) and
admin (adminq-based) commands.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 210
drivers/crypto/ionic/ionic_crypto_cmds.c | 651 +++
drivers/crypto/ionic/ionic_crypto_main.c
Introduce a new crypto PMD for AMD Pensando hardware accelerators. It
allows applications running directly on the AMD Pensando DSC to offload
cryptographic operations to hardware cryptographic blocks.
This commit adds the firmware interface definition file.
Signed-off-by: Andrew Boyer
This patchset introduces a new crypto PMD for AMD Pensando hardware
accelerators. It allows applications running directly on the AMD Pensando
DSC to offload cryptographic operations to hardware cryptographic blocks.
Andrew Boyer (6):
crypto/ionic: introduce AMD Pensando ionic crypto driver
Add a features list and (basic) guide for the ionic crypto PMD.
Signed-off-by: Andrew Boyer
---
MAINTAINERS | 7 +
doc/guides/cryptodevs/features/ionic.ini | 40
doc/guides/cryptodevs/ionic.rst | 39 +++
3
This defines the main datapath and reports the device
capabilities to the stack.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 62 +++
drivers/crypto/ionic/ionic_crypto_caps.c | 55 ++
drivers/crypto/ionic/ionic_crypto_main.c | 417 +++-
drivers/crypto
This defines the handlers used for device (register-based) and
admin (adminq-based) commands.
Signed-off-by: Andrew Boyer
---
drivers/crypto/ionic/ionic_crypto.h | 209
drivers/crypto/ionic/ionic_crypto_cmds.c | 651 +++
drivers/crypto/ionic/ionic_crypto_main.c
This defines the main device object routines and the vdev
support code. The vdev code uses the common library.
Signed-off-by: Andrew Boyer
---
drivers/common/ionic/ionic_common.h | 2 +
drivers/common/ionic/ionic_common_uio.c | 48 +-
drivers/common/ionic/version.map | 1
Introduce a new crypto PMD for AMD Pensando hardware accelerators. It
allows applications running directly on the AMD Pensando DSC to offload
cryptographic operations to hardware cryptographic blocks.
This commit adds the firmware interface definition file.
Signed-off-by: Andrew Boyer
This patchset introduces a new crypto PMD for AMD Pensando hardware
accelerators. It allows applications running directly on the AMD Pensando
DSC to offload cryptographic operations to hardware cryptographic blocks.
Depends-on: series-31157 ("net/ionic, common/ionic: add vdev support"
Very minor improvements.
Fixes: 2717246ecd7d ("cryptodev: replace mbuf scatter gather flag")
Cc: pablo.de.lara.gua...@intel.com
Cc: sta...@dpdk.org
Signed-off-by: Andrew Boyer
---
doc/guides/cryptodevs/overview.rst | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff -
, use an ARM 'dsb st' barrier. The normal
barrier inside rte_write64() is insufficient on these devices due to
a chip errata.
Signed-off-by: Andrew Boyer
Signed-off-by: Neel Patel
Signed-off-by: R Mohamed Shah
Signed-off-by: Alfredo Cardigliano
---
config/arm/arm64_capri_linu
These barriers are duplicated by the barriers inside
rte_write64(). Remove them to improve performance.
Signed-off-by: Neel Patel
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_main.c| 1 -
drivers/net/ionic/ionic_rxtx_sg.c | 1 -
drivers/net/ionic/ionic_rxtx_simple.c | 1
Move definitions that will be shared by net/ionic and crypto/ionic.
Add the code used for discovering UIO vdevs.
Signed-off-by: Andrew Boyer
---
MAINTAINERS | 1 +
drivers/common/ionic/ionic_common.h | 41 +++
drivers/common/ionic/ionic_common_uio.c
driver.
V2:
- Redesign vdev device scan as suggested by review.
- Re-sort entries in config/arm/meson.build as suggested by review.
Andrew Boyer (3):
common/ionic: create common code library for ionic
net/ionic: remove duplicate barriers
net/ionic: add vdev support for embedded applications
, use an ARM 'dsb st' barrier. The normal
barrier inside rte_write64() is insufficient on these devices due to
a chip errata.
Signed-off-by: Andrew Boyer
Signed-off-by: Neel Patel
Signed-off-by: R Mohamed Shah
Signed-off-by: Alfredo Cardigliano
---
config/arm/arm64_capri_lin
These barriers are duplicated by the barriers inside
rte_write64(). Remove them to improve performance.
Signed-off-by: Neel Patel
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_main.c| 1 -
drivers/net/ionic/ionic_rxtx_sg.c | 1 -
drivers/net/ionic/ionic_rxtx_simple.c | 1
Move definitions that will be shared by net/ionic and crypto/ionic.
Add the code used for discovering UIO vdevs.
Signed-off-by: Andrew Boyer
---
MAINTAINERS | 1 +
drivers/common/ionic/ionic_common.h | 41 +++
drivers/common/ionic/ionic_common_uio.c
driver.
Andrew Boyer (3):
common/ionic: create common code library for ionic
net/ionic: remove duplicate barriers
net/ionic: add vdev support for embedded applications
MAINTAINERS | 1 +
config/arm/arm64_capri_linux_gcc| 16 ++
config/arm
Use rte_mempool_virt2iova(), which uses arithmetic based on the mempool
state, rather than rte_mem_virt2iova(), which uses syscalls to look at
the proc filesystem. This speeds up pool create by more than 90%.
Signed-off-by: Andrew Boyer
---
lib/cryptodev/rte_cryptodev.c | 2 +-
1 file changed
Split the queue_start operation into first-half and second-half helpers.
This allows us to batch up the queue commands during dev_start(), reducing
the outage window when restarting the process by about 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 136
process by about 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic.h | 3 ++
drivers/net/ionic/ionic_lif.c | 68 ++---
drivers/net/ionic/ionic_lif.h | 12 --
drivers/net/ionic/ionic_main.c | 17 +++-
drivers/net/ionic/ionic_rxtx.c | 78
Use a single device reset command to speed up dev_close(). The LIF stop
and port reset commands are not needed.
This reduces the outage window when restarting the process by about 2ms
plus another 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 3 ---
drivers
rt RSS")
Cc: cardigli...@ntop.org
Cc: sta...@dpdk.org
Signed-off-by: Akshay Dorwat
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ionic_eth
From: Brad Larson
Assign 32 counters for flexible firmware events. These can be used as
per-port or per-queue counters in certain firmware configurations.
They are displayed as fw_flex_eventX in xstats.
Signed-off-by: Andrew Boyer
Signed-off-by: Brad Larson
---
drivers/net/ionic
From: Neel Patel
They can be batched together this way, reducing the number
of PCIe transactions. This improves transmit PPS by up to 50% in
some configurations.
Signed-off-by: Andrew Boyer
Signed-off-by: Neel Patel
---
drivers/net/ionic/ionic_dev.c | 9 +++--
drivers/net/ionic
("net/ionic: complete release on close")
Cc: sta...@dpdk.org
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ioni
From: Vamsi Krishna Atluri
The hardware supports these speeds, so we should report them
correctly.
Signed-off-by: Andrew Boyer
Signed-off-by: Vamsi Krishna Atluri
---
drivers/net/ionic/ionic_ethdev.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/net/ionic/ionic_ethdev.c b
From: Neel Patel
This memory may be changed by the hardware, so the volatile
keyword is required for correctness.
Fixes: e86a6fcc7cf3 ("net/ionic: add optimized non-scattered Rx/Tx")
cc: sta...@dpdk.org
Signed-off-by: Andrew Boyer
Signed-off-by: Neel Patel
---
drivers/net/ionic/io
Use uint8_t instead of u_char. This simplifies the code.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev_pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ionic/ionic_dev_pci.c
b/drivers/net/ionic/ionic_dev_pci.c
index 5e74a6da71..cbaac2c5bc
Some configurations want to use values this high internally.
Allow them to do so without modifying the code.
Signed-off-by: Andrew Boyer
Signed-off-by: Bhuvan Mital
---
drivers/net/ionic/ionic_dev.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ionic
The receive side will enable scatter-gather if required based on the
mbuf size. If the client already enabled it in the config, it does
not need to be enabled again. This reduces log output.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 3 ++-
1 file changed, 2 insertions
When completion coalescing is turned on in the FW, there will be
fewer CQE than Tx packets. Expose the stat through debug logging.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.h | 1 +
drivers/net/ionic/ionic_rxtx.c| 3 +++
drivers/net/ionic/ionic_rxtx_sg.c
RSS query routine
Andrew Boyer (8):
net/ionic: add stat for completion queue entries processed
net/ionic: increase max supported MTU to 9750 bytes
net/ionic: don't auto-enable Rx scatter-gather a second time
net/ionic: replace non-standard type in structure definition
net/ionic: fix d
Split the queue_start operation into first-half and second-half helpers.
This allows us to batch up the queue commands during dev_start(), reducing
the outage window when restarting the process by about 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 136
process by about 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic.h | 3 ++
drivers/net/ionic/ionic_lif.c | 68 ++---
drivers/net/ionic/ionic_lif.h | 12 --
drivers/net/ionic/ionic_main.c | 17 +++-
drivers/net/ionic/ionic_rxtx.c | 78
("net/ionic: complete release on close")
Cc: sta...@dpdk.org
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ioni
Use a single device reset command to speed up dev_close(). The LIF stop
and port reset commands are not needed.
This reduces the outage window when restarting the process by about 2ms
plus another 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 3 ---
drivers
From: Brad Larson
Assign 32 counters for flexible firmware events. These can be used as
per-port or per-queue counters in certain firmware configurations.
They are displayed as fw_flex_eventX in xstats.
Signed-off-by: Andrew Boyer
Signed-off-by: Brad Larson
---
drivers/net/ionic
From: Vamsi Krishna Atluri
The hardware supports these speeds, so we should report them
correctly.
Signed-off-by: Andrew Boyer
Signed-off-by: Vamsi Krishna Atluri
---
drivers/net/ionic/ionic_ethdev.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/net/ionic/ionic_ethdev.c b
rt RSS")
Cc: cardigli...@ntop.org
Cc: sta...@dpdk.org
Signed-off-by: Akshay Dorwat
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ionic_eth
From: Neel Patel
They can be batched together this way, reducing the number
of PCIe transactions. This improves transmit PPS by up to 50% in
some configurations.
Signed-off-by: Andrew Boyer
Signed-off-by: Neel Patel
---
drivers/net/ionic/ionic_dev.c | 9 +++--
drivers/net/ionic
Use uint8_t instead of u_char. This simplifies the code.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev_pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ionic/ionic_dev_pci.c
b/drivers/net/ionic/ionic_dev_pci.c
index 5e74a6da71..cbaac2c5bc
From: Neel Patel
This memory may be changed by the hardware, so the volatile
keyword is required for correctness.
Fixes: e86a6fcc7cf3 ("net/ionic: add optimized non-scattered Rx/Tx")
cc: sta...@dpdk.org
Signed-off-by: Andrew Boyer
Signed-off-by: Neel Patel
---
drivers/net/ionic/io
The receive side will enable scatter-gather if required based on the
mbuf size. If the client already enabled it in the config, it does
not need to be enabled again. This reduces log output.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 3 ++-
1 file changed, 2 insertions
Some configurations want to use values this high internally.
Allow them to do so without modifying the code.
Signed-off-by: Andrew Boyer
Signed-off-by: Bhuvan Mital
---
drivers/net/ionic/ionic_dev.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ionic
When completion coalescing is turned on in the FW, there will be
fewer CQE than Tx packets. Expose the stat through debug logging.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.h | 1 +
drivers/net/ionic/ionic_rxtx.c| 3 +++
drivers/net/ionic/ionic_rxtx_sg.c
This patchset provides miscellaneous fixes and improvements for
the net/ionic driver used by AMD Pensando devices.
V2:
- Update device stop and device start patches to use compound literals
as suggested by review.
Akshay Dorwat (1):
net/ionic: fix RSS query routine
Andrew Boyer (8):
net
Split the queue_start operation into first-half and second-half helpers.
This allows us to batch up the queue commands during dev_start(), reducing
the outage window when restarting the process by about 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 178
process by about 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic.h | 3 ++
drivers/net/ionic/ionic_lif.c | 81 --
drivers/net/ionic/ionic_lif.h | 12 +++--
drivers/net/ionic/ionic_main.c | 17 ++-
drivers/net/ionic/ionic_rxtx.c | 78
Use a single device reset command to speed up dev_close(). The LIF stop
and port reset commands are not needed.
This reduces the outage window when restarting the process by about 2ms
plus another 1ms per queue.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 3 ---
drivers
("net/ionic: complete release on close")
Cc: sta...@dpdk.org
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ioni
From: Neel Patel
They can be batched together this way, reducing the number
of PCIe transactions. This improves transmit PPS by up to 50% in
some configurations.
Signed-off-by: Andrew Boyer
Signed-off-by: Neel Patel
---
drivers/net/ionic/ionic_dev.c | 9 +++--
drivers/net/ionic
From: Brad Larson
Assign 32 counters for flexible firmware events. These can be used as
per-port or per-queue counters in certain firmware configurations.
They are displayed as fw_flex_eventX in xstats.
Signed-off-by: Andrew Boyer
Signed-off-by: Brad Larson
---
drivers/net/ionic
rt RSS")
Cc: cardigli...@ntop.org
Cc: sta...@dpdk.org
Signed-off-by: Akshay Dorwat
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ionic_eth
From: Vamsi Krishna Atluri
The hardware supports these speeds, so we should report them
correctly.
Signed-off-by: Andrew Boyer
Signed-off-by: Vamsi Krishna Atluri
---
drivers/net/ionic/ionic_ethdev.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/net/ionic/ionic_ethdev.c b
From: Neel Patel
This memory may be changed by the hardware, so the volatile
keyword is required for correctness.
Fixes: e86a6fcc7cf3 ("net/ionic: add optimized non-scattered Rx/Tx")
cc: sta...@dpdk.org
Signed-off-by: Andrew Boyer
Signed-off-by: Neel Patel
---
drivers/net/ionic/io
Use uint8_t instead of u_char. This simplifies the code.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev_pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ionic/ionic_dev_pci.c
b/drivers/net/ionic/ionic_dev_pci.c
index 5e74a6da71..cbaac2c5bc
The receive side will enable scatter-gather if required based on the
mbuf size. If the client already enabled it in the config, it does
not need to be enabled again. This reduces log output.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 3 ++-
1 file changed, 2 insertions
Some configurations want to use values this high internally.
Allow them to do so without modifying the code.
Signed-off-by: Andrew Boyer
Signed-off-by: Bhuvan Mital
---
drivers/net/ionic/ionic_dev.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ionic
When completion coalescing is turned on in the FW, there will be
fewer CQE than Tx packets. Expose the stat through debug logging.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.h | 1 +
drivers/net/ionic/ionic_rxtx.c| 3 +++
drivers/net/ionic/ionic_rxtx_sg.c
This patchset provides miscellaneous fixes and improvements for
the net/ionic driver used by AMD Pensando devices.
Akshay Dorwat (1):
net/ionic: fix RSS query routine
Andrew Boyer (8):
net/ionic: add stat for completion queue entries processed
net/ionic: increase max supported MTU to 9750
Cache the most recent VA -> PA mapping found so that we can skip
most of the system calls. With 4K pages this reduces pool create
time by about 90%.
Signed-off-by: Andrew Boyer
---
lib/cryptodev/rte_crypto.h| 5 +
lib/cryptodev/rte_cryptodev.c | 23 ++-
2 fi
t;)
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_rxtx_sg.c | 2 +-
drivers/net/ionic/ionic_rxtx_simple.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ionic/ionic_rxtx_sg.c
b/drivers/net/ionic/ionic_rxtx_sg.c
index 2752ba2acd..ab8e56e91c 10064
(uint16_t * uint16_t) promoted to uint64_t has a sign extension
problem reported by Coverity. Cast one arg to uint64_t first
to eliminate the sign extension.
Coverity issue: 381617
Coverity issue: 381618
Fixes: 7b20fc2f3c06 ("net/ionic: overhaul Rx for performance")
Signed-off-by: An
These patches will make no functional difference, but should
eliminate four coverity issues.
Alternatively, we could just mark them as False Positive or
Ignored in coverity.
Signed-off-by: Andrew Boyer
Andrew Boyer (2):
net/ionic: fix up minor coverity issue in queue allocation
net/ionic
doorbell is missed by the FW.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev.h | 4 +++
drivers/net/ionic/ionic_lif.h | 3 ++
drivers/net/ionic/ionic_main.c| 22
drivers/net/ionic/ionic_rxtx.c| 1 +
drivers/net/ionic/ionic_rxtx_sg.c
In some configurations, the FW may return EAGAIN if it is not able
to respond to commands immediately. Retry the init commands in this
case to prevent errors from reaching the client.
Fix up some return-code stuff while here, for clarity.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic
This improves readability.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 26 --
1 file changed, 12 insertions(+), 14 deletions(-)
diff --git a/drivers/net/ionic/ionic_lif.c b/drivers/net/ionic/ionic_lif.c
index 383fff9ffd..1b5932db12 100644
--- a
the Rx side if the provided mbufs are
too small to hold the maximum possible frame.
To enable S/G in testpmd, add these args:
--rx-offloads=0x2000 --tx-offloads=0x8000
Signed-off-by: Andrew Boyer
Signed-off-by: R Mohamed Shah
---
doc/guides/rel_notes/release_22_11.rst | 1 +
drivers/net
Some clients have opinions about how often to flush the
transmit ring.
The default value is the number of Tx descriptors minus the
default Tx burst size.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev.h| 1 +
drivers/net/ionic/ionic_ethdev.c | 4 ++--
drivers/net/ionic
This will avoid memory access penalties on NUMA systems.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c| 5 +++--
drivers/net/ionic/ionic_lif.c | 13 -
drivers/net/ionic/ionic_rx_filter.c | 3 +--
3 files changed, 12 insertions(+), 9 deletions(-)
diff
Use rte_calloc() where appropriate.
This makes the code clearer.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_ethdev.c | 6 +++---
drivers/net/ionic/ionic_lif.c| 12 ++--
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/net/ionic/ionic_ethdev.c b
1 - 100 of 366 matches
Mail list logo
