Hi, > -----Original Message----- > From: Alexander Kozyrev <akozy...@mellanox.com> > Sent: Monday, April 27, 2020 9:23 PM > To: dev@dpdk.org > Cc: sta...@dpdk.org; Raslan Darawsheh <rasl...@mellanox.com>; Slava > Ovsiienko <viachesl...@mellanox.com> > Subject: [PATCH] net/mlx5: fix packet length check assert in MPRQ > > 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
Patch applied to next-net-mlx, Kindest regards, Raslan Darawsheh