From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 01DD52C6E for ; Thu, 13 Apr 2017 11:14:23 +0200 (CEST) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 13 Apr 2017 02:14:22 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.37,194,1488873600"; d="scan'208";a="88614742" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.122]) ([10.237.220.122]) by fmsmga006.fm.intel.com with ESMTP; 13 Apr 2017 02:14:21 -0700 To: Bruce Richardson , "Lu, Wenzhuo" Cc: "Zhang, Helin" , Olivier Matz , "dev@dpdk.org" , "Wu, Jingjing" References: <20170406141655.15097-1-olivier.matz@6wind.com> <20170406143642.GA9928@bricha3-MOBL3.ger.corp.intel.com> <6A0DE07E22DDAD4C9103DF62FEBC09093B597D61@shsmsx102.ccr.corp.intel.com> <20170412100212.GA20092@bricha3-MOBL3.ger.corp.intel.com> From: Ferruh Yigit Message-ID: Date: Thu, 13 Apr 2017 10:14:20 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.0 MIME-Version: 1.0 In-Reply-To: <20170412100212.GA20092@bricha3-MOBL3.ger.corp.intel.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH] net/i40e: disable init and driver logs by default X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 13 Apr 2017 09:14:24 -0000 On 4/12/2017 11:02 AM, Bruce Richardson wrote: > On Wed, Apr 12, 2017 at 04:12:18AM +0100, Lu, Wenzhuo wrote: >> Hi, >> >> >>> -----Original Message----- >>> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Zhang, Helin >>> Sent: Friday, April 7, 2017 10:03 AM >>> To: Richardson, Bruce >>> Cc: Olivier Matz; dev@dpdk.org; Wu, Jingjing; Yigit, Ferruh >>> Subject: Re: [dpdk-dev] [PATCH] net/i40e: disable init and driver logs by >>> default >>> >>> >>> >>>> -----Original Message----- >>>> From: Richardson, Bruce >>>> Sent: Thursday, April 6, 2017 10:37 PM >>>> To: Zhang, Helin >>>> Cc: Olivier Matz; dev@dpdk.org; Wu, Jingjing; Yigit, Ferruh >>>> Subject: Re: [PATCH] net/i40e: disable init and driver logs by default >>>> >>>> On Thu, Apr 06, 2017 at 03:31:23PM +0100, Zhang, Helin wrote: >>>>> >>>>> >>>>> -----Original Message----- >>>>> From: Olivier Matz [mailto:olivier.matz@6wind.com] >>>>> Sent: Thursday, April 6, 2017 10:17 PM >>>>> To: dev@dpdk.org >>>>> Cc: Zhang, Helin ; Wu, Jingjing >>>>> ; Richardson, Bruce >>>>> ; Yigit, Ferruh >>>>> Subject: [PATCH] net/i40e: disable init and driver logs by default >>>>> >>>>> Since "net/i40e: use dynamic log type for control logs", the i40e >>>>> driver is >>>> more verbose by default, which could result in testpmd being flooded >>>> by log messages in some conditions: >>>>> >>>>> Checking link statuses... >>>>> i40e_dev_handle_aq_msg(): Request 2561 is not supported yet >>>>> i40e_dev_handle_aq_msg(): Request 2561 is not supported yet >>>>> i40e_dev_handle_aq_msg(): Request 2561 is not supported yet >>>>> i40e_dev_handle_aq_msg(): Request 2561 is not supported yet >>>>> Port 0 Link Up - speed 40000 Mbps - full-duplex >>>>> Port 1 Link Up - speed 40000 Mbps - full-duplex >>>>> Done >>>>> testpmd> i40e_dev_handle_aq_msg(): Request 4097 is not supported >>> yet >>>>> i40e_dev_handle_aq_msg(): Request 4097 is not supported yet >>>>> i40e_dev_handle_aq_msg(): Request 4097 is not supported yet >>>>> >>>>> Fix this by disabling the dynamic logs by default. It is still >>>>> possible to enable >>>> them at runtime. >>>>> >>>>> Fixes: c143e5a3d9e1 ("net/i40e: use dynamic log type for control >>>>> logs") >>>>> >>>>> Signed-off-by: Olivier Matz >>>>> Acked-by: Helin Zhang <...> >>>> >>>> Hi Helin, >>>> >>>> Is this the correct fix? IMHO, if this is a problem, then we should >>>> surely not be hiding and ignoring the error. If it's not a problem, >>>> then the log level should be reduced to a lower level, e.g. INFO or NOTICE. >>>> >>>> Alternatively, this code could be modified to only print an error once >>>> for each unsupported request type. >>>> >>>> Ideally both solutions should be used, I think. I'm not sure I like >>>> setting the default log level to just show EMERG messages. >>>> >>>> Regards, >>>> /Bruce >>> Bruce, this is not a fix for the issue, which is under investigation by >>> developers now. >>> >>> For the by default log level, any guideline for that? >>> I was OK with that changes, but not sure if there is any better choices. >> >> To my opinion, it's not appropriate to say something not supported is an error. How about fix this problem like, >> >> diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c >> index 6927fde..91df587 100644 >> --- a/drivers/net/i40e/i40e_ethdev.c >> +++ b/drivers/net/i40e/i40e_ethdev.c >> @@ -5794,7 +5794,7 @@ struct i40e_vsi * >> } >> break; >> default: >> - PMD_DRV_LOG(ERR, "Request %u is not supported yet", >> + PMD_DRV_LOG(DEBUG, "Request %u is not supported yet", >> opcode); >> break; >> } > > Ok to me. +1. Wenzhuo would you mind sending this as patch? Thanks, ferruh > > /Bruce >