On Fri, 03 Feb 2023 21:18:40 +0100 Thomas Monjalon <tho...@monjalon.net> wrote:
> 03/02/2023 18:33, Stephen Hemminger: > > On Fri, 03 Feb 2023 09:42:45 +0100 > > Thomas Monjalon <tho...@monjalon.net> wrote: > > > > > 03/02/2023 01:25, Stephen Hemminger: > > > > On Wed, 1 Feb 2023 13:34:41 +0000 > > > > Shivah Shankar Shankar Narayan Rao <sshankarn...@marvell.com> wrote: > > > > > > > > > --- a/lib/eal/include/rte_log.h > > > > > +++ b/lib/eal/include/rte_log.h > > > > > @@ -48,6 +48,7 @@ extern "C" { > > > > > #define RTE_LOGTYPE_EFD 18 /**< Log related to EFD. */ > > > > > #define RTE_LOGTYPE_EVENTDEV 19 /**< Log related to eventdev. */ > > > > > #define RTE_LOGTYPE_GSO 20 /**< Log related to GSO. */ > > > > > +#define RTE_LOGTYPE_MLDEV 21 /**< Log related to mldev. */ > > > > > > > > NAK to this part. > > > > No new static logtypes please. > > > > > > Good catch. > > > By the way, we should remove unused RTE_LOGTYPE_*. > > > > Yes, for 23.11 would like to work down the list. > > Do we need to wait 23.11? > It is not an ABI breakage. > And most of these defines are already unused. Turning them into deprecated would be API breakage though