diff mbox series

[v2] net/mlx5: fix threshold for mbuf replenishment in MPRQ

Message ID 20210713152112.393741-1-akozyrev@nvidia.com (mailing list archive)
State Accepted, archived
Delegated to: Raslan Darawsheh
Headers
Series [v2] net/mlx5: fix threshold for mbuf replenishment in MPRQ |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/github-robot success github build: passed
ci/iol-abi-testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-testing fail Testing issues
ci/Intel-compilation warning apply issues
ci/iol-intel-Performance fail Performance Testing issues

Commit Message

Alexander Kozyrev July 13, 2021, 3:21 p.m. UTC 
  The replenishment scheme for the vectorized MPRQ Rx burst aims
to improve the cache locality by allocating new mbufs only when
there are almost no mbufs left: one burst gap between allocated
and consumed indexes.

This gap is not big enough to accommodate a corner case when we
have a very aggressive CQE compression with multiple regular CQEs
at the beginning and 64 zipped CQEs at the end.

Need to keep in mind this case and extend the replenishment
threshold by MLX5_VPMD_RX_MAX_BURST (64) to avoid mbuf overflow.

Fixes: 5fc2e5c27d6 ("net/mlx5: fix mbuf overflow in vectorized MPRQ")
Cc: stable@dpdk.org

Signed-off-by: Alexander Kozyrev <akozyrev@nvidia.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
---

v2: increased the replenishment gap to MLX5_VPMD_RX_MAX_BURST
v1: https://patchwork.dpdk.org/project/dpdk/patch/20210712142910.314572-1-akozyrev@nvidia.com/

 drivers/net/mlx5/mlx5_rxtx_vec.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Raslan Darawsheh July 14, 2021, 11:57 a.m. UTC | #1 
Hi,

> -----Original Message-----
> From: Alexander Kozyrev <akozyrev@nvidia.com>
> Sent: Tuesday, July 13, 2021 6:21 PM
> To: dev@dpdk.org
> Cc: Raslan Darawsheh <rasland@nvidia.com>; Matan Azrad
> <matan@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>;
> stable@dpdk.org
> Subject: [PATCH v2] net/mlx5: fix threshold for mbuf replenishment in MPRQ
> 
> The replenishment scheme for the vectorized MPRQ Rx burst aims
> to improve the cache locality by allocating new mbufs only when
> there are almost no mbufs left: one burst gap between allocated
> and consumed indexes.
> 
> This gap is not big enough to accommodate a corner case when we
> have a very aggressive CQE compression with multiple regular CQEs
> at the beginning and 64 zipped CQEs at the end.
> 
> Need to keep in mind this case and extend the replenishment
> threshold by MLX5_VPMD_RX_MAX_BURST (64) to avoid mbuf overflow.
> 
> Fixes: 5fc2e5c27d6 ("net/mlx5: fix mbuf overflow in vectorized MPRQ")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Alexander Kozyrev <akozyrev@nvidia.com>
> Acked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
> ---
> 
> v2: increased the replenishment gap to MLX5_VPMD_RX_MAX_BURST
> v1:
> https://patchwork.dpdk.org/project/dpdk/patch/20210712142910.314572-1-
> akozyrev@nvidia.com/
> 
>  drivers/net/mlx5/mlx5_rxtx_vec.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Patch applied to next-net-mlx,

Kindest regards,
Raslan Darawsheh
diff mbox series

Patch

diff --git a/drivers/net/mlx5/mlx5_rxtx_vec.c b/drivers/net/mlx5/mlx5_rxtx_vec.c
index e64ef70181..e1b6d5422a 100644
--- a/drivers/net/mlx5/mlx5_rxtx_vec.c
+++ b/drivers/net/mlx5/mlx5_rxtx_vec.c
@@ -157,7 +157,8 @@  mlx5_rx_mprq_replenish_bulk_mbuf(struct mlx5_rxq_data *rxq)
 	unsigned int i;
 
 	if (n >= rxq->rq_repl_thresh &&
-	    rxq->elts_ci - rxq->rq_pi <= rxq->rq_repl_thresh) {
+	    rxq->elts_ci - rxq->rq_pi <=
+	    rxq->rq_repl_thresh + MLX5_VPMD_RX_MAX_BURST) {
 		MLX5_ASSERT(n >= MLX5_VPMD_RXQ_RPLNSH_THRESH(elts_n));
 		MLX5_ASSERT(MLX5_VPMD_RXQ_RPLNSH_THRESH(elts_n) >
 			     MLX5_VPMD_DESCS_PER_LOOP);