19/03/2021 17:29, David Marchand: > On Wed, Mar 10, 2021 at 3:02 PM Thomas Monjalon <tho...@monjalon.net> wrote: > > > > The log levels are configured by using the name of the logs. > > Some drivers are aligned to follow a common log name standard: > > pmd.class.driver[.sub] > > Some "common" drivers skip the "class" part: > > pmd.driver.sub > > Without a check, this is likely to happen again. > I cooked this: [...] > +RTE_LOG_REGISTER(ixgbe_logtype_init, pmd.net.ixgb.init, NOTICE); > Added logtype does not comply with pattern: pmd.net.ixgbe > +RTE_LOG_REGISTER(compressdev_logtype, lib.compressde, NOTICE); > Added logtype does not comply with pattern: lib.compressdev > > WDYT?
I cannot parse awk but it looks nice. And in general, have more tools is a good thing. Would you submit this tool as a patch?