diff mbox series

net/mana: start secondary process queues by default

Message ID 1706661993-5636-1-git-send-email-longli@linuxonhyperv.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series net/mana: start secondary process queues by default |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-sample-apps-testing success Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/iol-mellanox-Performance success Performance Testing PASS
ci/intel-Testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/intel-Functional success Functional PASS
ci/iol-compile-arm64-testing success Testing PASS

Commit Message

Long Li Jan. 31, 2024, 12:46 a.m. UTC 
  From: Long Li <longli@microsoft.com>

Secondary processes are started after primary, and in most cases with
the device already started. Make them being able to process packets as
soon as they start.

This also works with the case where the primary process decides to start
the device at a later time after secondary processes have started. The
application should guarantee not to send any packets before the device is
started.

Signed-off-by: Long Li <longli@microsoft.com>
---
 drivers/net/mana/mana.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Ferruh Yigit Feb. 7, 2024, 10:58 a.m. UTC | #1 
On 1/31/2024 12:46 AM, longli@linuxonhyperv.com wrote:
> From: Long Li <longli@microsoft.com>
> 
> Secondary processes are started after primary, and in most cases with
> the device already started. Make them being able to process packets as
> soon as they start.
> 
> This also works with the case where the primary process decides to start
> the device at a later time after secondary processes have started. The
> application should guarantee not to send any packets before the device is
> started.
> 
> Signed-off-by: Long Li <longli@microsoft.com>
> 

Applied to dpdk-next-net/main, thanks.
diff mbox series

Patch

diff --git a/drivers/net/mana/mana.c b/drivers/net/mana/mana.c
index 58257c971e..386081f667 100644
--- a/drivers/net/mana/mana.c
+++ b/drivers/net/mana/mana.c
@@ -1312,8 +1312,8 @@  mana_probe_port(struct ibv_device *ibdev, struct ibv_device_attr_ex *dev_attr,
 		/* fd is no not used after mapping doorbell */
 		close(fd);
 
-		eth_dev->tx_pkt_burst = mana_tx_burst_removed;
-		eth_dev->rx_pkt_burst = mana_rx_burst_removed;
+		eth_dev->tx_pkt_burst = mana_tx_burst;
+		eth_dev->rx_pkt_burst = mana_rx_burst;
 
 		rte_spinlock_lock(&mana_shared_data->lock);
 		mana_shared_data->secondary_cnt++;