The assert that checks if there is a enough room for the
whole packet minus headroom data is written incorrectly.
The check should be negated in order to work properly.

Fixes: bd0d593 ("net/mlx5: enable MPRQ multi-stride operations")
Cc: sta...@dpdk.org

Signed-off-by: Alexander Kozyrev <akozy...@mellanox.com>
Acked-by: Viacheslav Ovsiienko <viachesl...@mellanox.com>
---
 drivers/net/mlx5/mlx5_rxtx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c
index a212010..6a17a9a 100644
--- a/drivers/net/mlx5/mlx5_rxtx.c
+++ b/drivers/net/mlx5/mlx5_rxtx.c
@@ -1814,7 +1814,7 @@ enum mlx5_txcmp_code {
                        /* Set mbuf head-room. */
                        SET_DATA_OFF(pkt, RTE_PKTMBUF_HEADROOM);
                        MLX5_ASSERT(pkt->ol_flags == EXT_ATTACHED_MBUF);
-                       MLX5_ASSERT(rte_pktmbuf_tailroom(pkt) <
+                       MLX5_ASSERT(rte_pktmbuf_tailroom(pkt) >=
                                len - (hdrm_overlap > 0 ? hdrm_overlap : 0));
                        DATA_LEN(pkt) = len;
                        /*
-- 
1.8.3.1

Reply via email to