From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v7 4/5] app/testpmd: enable queue ring size configure Date: Tue, 24 Apr 2018 12:05:37 +0100 Message-ID: <7d8debb8-7b90-7abf-cb14-acc217b5b749@intel.com> References: <20180212045314.171616-1-qi.z.zhang@intel.com> <20180422115824.105219-1-qi.z.zhang@intel.com> <20180422115824.105219-5-qi.z.zhang@intel.com> <039ED4275CED7440929022BC67E70611531A6206@SHSMSX103.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: "Ananyev, Konstantin" , "dev@dpdk.org" , "Xing, Beilei" , "Wu, Jingjing" , "Lu, Wenzhuo" To: "Zhang, Qi Z" , "thomas@monjalon.net" Return-path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id 746A7271 for ; Tue, 24 Apr 2018 13:05:41 +0200 (CEST) In-Reply-To: <039ED4275CED7440929022BC67E70611531A6206@SHSMSX103.ccr.corp.intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 4/24/2018 4:16 AM, Zhang, Qi Z wrote: > > >> -----Original Message----- >> From: Yigit, Ferruh >> Sent: Tuesday, April 24, 2018 1:45 AM >> To: Zhang, Qi Z ; thomas@monjalon.net >> Cc: Ananyev, Konstantin ; dev@dpdk.org; >> Xing, Beilei ; Wu, Jingjing ; >> Lu, Wenzhuo >> Subject: Re: [PATCH v7 4/5] app/testpmd: enable queue ring size configure >> >> On 4/22/2018 12:58 PM, Qi Zhang wrote: >>> Add command to change specific queue's ring size configure, the new >>> value will only take effect after command that restart the device(port >>> stop /port start ) or command that setup the >>> queue(port rxq setup) at runtime. >>> >>> Signed-off-by: Qi Zhang >>> --- >>> app/test-pmd/cmdline.c | 102 >> ++++++++++++++++++++++++++++ >>> doc/guides/testpmd_app_ug/testpmd_funcs.rst | 9 +++ >>> 2 files changed, 111 insertions(+) >>> >>> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index >>> b50e11e60..22e4d4585 100644 >>> --- a/app/test-pmd/cmdline.c >>> +++ b/app/test-pmd/cmdline.c >>> @@ -846,6 +846,11 @@ static void cmd_help_long_parsed(void >> *parsed_result, >>> "port config mtu X value\n" >>> " Set the MTU of port X to a given value\n\n" >>> >>> + "port config (port_id) (rxq|txq) (queue_id) ring_size (value)\n" >>> + " Set a rx/tx queue's ring size configuration, the new" >>> + " value will take effect after command that (re-)start the port" >>> + " or command that setup the specific queue\n\n" >> >> "port config all rxq|txq|rxd|txd " is used to set number of queues >> (rxq) or number of descriptors in queue (rxd). >> >> Problem is this is not flexible and your version is better. >> >> What do you think removing old rxd|txd part with this patch, to prevent >> duplication? > > I'm not sure. > Do we need some command to reset all queue's ring size to default value. > Probably we need to support "all" syntax on new command before consider remove this. My concern was having to multiple commands with different syntax for same result, if both have different usecase that is OK. > > Also per queue config will be reset to original command line parameter by any command that call init_port_config, (for example: port config all rxq ...) > while "port config all rxd ..." modify the command line value, so they are different > > Regards > Qi >