[dpdk-dev] [PATCH] net/mlx5: fix inline packet size for ConnectX-4LX

Slava Ovsiienko viacheslavo at mellanox.com
Fri Feb 21 10:51:14 CET 2020


> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit at intel.com>
> Sent: Friday, February 21, 2020 11:44
> To: Slava Ovsiienko <viacheslavo at mellanox.com>; Matan Azrad
> <matan at mellanox.com>; dev at dpdk.org
> Cc: Raslan Darawsheh <rasland at mellanox.com>; Thomas Monjalon
> <thomas at monjalon.net>
> Subject: Re: [PATCH] net/mlx5: fix inline packet size for ConnectX-4LX
> 
> On 2/21/2020 7:39 AM, Slava Ovsiienko wrote:
> >> -----Original Message-----
> >> From: Matan Azrad <matan at mellanox.com>
> >> Sent: Friday, February 21, 2020 9:38
> >> To: Slava Ovsiienko <viacheslavo at mellanox.com>; dev at dpdk.org
> >> Cc: Raslan Darawsheh <rasland at mellanox.com>; Thomas Monjalon
> >> <thomas at monjalon.net>; ferruh.yigit at intel.com
> >> Subject: RE: [PATCH] net/mlx5: fix inline packet size for
> >> ConnectX-4LX
> >>
> >>
> >>
> >> From: Viacheslav Ovsiienko
> >>> This patch does extra inline packet size check to tune the
> >>> ConnectX-4LX performance in the legacy Multi-Packet Write mode.
> >>>
> >>> Signed-off-by: Viacheslav Ovsiienko <viacheslavo at mellanox.com>
> >>
> >> Slava, Don't you think we want it for stable releases?
> > Yes, thanks for reminding, just forgot "cc: stable", will send to stable ML.
> 
> Is the fixed commit known? (fixes line)

Yes, sorry for missed "Fixes" tag
It should be:
Fixes: 7593cf1d3500 ("net/mlx5: fix legacy multi-packet write session")

With best regards, Slava
> 
> >
> >>
> >> Besides it:
> >> Acked-by: Matan Azrad <matan at mellanox.com>
> >>
> >>>  drivers/net/mlx5/mlx5_rxtx.c | 2 ++
> >>>  1 file changed, 2 insertions(+)
> >>>
> >>> diff --git a/drivers/net/mlx5/mlx5_rxtx.c
> >>> b/drivers/net/mlx5/mlx5_rxtx.c index b55db4f..0df811b 100644
> >>> --- a/drivers/net/mlx5/mlx5_rxtx.c
> >>> +++ b/drivers/net/mlx5/mlx5_rxtx.c
> >>> @@ -4281,6 +4281,8 @@ enum mlx5_txcmp_code {
> >>>  			    loc->mbuf->ol_flags & PKT_TX_DYNF_NOINLINE)
> >>>  				goto pointer_empw;
> >>>  			if (MLX5_TXOFF_CONFIG(MPW)) {
> >>> +				if (dlen > txq->inlen_send)
> >>> +					goto pointer_empw;
> >>>  				tlen = dlen;
> >>>  				if (part == room) {
> >>>  					/* Open new inline MPW session. */
> >>> --
> >>> 1.8.3.1
> >



More information about the dev mailing list