[dpdk-dev] [PATCH v2] eal: redefine logtype values

[De Lara Guarch, Pablo]

> -----Original Message-----
> From: Stephen Hemminger [mailto:stephen at networkplumber.org]
> Sent: Wednesday, April 12, 2017 10:41 PM
> To: De Lara Guarch, Pablo
> Cc: olivier.matz at 6wind.com; thomas.monjalon at 6wind.com;
> dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] eal: redefine logtype values
> 
> On Wed, 12 Apr 2017 16:35:32 +0100
> Pablo de Lara <pablo.de.lara.guarch at intel.com> wrote:
> 
> > +	/* register legacy log types */
> > +	__rte_log_register("eal",       RTE_LOGTYPE_EAL);
> > +	__rte_log_register("malloc",    RTE_LOGTYPE_MALLOC);
> > +	__rte_log_register("ring",      RTE_LOGTYPE_RING);
> > +	__rte_log_register("mempool",   RTE_LOGTYPE_MEMPOOL);
> > +	__rte_log_register("timer",     RTE_LOGTYPE_TIMER);
> > +	__rte_log_register("pmd",       RTE_LOGTYPE_PMD);
> > +	__rte_log_register("hash",      RTE_LOGTYPE_HASH);
> > +	__rte_log_register("lpm",       RTE_LOGTYPE_LPM);
> > +	__rte_log_register("kni",       RTE_LOGTYPE_KNI);
> > +	__rte_log_register("acl",       RTE_LOGTYPE_ACL);
> > +	__rte_log_register("power",     RTE_LOGTYPE_POWER);
> > +	__rte_log_register("meter",     RTE_LOGTYPE_METER);
> > +	__rte_log_register("sched",     RTE_LOGTYPE_SCHED);
> > +	__rte_log_register("port",      RTE_LOGTYPE_PORT);
> > +	__rte_log_register("table",     RTE_LOGTYPE_TABLE);
> > +	__rte_log_register("pipeline",  RTE_LOGTYPE_PIPELINE);
> > +	__rte_log_register("mbuf",      RTE_LOGTYPE_MBUF);
> > +	__rte_log_register("cryptodev", RTE_LOGTYPE_CRYPTODEV);
> > +	__rte_log_register("efd",       RTE_LOGTYPE_EFD);
> > +	__rte_log_register("eventdev",  RTE_LOGTYPE_EVENTDEV);
> > +	__rte_log_register("user1",     RTE_LOGTYPE_USER1);
> > +	__rte_log_register("user2",     RTE_LOGTYPE_USER2);
> > +	__rte_log_register("user3",     RTE_LOGTYPE_USER3);
> > +	__rte_log_register("user4",     RTE_LOGTYPE_USER4);
> > +	__rte_log_register("user5",     RTE_LOGTYPE_USER5);
> > +	__rte_log_register("user6",     RTE_LOGTYPE_USER6);
> > +	__rte_log_register("user7",     RTE_LOGTYPE_USER7);
> > +	__rte_log_register("user8",     RTE_LOGTYPE_USER8);
> 
> Why is this not a table rather than code?
> Data driven design is better.

Hi Stephen,

I just sent a v3 with your change proposed.

Thanks!
Pablo