diff mbox

[dpdk-dev,1/2] enic: silence log message

Message ID 1428514670-7982-2-git-send-email-stephen@networkplumber.org (mailing list archive)
State Rejected, archived
Headers

Commit Message

Stephen Hemminger April 8, 2015, 5:37 p.m. UTC 
  From: Stephen Hemminger <shemming@brocade.com>

Silence is normal. drivers should speak only when spoken to and not
be chatty.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_pmd_enic/enic_main.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Bruce Richardson May 19, 2015, 10:08 a.m. UTC | #1 
On Wed, Apr 08, 2015 at 10:37:49AM -0700, Stephen Hemminger wrote:
> From: Stephen Hemminger <shemming@brocade.com>
> 
> Silence is normal. drivers should speak only when spoken to and not
> be chatty.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

> ---
>  lib/librte_pmd_enic/enic_main.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/lib/librte_pmd_enic/enic_main.c b/lib/librte_pmd_enic/enic_main.c
> index 0892b3e..508621e 100644
> --- a/lib/librte_pmd_enic/enic_main.c
> +++ b/lib/librte_pmd_enic/enic_main.c
> @@ -1048,8 +1048,6 @@ int enic_probe(struct enic *enic)
>  	struct rte_pci_device *pdev = enic->pdev;
>  	int err = -1;
>  
> -	dev_debug(enic, " Initializing ENIC PMD version %s\n", DRV_VERSION);
> -
>  	enic->bar0.vaddr = (void *)pdev->mem_resource[0].addr;
>  	enic->bar0.len = pdev->mem_resource[0].len;
>  
> -- 
> 2.1.4
>
Sujith Sankar May 20, 2015, 4:30 p.m. UTC | #2 
On 19/05/15 3:38 pm, "Bruce Richardson" <bruce.richardson@intel.com> wrote:

>On Wed, Apr 08, 2015 at 10:37:49AM -0700, Stephen Hemminger wrote:
>> From: Stephen Hemminger <shemming@brocade.com>
>> 
>> Silence is normal. drivers should speak only when spoken to and not
>> be chatty.
>> 
>> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
>
>Acked-by: Bruce Richardson <bruce.richardson@intel.com>
>
>> ---
>>  lib/librte_pmd_enic/enic_main.c | 2 --
>>  1 file changed, 2 deletions(-)
>> 
>> diff --git a/lib/librte_pmd_enic/enic_main.c
>>b/lib/librte_pmd_enic/enic_main.c
>> index 0892b3e..508621e 100644
>> --- a/lib/librte_pmd_enic/enic_main.c
>> +++ b/lib/librte_pmd_enic/enic_main.c
>> @@ -1048,8 +1048,6 @@ int enic_probe(struct enic *enic)
>>  	struct rte_pci_device *pdev = enic->pdev;
>>  	int err = -1;
>>  
>> -	dev_debug(enic, " Initializing ENIC PMD version %s\n", DRV_VERSION);
>> -

It would be good to retain this under RTE_LIBRTE_ENIC_DEBUG.

>>  	enic->bar0.vaddr = (void *)pdev->mem_resource[0].addr;
>>  	enic->bar0.len = pdev->mem_resource[0].len;
>>  
>> -- 
>> 2.1.4
>>
Stephen Hemminger May 20, 2015, 5:47 p.m. UTC | #3 
Other drivers are quiet, why does this one need to be noisy

On Wed, May 20, 2015 at 9:30 AM, Sujith Sankar (ssujith) <ssujith@cisco.com>
wrote:

