From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 3AB6B2C15 for ; Thu, 21 Apr 2016 12:01:15 +0200 (CEST) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga104.fm.intel.com with ESMTP; 21 Apr 2016 03:01:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,512,1455004800"; d="scan'208";a="937109978" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.220.132]) by orsmga001.jf.intel.com with SMTP; 21 Apr 2016 03:01:11 -0700 Received: by (sSMTP sendmail emulation); Thu, 21 Apr 2016 11:01:10 +0025 Date: Thu, 21 Apr 2016 11:01:10 +0100 From: Bruce Richardson To: Harish Patil Cc: Thomas Monjalon , Rasesh Mody , "dev@dpdk.org" , "stephen@networkplumber.org" , Ameen Rahman , Sony Chacko Message-ID: <20160421100110.GB976@bricha3-MOBL3> References: <1459476927-21387-1-git-send-email-rasesh.mody@qlogic.com> <4202835.rtEgrshY1X@xps13> <2411922.nRbcghIYQB@xps13> <20160420092502.GB4080@bricha3-MOBL3> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH v5 10/10] qede: Enable PMD build X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Apr 2016 10:01:15 -0000 On Wed, Apr 20, 2016 at 04:43:44PM +0000, Harish Patil wrote: > > >On Wed, Apr 20, 2016 at 10:51:06AM +0200, Thomas Monjalon wrote: > >> 2016-04-20 00:14, Harish Patil: > >> > >2016-03-31 19:15, Rasesh Mody: > >> > >> --- a/config/common_base > >> > >> +++ b/config/common_base > >> > >> +CONFIG_RTE_LIBRTE_QEDE_RX_COAL_US=24 > >> > >> +CONFIG_RTE_LIBRTE_QEDE_TX_COAL_US=48 > >> > > > >> > >It looks to be some tuning which could be done at runtime. Isn't it? > >> > > >> > That’s right. Can you please suggest if there is any better option to > >>make > >> > it a runtime? There is no PMD API for that. > >> > >> There are some devargs for that. > >> For PCI dev, it can be passed in the whitelist option. > >> We should remove this limitation by having a devargs API (and command > >>line > >> options) independent of whitelisting. > > > >But back to the original setting. Are these likely to be values that are > >tunable > >or need to be tunable by the user? > > > > This is a tunable which is equivalent of ethtool -c (in linux) which > controls the rate at which status block is updated. > > >If not, I see little reason to make them > >run-time configurable - they could be defines inside the driver itself. > > There are defines already which set them to the defaults. The reason to > expose it as config option is that the user don’t have to change the > driver and rather just control it via config file. For now, I can remove > it till we find a better alternative to make those run time. Please > confirm and I can have it removed in next patch submission. > Removing config options is always a good thing. As you say we can always find a way to make them tunable in a later patchset. /Bruce