> -----Original Message----- > From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Wednesday, April 12, 2017 10:41 PM > To: De Lara Guarch, Pablo > Cc: olivier.m...@6wind.com; thomas.monja...@6wind.com; > dev@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.gua...@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
