Re: [dpdk-dev] [PATCH 01/11] avp: implement dynamic logging

2017-12-22 Thread Olivier MATZ
On Thu, Dec 21, 2017 at 10:02:14AM -0800, Ferruh Yigit wrote: > On 12/20/2017 10:58 AM, Stephen Hemminger wrote: > >> [1] something like: > >> #define INIT_LOG_VAR_NAME(pmd, type) logtype_ ## pmd ## _ ## type > >> #define INIT_LOG_FUNC_NAME(pmd, type) log_ ## pmd ## _ ## type > >> > >> #defin

Re: [dpdk-dev] [PATCH 01/11] avp: implement dynamic logging

2017-12-21 Thread Ferruh Yigit
On 12/20/2017 10:58 AM, Stephen Hemminger wrote: > On Tue, 19 Dec 2017 17:53:50 -0800 > Ferruh Yigit wrote: > >> On 12/18/2017 10:38 PM, Stephen Hemminger wrote: >>> All PMD should be using dynamic log levels. >>> >>> Signed-off-by: Stephen Hemminger <...> >>> @@ -2312,3 +2313,12 @@ avp_dev_st

Re: [dpdk-dev] [PATCH 01/11] avp: implement dynamic logging

2017-12-20 Thread Stephen Hemminger
On Tue, 19 Dec 2017 17:53:50 -0800 Ferruh Yigit wrote: > On 12/18/2017 10:38 PM, Stephen Hemminger wrote: > > All PMD should be using dynamic log levels. > > > > Signed-off-by: Stephen Hemminger > > --- > > config/common_base | 1 - > > drivers/net/avp/avp_ethdev.c | 10 ++ >

Re: [dpdk-dev] [PATCH 01/11] avp: implement dynamic logging

2017-12-19 Thread Ferruh Yigit
On 12/18/2017 10:38 PM, Stephen Hemminger wrote: > All PMD should be using dynamic log levels. > > Signed-off-by: Stephen Hemminger > --- > config/common_base | 1 - > drivers/net/avp/avp_ethdev.c | 10 ++ > drivers/net/avp/avp_logs.h | 9 - > 3 files changed, 14 in

[dpdk-dev] [PATCH 01/11] avp: implement dynamic logging

2017-12-18 Thread Stephen Hemminger
All PMD should be using dynamic log levels. Signed-off-by: Stephen Hemminger --- config/common_base | 1 - drivers/net/avp/avp_ethdev.c | 10 ++ drivers/net/avp/avp_logs.h | 9 - 3 files changed, 14 insertions(+), 6 deletions(-) diff --git a/config/common_base b/co