net/octeon_ep: fix init failure handling

In the changeset 304ba46be396, a goto statement was
mistakenly added after a return statement. This patch
corrects the code by removing the erroneous return.

net/octeon_ep: fix init failure handling

In the changeset 304ba46be396, a goto statement was
mistakenly added after a return statement. This patch
corrects the code by removing the erroneous return.

Coverity issue: 445484
Fixes: 304ba46be396 ("net/octeon_ep: handle device removal on probe")

Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>

show more ...

net/octeon_ep: handle device removal on probe

Adds an event callback to catch any device removal
event occurred during driver probe. This callback helps
in terminating the execution if there is any

net/octeon_ep: handle device removal on probe

Adds an event callback to catch any device removal
event occurred during driver probe. This callback helps
in terminating the execution if there is any device removal
event during the driver probe.

Also moved global register configuration into dev_configure()
routine and also validates register reads for any invalid
return values from hardware during driver probe.

Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>

show more ...

net/octeon_ep: extend mailbox functionality

Patch extends mbox functionality to handle pf to VF mbox
messages and also updates current mbox version to v3.

As part of PF FLR notify event, event hand

net/octeon_ep: extend mailbox functionality

Patch extends mbox functionality to handle pf to VF mbox
messages and also updates current mbox version to v3.

As part of PF FLR notify event, event handler invokes
device removal event callback to tear down the driver.

Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>

show more ...

net/octeon_ep: avoid warning on uninitialized variable

In the next commit, a (unrelated) change will be done on the logging
macros in this driver.

After this change, compilation with O3 fails with

net/octeon_ep: avoid warning on uninitialized variable

In the next commit, a (unrelated) change will be done on the logging
macros in this driver.

