Re: [dpdk-dev] [PATCH v2 2/3] log: track log level changes

2021-04-09 Thread Thomas Monjalon
09/04/2021 11:08, David Marchand: > On Thu, Apr 8, 2021 at 5:55 PM Thomas Monjalon wrote: > > > > 24/03/2021 11:32, David Marchand: > > > Add a log message when registering log types and changing log levels. > > [...] > > > + RTE_LOG(DEBUG, EAL, "%s logtype level changed from %s to >

Re: [dpdk-dev] [PATCH v2 2/3] log: track log level changes

2021-04-09 Thread David Marchand
On Thu, Apr 8, 2021 at 5:55 PM Thomas Monjalon wrote: > > 24/03/2021 11:32, David Marchand: > > Add a log message when registering log types and changing log levels. > [...] > > + RTE_LOG(DEBUG, EAL, "%s logtype level changed from %s to > > %s\n", > > I would prefer "foo log level cha

Re: [dpdk-dev] [PATCH v2 2/3] log: track log level changes

2021-04-08 Thread Thomas Monjalon
24/03/2021 11:32, David Marchand: > Add a log message when registering log types and changing log levels. [...] > + RTE_LOG(DEBUG, EAL, "%s logtype level changed from %s to %s\n", I would prefer "foo log level changed" instead of "foo logtype level changed". Opinions? > +

[dpdk-dev] [PATCH v2 2/3] log: track log level changes

2021-03-24 Thread David Marchand
Add a log message when registering log types and changing log levels. __rte_log_register previous handled both legacy and dynamic logtypes. To simplify the code, __rte_log_register is reworked to only handle dynamic logtypes and takes a log level. Example: $ DPDK_TEST=logs_autotest ./build/app/te