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 2BA4F101B for ; Tue, 28 Mar 2017 14:28:58 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=intel; t=1490704139; x=1522240139; h=subject:to:references:cc:from:message-id:date: mime-version:in-reply-to:content-transfer-encoding; bh=HFn/r88QA1FVStOP/mbTmqiGiwyaSKzXD971Qx8rbrU=; b=ub50KVLDCRqOj24IoHVeB+XxCLYOriDJ7uLE8n+Mvd92gXWsq7T0lS7G 63fpLBaCWWOmgtgvamRIWO+I3kDMhQ==; Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Mar 2017 05:28:57 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,236,1486454400"; d="scan'208";a="1112921373" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.122]) ([10.237.220.122]) by orsmga001.jf.intel.com with ESMTP; 28 Mar 2017 05:28:56 -0700 To: "Wu, Jingjing" , "Zhang, Qi Z" , "Zhang, Helin" References: <20170227045612.48030-1-qi.z.zhang@intel.com> <20170312120845.31666-1-qi.z.zhang@intel.com> <20170312120845.31666-4-qi.z.zhang@intel.com> <18f75650-47bd-e82f-dcc6-c0d4f7000b2a@intel.com> <9BB6961774997848B5B42BEC655768F810D17B9C@SHSMSX103.ccr.corp.intel.com> Cc: "dev@dpdk.org" , "De Lara Guarch, Pablo" From: Ferruh Yigit Message-ID: Date: Tue, 28 Mar 2017 13:28:56 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <9BB6961774997848B5B42BEC655768F810D17B9C@SHSMSX103.ccr.corp.intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 3/3] app/testpmd: add CL for ptype mapping configure 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: Tue, 28 Mar 2017 12:28:59 -0000 On 3/28/2017 4:34 AM, Wu, Jingjing wrote: > > >> -----Original Message----- >> From: Yigit, Ferruh >> Sent: Friday, March 17, 2017 12:27 AM >> To: Zhang, Qi Z ; Wu, Jingjing ; >> Zhang, Helin >> Cc: dev@dpdk.org; De Lara Guarch, Pablo >> Subject: Re: [dpdk-dev] [PATCH v2 3/3] app/testpmd: add CL for ptype mapping >> configure >> >> On 3/12/2017 12:08 PM, Qi Zhang wrote: >>> Add below command line to configure ptype mapping. >>> ptype mapping get . >>> ptype mapping replace . >>> ptype mapping reset . >>> ptype mapping update . >>> >>> Signed-off-by: Qi Zhang >>> --- >>> app/test-pmd/cmdline.c | 372 >>> +++++++++++++++++++++++++++++++++++++++++++++++++ >>> 1 file changed, 372 insertions(+) >>> >>> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index >>> 43fc636..85f07ba 100644 >>> --- a/app/test-pmd/cmdline.c >>> +++ b/app/test-pmd/cmdline.c >>> @@ -575,6 +575,18 @@ static void cmd_help_long_parsed(void >> *parsed_result, >>> "E-tag set filter del e-tag-id (value) port (port_id)\n" >>> " Delete an E-tag forwarding filter on a port\n\n" >>> >>> + "ptype mapping get (port_id) (valid_only)\n" >>> + " Get ptype mapping on a port\n\n" >>> + >>> + "ptype mapping replace (port_id) (target) (mask) >> (pky_type)\n" >>> + " Replace target with the pkt_type in ptype >> mapping\n\n" >>> + >>> + "ptype mapping reset (port_id)\n" >>> + " Reset ptype mapping on a port\n\n" >>> + >>> + "ptype mapping update (port_id) (hw_ptype) >> (sw_ptype)\n" >>> + " Update a ptype mapping item on a port\n\n" >>> + >> >> This adds new root level "ptype" command, I think it would be nice to get an Ack >> from testpmd maintainers for this. >> >> testpmd now supports many commands, and I don't know if it is only me, but I >> am having hard time to find some commands. Perhaps commands can be >> grouped better. > > Now following groups are used. I think the most confusing one > is config or ports. Too many commands are included in it. > But it looks like new group is not necessary for Qi's commands. > We need extra work to refine it. > > "Help is available for the following sections:\n\n" > " help control : Start and stop forwarding.\n" > " help display : Displaying port, stats and config " > "information.\n" > " help config : Configuration information.\n" > " help ports : Configuring ports.\n" > " help registers : Reading and setting port registers.\n" > " help filters : Filters configuration help.\n" > " help all : All of the above sections.\n\n" > I don't know about new config group, but I was worried about new root level config option, these mostly makes sense when you add, in limited context, but can be hard to use them later, in whole application level. If you are OK with new commands, as testpmd maintainer, I don't have any objection. Thanks, ferruh