After this change, compilation with O3 fails with the following warning:
../drivers/net/octeon_ep/otx_ep_ethdev.c: In function
‘otx_ep_dev_mtu_set’:
../drivers/net/octeon_ep/otx_ep_ethdev.c:200:12: error: ‘devinfo.max_mtu’
may be used uninitialized [-Werror=maybe-uninitialized]
200 | if (mtu > devinfo.max_mtu) {
| ^
../drivers/net/octeon_ep/otx_ep_ethdev.c:186:33: note: ‘devinfo.max_mtu’
was declared here
186 | struct rte_eth_dev_info devinfo;
| ^~~~~~~
cc1: all warnings being treated as errors

The devinfo object is passed through otx_ep_dev_info_get() which
initializes devinfo.max_mtu and returns 0.
This return code is checked at line 189 and so devinfo.max_mtu is
supposed to be initialized at this point.
This seems like a compiler optimisation bug, but we will have to live
with it.

Signed-off-by: David Marchand <david.marchand@redhat.com>

show more ...

drivers: remove redundant newline from logs

Fix places where two newline characters may be logged.

Cc: stable@dpdk.org

Signed-off-by: David Marchand <david.marchand@redhat.com>
Acked-by: Chengwen

drivers: remove redundant newline from logs

Fix places where two newline characters may be logged.

Cc: stable@dpdk.org

Signed-off-by: David Marchand <david.marchand@redhat.com>
Acked-by: Chengwen Feng <fengchengwen@huawei.com>

show more ...

net/octeon_ep: enable ISM access

Adds a devarg option to enable/disable ISM memory accesses
for reading packet count details. This option is disabled
by default, as ISM memory accesses effect throug

net/octeon_ep: enable ISM access

Adds a devarg option to enable/disable ISM memory accesses
for reading packet count details. This option is disabled
by default, as ISM memory accesses effect throughput of
bigger size packets.

Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>

show more ...

net/octeon_ep: add NEON Rx

Add Rx ARM NEON SIMD routine.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

net/octeon_ep: add AVX2 Rx

Optimize Rx routine to use AVX2 instructions when underlying
architecture supports it.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

net/octeon_ep: add SSE Rx

Optimize Rx routine to use SSE instructions.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

net/octeon_ep: add new fastpath routines

Adds new fastpath routines for cn10k & cn9k endpoint
devices and assigns the fastpath routines based on
the offload flags.

Patch also adds misc changes to i

net/octeon_ep: add new fastpath routines

Adds new fastpath routines for cn10k & cn9k endpoint
devices and assigns the fastpath routines based on
the offload flags.

Patch also adds misc changes to improve performance
and code-readability.

Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>

show more ...

net/octeon_ep: fix Rx and Tx queue state

The DPDK framework reports the queue state, which is stored in
dev->data->tx_queue_state and dev->data->rx_queue_state. The
state is maintained by the driver

net/octeon_ep: fix Rx and Tx queue state

The DPDK framework reports the queue state, which is stored in
dev->data->tx_queue_state and dev->data->rx_queue_state. The
state is maintained by the driver. Users may determine whether
a queue participates in packet forwarding based on the state.
Therefore, the driver needs to modify the queue state in time
according to the actual situation.

Fixes: 9ad9ff476cac ("ethdev: add queue state in queried queue information")
Cc: stable@dpdk.org

Signed-off-by: Jie Hai <haijie1@huawei.com>
Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>

show more ...

net/octeon_ep: support backward compatibility

Add backward compatibility support between VF
and PF mailbox messages.

Signed-off-by: Sathesh Edara <sedara@marvell.com>
Signed-off-by: Vamsi Attunuru

net/octeon_ep: support backward compatibility

Add backward compatibility support between VF
and PF mailbox messages.

Signed-off-by: Sathesh Edara <sedara@marvell.com>
Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>

show more ...

net/octeon_ep: set secondary process operations

Sets the dev ops and transmit/receive callbacks
for secondary process.

Signed-off-by: Sathesh Edara <sedara@marvell.com>

net/octeon_ep: support mailbox between VF and PF

Adds the mailbox communication between VF and
PF and supports the following mailbox messages.
- Get and set MAC address
- Get link information
- Get

net/octeon_ep: support mailbox between VF and PF

Adds the mailbox communication between VF and
PF and supports the following mailbox messages.
- Get and set MAC address
- Get link information
- Get stats
- Set and get MTU
- Send notification to PF

Signed-off-by: Sathesh Edara <sedara@marvell.com>

show more ...

net/octeon_ep: update queue size checks

Updates the output queue size checks to ensure
that queue is larger than back pressure watermark.
Add setting of default queue sizes to the minimum
so that ap

net/octeon_ep: update queue size checks

Updates the output queue size checks to ensure
that queue is larger than back pressure watermark.
Add setting of default queue sizes to the minimum
so that applications like testpmd can be started
without explicit queue size arguments.

Signed-off-by: Sathesh Edara <sedara@marvell.com>

show more ...

net/octeon_ep: add ISM

Adds the ISM specific functionality.

Signed-off-by: Sathesh Edara <sedara@marvell.com>

net/octeon_ep: propagate errors

Adds detection of loop limits being hit, and propagate errors up
the call chain when this happens.

Signed-off-by: Sathesh Edara <sedara@marvell.com>

net/octeon_ep: support CNX10K series SoC

Adds the required functionality in the Octeon endpoint driver
to support the following CNX10K series endpoint devices.
- CN10KA
- CN10KB
- CNF10KA
- CNF10KB

net/octeon_ep: support CNX10K series SoC

Adds the required functionality in the Octeon endpoint driver
to support the following CNX10K series endpoint devices.
- CN10KA
- CN10KB
- CNF10KA
- CNF10KB

Signed-off-by: Sathesh Edara <sedara@marvell.com>

show more ...

net/octeon_ep: support CNF95N and CNF95O SoC

Adds the required functionality in the Octeon endpoint
driver to support the cnf95n and cnf95o endpoint device.

Signed-off-by: Sathesh Edara <sedara@mar

net/octeon_ep: support CNF95N and CNF95O SoC

Adds the required functionality in the Octeon endpoint
driver to support the cnf95n and cnf95o endpoint device.

Signed-off-by: Sathesh Edara <sedara@marvell.com>

show more ...

net/octeon_ep: support port kind

Added port kind functionality.

Signed-off-by: Sathesh Edara <sedara@marvell.com>

net/octeon_ep: support device close

Added dev close functionality in the ethdev ops and moves
input and output queue deletion functionality into it
from otx_epdev_exit() routine.

Signed-off-by: Sat

net/octeon_ep: support device close

Added dev close functionality in the ethdev ops and moves
input and output queue deletion functionality into it
from otx_epdev_exit() routine.

Signed-off-by: Sathesh Edara <sedara@marvell.com>

show more ...

net/octeon_ep: support CN10K SoC

This patch adds the required functionality in the Octeon endpoint
driver to support the CN10K endpoint device. It adds the CN10K SoC
specific routines to configure,

net/octeon_ep: support CN10K SoC

This patch adds the required functionality in the Octeon endpoint
driver to support the CN10K endpoint device. It adds the CN10K SoC
specific routines to configure, enable, and disable input and output
queues to establish basic data transfers.

Signed-off-by: Sathesh Edara <sedara@marvell.com>

show more ...

net/octeon_ep: support link status

Added functionality to update link speed, duplex mode and link state.

Signed-off-by: Sathesh Edara <sedara@marvell.com>
Acked-by: Veerasenareddy Burru <vburru@mar

net/octeon_ep: support link status

Added functionality to update link speed, duplex mode and link state.

Signed-off-by: Sathesh Edara <sedara@marvell.com>
Acked-by: Veerasenareddy Burru <vburru@marvell.com>

show more ...

net/octeon_ep: support basic statistics

Added functionality to fetch and reset ethdev stats.

Signed-off-by: Sathesh Edara <sedara@marvell.com>
Acked-by: Veerasenareddy Burru <vburru@marvell.com>

net/octeon_ep: rename octeontx_ep

This patch renames octeon end point driver from octeontx_ep to
octeon_ep to enable single unified driver to support current
OcteonTx and future Octeon PCI endpoint

net/octeon_ep: rename octeontx_ep

This patch renames octeon end point driver from octeontx_ep to
octeon_ep to enable single unified driver to support current
OcteonTx and future Octeon PCI endpoint NICs to reflect common
driver for all Octeon based PCI endpoint NICs.

Signed-off-by: Sathesh Edara <sedara@marvell.com>
Acked-by: Veerasenareddy Burru <vburru@marvell.com>

show more ...