Hi, > -----Original Message----- > From: Alexander Kozyrev <[email protected]> > Sent: Monday, April 27, 2020 9:23 PM > To: [email protected] > Cc: [email protected]; Raslan Darawsheh <[email protected]>; Slava > Ovsiienko <[email protected]> > 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: [email protected] > > Signed-off-by: Alexander Kozyrev <[email protected]> > Acked-by: Viacheslav Ovsiienko <[email protected]> > --- > 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
