[dpdk-dev] [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option

2015-11-08 Thread Harish Patil
>> >Either other "debug" logs can be put as INFO level, or >> >you need a level 9 for verbose debug. >> >But I don't really understand what is the problem of a verbose debug. >> > >> >It is not critical for the acceptance of this patch. Just wanted to >> >discuss >> >it as we have many config

[dpdk-dev] [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option

2015-11-07 Thread Thomas Monjalon
> >Either other "debug" logs can be put as INFO level, or > >you need a level 9 for verbose debug. > >But I don't really understand what is the problem of a verbose debug. > > > >It is not critical for the acceptance of this patch. Just wanted to > >discuss > >it as we have many config options to

[dpdk-dev] [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option

2015-11-07 Thread Harish Patil
>2015-10-23 22:28, Harish Patil: >> >2015-09-11 17:42, Rasesh Mody: >> >> From: Harish Patil >> >> --- a/config/common_linuxapp >> >> +++ b/config/common_linuxapp >> >> @@ -220,6 +220,7 @@ CONFIG_RTE_LIBRTE_BNX2X_DEBUG_INIT=n >> >> CONFIG_RTE_LIBRTE_BNX2X_DEBUG_RX=n >> >>

[dpdk-dev] [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option

2015-10-24 Thread Thomas Monjalon
2015-10-23 22:28, Harish Patil: > >2015-09-11 17:42, Rasesh Mody: > >> From: Harish Patil > >> --- a/config/common_linuxapp > >> +++ b/config/common_linuxapp > >> @@ -220,6 +220,7 @@ CONFIG_RTE_LIBRTE_BNX2X_DEBUG_INIT=n > >> CONFIG_RTE_LIBRTE_BNX2X_DEBUG_RX=n > >>

[dpdk-dev] [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option

2015-10-23 Thread Harish Patil
> >2015-09-11 17:42, Rasesh Mody: >> From: Harish Patil >> --- a/config/common_linuxapp >> +++ b/config/common_linuxapp >> @@ -220,6 +220,7 @@ CONFIG_RTE_LIBRTE_BNX2X_DEBUG_INIT=n >> CONFIG_RTE_LIBRTE_BNX2X_DEBUG_RX=n >> CONFIG_RTE_LIBRTE_BNX2X_DEBUG_TX=n >>

[dpdk-dev] [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option

2015-10-20 Thread Thomas Monjalon
2015-09-11 17:42, Rasesh Mody: > From: Harish Patil > --- a/config/common_linuxapp > +++ b/config/common_linuxapp > @@ -220,6 +220,7 @@ CONFIG_RTE_LIBRTE_BNX2X_DEBUG_INIT=n > CONFIG_RTE_LIBRTE_BNX2X_DEBUG_RX=n > CONFIG_RTE_LIBRTE_BNX2X_DEBUG_TX=n > CONFIG_RTE_LIBRTE_BNX2X_MF_SUPPORT=n >

[dpdk-dev] [PATCH 3/5] bnx2x: Add RTE_LIBRTE_BNX2X_REG_ACCESS config option

2015-09-11 Thread Rasesh Mody
From: Harish Patil Signed-off-by: Harish Patil --- config/common_linuxapp |1 + drivers/net/bnx2x/Makefile |2 +- drivers/net/bnx2x/bnx2x.h |2 +- drivers/net/bnx2x/bnx2x_logs.h |8 drivers/net/bnx2x/debug.c | 24