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 7A2F5532D for ; Thu, 9 Mar 2017 03:59:13 +0100 (CET) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Mar 2017 18:59:12 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,266,1486454400"; d="scan'208";a="65667685" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga004.jf.intel.com with ESMTP; 08 Mar 2017 18:59:12 -0800 Received: from fmsmsx112.amr.corp.intel.com (10.18.116.6) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.248.2; Wed, 8 Mar 2017 18:59:11 -0800 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by FMSMSX112.amr.corp.intel.com (10.18.116.6) with Microsoft SMTP Server (TLS) id 14.3.248.2; Wed, 8 Mar 2017 18:59:11 -0800 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.20]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.132]) with mapi id 14.03.0248.002; Thu, 9 Mar 2017 10:59:08 +0800 From: "Zhang, Qi Z" To: "Yigit, Ferruh" , "Wu, Jingjing" , "Zhang, Helin" CC: "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH 2/2] app/testpmd: enable VF untag drop in testpmd Thread-Index: AQHSk7q0JYm5pWVgGEC+yy7dd0rlEKGIujcAgAMNnUA= Date: Thu, 9 Mar 2017 02:59:07 +0000 Message-ID: <039ED4275CED7440929022BC67E7061153064A3C@SHSMSX103.ccr.corp.intel.com> References: <20170303015924.68986-1-qi.z.zhang@intel.com> <20170303015924.68986-3-qi.z.zhang@intel.com> <8d111f4c-274d-e59f-1e7e-a2550b80d1bb@intel.com> In-Reply-To: <8d111f4c-274d-e59f-1e7e-a2550b80d1bb@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZWNiNGI2NmUtMTQ5NC00NDJmLWI1NDctZDk3M2M3OTMzYTFkIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6IkJGOVFETkFrck9EbVYwWnU0RzBVcFczc1RvR3UwK1BVWmxlNFVWZkNHNTQ9In0= x-ctpclassification: CTP_IC 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 2/2] app/testpmd: enable VF untag drop in testpmd 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, 09 Mar 2017 02:59:14 -0000 > -----Original Message----- > From: Yigit, Ferruh > Sent: Tuesday, March 7, 2017 7:14 PM > To: Zhang, Qi Z ; Wu, Jingjing ; > Zhang, Helin > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH 2/2] app/testpmd: enable VF untag drop in > testpmd >=20 > On 3/3/2017 1:59 AM, Qi Zhang wrote: > > Add command line to support untag drop to specific VF in testpmd. > > > > Signed-off-by: Qi Zhang > > --- > > app/test-pmd/cmdline.c | 104 > > +++++++++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 104 insertions(+) > > > > diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index > > 43fc636..4ddc2c9 100644 > > --- a/app/test-pmd/cmdline.c > > +++ b/app/test-pmd/cmdline.c > > @@ -311,6 +311,10 @@ static void cmd_help_long_parsed(void > > *parsed_result, > > > > "set vf vlan antispoof (port_id) (vf_id) (on|off)\n" > > " Set VLAN antispoof for a VF from the PF.\n\n" > > +#ifdef RTE_LIBRTE_I40E_PMD > > + "set vf vlan untagdrop (port_id) (vf_id) (on|off)\n" > > + " Set VLAN untag drop for a VF from the PF.\n\n" > > +#endif >=20 > We should decide how to implement PMD specific APIs in testpmd, and be > consistent about it. >=20 > Currently there are two approaches: >=20 > 1- Wrap PMD specific feature and API with and PMD #ifdef, as done here. >=20 > 2- Enable feature by default, return -ENOTSUP for port_id that does not s= upport > it. Ex: cmd_vf_rxvlan_filter. > > I am for second option. And explicitly not disabling I40E driver does not= mean > you should have those NICs in your runtime environment, so the effect wil= l be > same as always enabling it. > Yes, I notice this problem, during implementation, I saw both patterns exis= t, so I have to choose one of them We'd better align this. Both option ok for me, but a little bit prefer option 1 , since it's not ne= cessary to explore a command if no backend device, that make the hint more = clean. >=20 > And since number of PMD specific APIs are increasing, perhaps we should f= ind a > better approach for testpmd to prevent them corrupting testpmd. Will think about this, also like to know if you or anyone have any good sug= gestion. >=20 > Also it may worth to discuss why number of PMD specific APIs are increasi= ng. >=20 > > > > "set vf vlan tag (port_id) (vf_id) (on|off)\n" > > " Set VLAN tag for a VF from the PF.\n\n" > > @@ -10995,6 +10999,103 @@ cmdline_parse_inst_t > cmd_set_vf_vlan_anti_spoof =3D { > > }, > > }; > > > <...>