>
>
> On 19/05/15 3:38 pm, "Bruce Richardson" <bruce.richardson@intel.com>
> wrote:
>
> >On Wed, Apr 08, 2015 at 10:37:49AM -0700, Stephen Hemminger wrote:
> >> From: Stephen Hemminger <shemming@brocade.com>
> >>
> >> Silence is normal. drivers should speak only when spoken to and not
> >> be chatty.
> >>
> >> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> >
> >Acked-by: Bruce Richardson <bruce.richardson@intel.com>
> >
> >> ---
> >>  lib/librte_pmd_enic/enic_main.c | 2 --
> >>  1 file changed, 2 deletions(-)
> >>
> >> diff --git a/lib/librte_pmd_enic/enic_main.c
> >>b/lib/librte_pmd_enic/enic_main.c
> >> index 0892b3e..508621e 100644
> >> --- a/lib/librte_pmd_enic/enic_main.c
> >> +++ b/lib/librte_pmd_enic/enic_main.c
> >> @@ -1048,8 +1048,6 @@ int enic_probe(struct enic *enic)
> >>      struct rte_pci_device *pdev = enic->pdev;
> >>      int err = -1;
> >>
> >> -    dev_debug(enic, " Initializing ENIC PMD version %s\n",
> DRV_VERSION);
> >> -
>
> It would be good to retain this under RTE_LIBRTE_ENIC_DEBUG.
>
> >>      enic->bar0.vaddr = (void *)pdev->mem_resource[0].addr;
> >>      enic->bar0.len = pdev->mem_resource[0].len;
> >>
> >> --
> >> 2.1.4
> >>
>
>
Sujith Sankar May 21, 2015, 1:52 a.m. UTC | #4 
Stephen,
The enic debug flag is off by default.  So it is going to print the
version only if the user wants it to.
Isn¹t that fine?

Thanks,
-Sujith 
From:  Stephen Hemminger <stephen@networkplumber.org>
Date:  Wednesday, 20 May 2015 11:17 pm
To:  "Sujith Sankar (ssujith)" <ssujith@cisco.com>
Cc:  Bruce Richardson <bruce.richardson@intel.com>, "dev@dpdk.org"
<dev@dpdk.org>, Stephen Hemminger <shemming@brocade.com>
Subject:  Re: [dpdk-dev] [PATCH 1/2] enic: silence log message


Other drivers are quiet, why does this one need to be noisy


On Wed, May 20, 2015 at 9:30 AM, Sujith Sankar (ssujith)
<ssujith@cisco.com> wrote:



On 19/05/15 3:38 pm, "Bruce Richardson" <bruce.richardson@intel.com> wrote:

>On Wed, Apr 08, 2015 at 10:37:49AM -0700, Stephen Hemminger wrote:
>> From: Stephen Hemminger <shemming@brocade.com>
>>
>> Silence is normal. drivers should speak only when spoken to and not
>> be chatty.
>>
>> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
>
>Acked-by: Bruce Richardson <bruce.richardson@intel.com>
>
>> ---
>>  lib/librte_pmd_enic/enic_main.c | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/lib/librte_pmd_enic/enic_main.c
>>b/lib/librte_pmd_enic/enic_main.c
>> index 0892b3e..508621e 100644
>> --- a/lib/librte_pmd_enic/enic_main.c
>> +++ b/lib/librte_pmd_enic/enic_main.c
>> @@ -1048,8 +1048,6 @@ int enic_probe(struct enic *enic)
>>      struct rte_pci_device *pdev = enic->pdev;
>>      int err = -1;
>>
>> -    dev_debug(enic, " Initializing ENIC PMD version %s\n",
>>DRV_VERSION);
>> -

It would be good to retain this under RTE_LIBRTE_ENIC_DEBUG.

>>      enic->bar0.vaddr = (void *)pdev->mem_resource[0].addr;
>>      enic->bar0.len = pdev->mem_resource[0].len;
>>
>> --
>> 2.1.4
>>
diff mbox

Patch

diff --git a/lib/librte_pmd_enic/enic_main.c b/lib/librte_pmd_enic/enic_main.c
index 0892b3e..508621e 100644
--- a/lib/librte_pmd_enic/enic_main.c
+++ b/lib/librte_pmd_enic/enic_main.c
@@ -1048,8 +1048,6 @@  int enic_probe(struct enic *enic)
 	struct rte_pci_device *pdev = enic->pdev;
 	int err = -1;
 
-	dev_debug(enic, " Initializing ENIC PMD version %s\n", DRV_VERSION);
-
 	enic->bar0.vaddr = (void *)pdev->mem_resource[0].addr;
 	enic->bar0.len = pdev->mem_resource[0].len;