[PATCH 01/13] net/mlx5/hws: move warn into debug level when needed

Raslan Darawsheh rasland at nvidia.com
Tue Mar 19 08:33:13 CET 2024


Hi,

> -----Original Message-----
> From: Thomas Monjalon <thomas at monjalon.net>
> Sent: Monday, March 18, 2024 4:49 PM
> To: Itamar Gozlan <igozlan at nvidia.com>; Erez Shitrit <erezsh at nvidia.com>;
> Hamdan Agbariya <hamdani at nvidia.com>; Yevgeny Kliteynik
> <kliteyn at nvidia.com>; Alex Vesker <valex at nvidia.com>; Raslan Darawsheh
> <rasland at nvidia.com>
> Cc: Slava Ovsiienko <viacheslavo at nvidia.com>; Dariusz Sosnowski
> <dsosnowski at nvidia.com>; Ori Kam <orika at nvidia.com>; Suanming Mou
> <suanmingm at nvidia.com>; Matan Azrad <matan at nvidia.com>; Mark Bloch
> <mbloch at nvidia.com>; dev at dpdk.org; Maayan Kashani
> <mkashani at nvidia.com>
> Subject: Re: [PATCH 01/13] net/mlx5/hws: move warn into debug level when
> needed
> 
> 18/03/2024 13:56, Raslan Darawsheh:
> > From: Itamar Gozlan <igozlan at nvidia.com>
> > > From: Erez Shitrit <erezsh at nvidia.com>
> > >
> > > When the user tries to create a matcher and if failed  with specific
> > > errno
> > > (E2BIG) the message will be in debug level and not in warning.
> > > It is a part of a feature when the user re-try to insert a new
> > > matching depends on that errno, no need the annoying message.
> > >
> > > Fixes: c55c2bf3533 ("net/mlx5/hws: net/mlx5/hws: add definer layer")
> > >
> > > Signed-off-by: Erez Shitrit <erezsh at nvidia.com>
> > > Acked-by: Matan Azrad <matan at nvidia.com>
> > Fixed Cc stable on several patches on this series, and reworded the
> > commits Series applied to next-net-mlx,
> 
> There is no cover letter for this series, so we are not able to understand how
> critical it is, and what is the general intent.
> 
> Is it supposed to be integrated in the last week of 24.03 release cycle?
> 
No, it's not critical for RC4 It's my fault, I'll drop it for now and we'll merge it in the next release cycle only.

Kindest regards
Raslan Darawsheh


More information about the dev mailing list