From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 7165F1E34 for ; Mon, 8 May 2017 03:01:45 +0200 (CEST) Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 May 2017 18:01:44 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,306,1491289200"; d="scan'208";a="96839099" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga005.jf.intel.com with ESMTP; 07 May 2017 18:01:43 -0700 Received: from fmsmsx124.amr.corp.intel.com (10.18.125.39) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.319.2; Sun, 7 May 2017 18:01:42 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by fmsmsx124.amr.corp.intel.com (10.18.125.39) with Microsoft SMTP Server (TLS) id 14.3.319.2; Sun, 7 May 2017 18:01:42 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.117]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.193]) with mapi id 14.03.0319.002; Mon, 8 May 2017 09:01:40 +0800 From: "Wu, Jingjing" To: Thomas Monjalon , "Dai, Wei" CC: "dev@dpdk.org" , "Lu, Wenzhuo" , "harish.patil@cavium.com" , "rasesh.mody@cavium.com" , "stephen.hurd@broadcom.com" , "ajit.khaparde@broadcom.com" , "Zhang, Helin" , "Ananyev, Konstantin" , "Chen, Jing D" , "adrien.mazarguil@6wind.com" , "nelio.laranjeiro@6wind.com" , "Richardson, Bruce" , "yuanhan.liu@linux.intel.com" , "maxime.coquelin@redhat.com" , "shepard.siegel@atomicrules.com" , "ed.czeck@atomicrules.com" , "john.miller@atomicrules.com" Thread-Topic: [dpdk-dev] [PATCH v7 3/3] app/testpmd: add a command to add many MAC addrs Thread-Index: AQHSxTlTRKzwLwFVpUiLqBpmeylPjKHlRfyAgARcd2A= Date: Mon, 8 May 2017 01:01:40 +0000 Message-ID: <9BB6961774997848B5B42BEC655768F810D638DE@SHSMSX103.ccr.corp.intel.com> References: <1493729065-17090-1-git-send-email-wei.dai@intel.com> <1493944802-52943-1-git-send-email-wei.dai@intel.com> <1493944802-52943-4-git-send-email-wei.dai@intel.com> <4247821.b6tKX1pDh8@xps> In-Reply-To: <4247821.b6tKX1pDh8@xps> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 10.0.102.7 dlp-reaction: no-action x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v7 3/3] app/testpmd: add a command to add many MAC addrs 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: Mon, 08 May 2017 01:01:46 -0000 > -----Original Message----- > From: Thomas Monjalon [mailto:thomas@monjalon.net] > Sent: Friday, May 5, 2017 10:24 PM > To: Dai, Wei > Cc: dev@dpdk.org; Lu, Wenzhuo ; > harish.patil@cavium.com; rasesh.mody@cavium.com; > stephen.hurd@broadcom.com; ajit.khaparde@broadcom.com; Zhang, Helin > ; Ananyev, Konstantin > ; Wu, Jingjing ; Che= n, > Jing D ; adrien.mazarguil@6wind.com; > nelio.laranjeiro@6wind.com; Richardson, Bruce ; > yuanhan.liu@linux.intel.com; maxime.coquelin@redhat.com; > shepard.siegel@atomicrules.com; ed.czeck@atomicrules.com; > john.miller@atomicrules.com > Subject: Re: [dpdk-dev] [PATCH v7 3/3] app/testpmd: add a command to add > many MAC addrs >=20 > 05/05/2017 02:40, Wei Dai: > > This patch is added to introduce a testpmd command which is used to > > add more than one MAC addresses one time. > > This command can simplify the test for the change where the type of > > return value of adding MAC address. > > Normally a MAC address may fails to be added only after many MAC > > addresses have been added. > > Without this command, a tester may only trigger failed MAC address by > > running many times of testpmd command 'mac_addr add' . > > > > Signed-off-by: Wei Dai > > --- > > app/test-pmd/cmdline.c | 55 > > ++++++++++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 55 insertions(+) >=20 > This patch has not been reviewed. > It is not a fix and I am not sure it is needed. The same doubt, what is this relationship with your fix in Patch 1/3?