[dpdk-dev] [PATCH 06/18] net/avp: align dynamic log names with standard

Legacy, Allain Allain.Legacy at windriver.com
Thu Jan 25 18:46:50 CET 2018


> -----Original Message-----
> From: Van Haaren, Harry [mailto:harry.van.haaren at intel.com]
> Sent: Thursday, January 25, 2018 12:06 PM
> Subject: RE: [PATCH 06/18] net/avp: align dynamic log names with standard
> 
> > From: Legacy, Allain [mailto:Allain.Legacy at windriver.com]
> > Sent: Thursday, January 25, 2018 12:06 PM
> > To: Van Haaren, Harry <harry.van.haaren at intel.com>; dev at dpdk.org
> > Cc: Van Haaren, Harry <harry.van.haaren at intel.com>; Peters, Matt (Wind
> > River) <matt.peters at windriver.com>
> > Subject: RE: [PATCH 06/18] net/avp: align dynamic log names with
> > standard
> >
> > > -----Original Message-----
> > > From: Harry van Haaren [mailto:harry.van.haaren at intel.com]
> > > Sent: Thursday, January 25, 2018 4:01 AM
> > > Subject: [PATCH 06/18] net/avp: align dynamic log names with
> > > standard
> > >
> > > This commit aligns the names for dynamic logging with the newly
> > > defined logging format.
> > >
> > > Signed-off-by: Harry van Haaren <harry.van.haaren at intel.com>
> > >
> > > ---
> > >
> > > Maintainer:
> > > Cc: allain.legacy at windriver.com
> > > Cc: matt.peters at windriver.com
> > > ---
> > >  drivers/net/avp/avp_ethdev.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/net/avp/avp_ethdev.c
> > > b/drivers/net/avp/avp_ethdev.c index e4ad7b0..dba9912 100644
> > > --- a/drivers/net/avp/avp_ethdev.c
> > > +++ b/drivers/net/avp/avp_ethdev.c
> > > @@ -2318,7 +2318,7 @@ RTE_INIT(avp_init_log);  static void
> > >  avp_init_log(void)
> > >  {
> > > -	avp_logtype_driver = rte_log_register("pmd.avp.driver");
> > > +	avp_logtype_driver = rte_log_register("pmd.net.avp.driver");
> > >  	if (avp_logtype_driver >= 0)
> > >  		rte_log_set_level(avp_logtype_driver, RTE_LOG_NOTICE);  }
> > > --
> > > 2.7.4
> >
> > Signed-off-by: Allain Legacy <allain.legacy at windriver.com>
> 
> 
> Hi Allain,
> 
> I presume you meant "Acked-by" above, not "Signed off by"?
> If so, please Ack :)
> 
> Thanks, -Harry

Yes, sorry!

Acked-by:  Allain Legacy <allain.legacy at windriver.com>



More information about the dev mailing list