From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 7DAB23DC for ; Fri, 16 Dec 2016 13:22:58 +0100 (CET) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga104.jf.intel.com with ESMTP; 16 Dec 2016 04:22:57 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,357,1477983600"; d="scan'208";a="18942782" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga002.jf.intel.com with ESMTP; 16 Dec 2016 04:22:56 -0800 Received: from fmsmsx115.amr.corp.intel.com (10.18.116.19) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.248.2; Fri, 16 Dec 2016 04:22:56 -0800 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by fmsmsx115.amr.corp.intel.com (10.18.116.19) with Microsoft SMTP Server (TLS) id 14.3.248.2; Fri, 16 Dec 2016 04:22:56 -0800 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.97]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.77]) with mapi id 14.03.0248.002; Fri, 16 Dec 2016 20:22:52 +0800 From: "Xing, Beilei" To: Adrien Mazarguil , "Pei, Yulong" CC: "dev@dpdk.org" , Thomas Monjalon , "De Lara Guarch, Pablo" , Olivier Matz Thread-Topic: [dpdk-dev] [PATCH 12/22] app/testpmd: add rte_flow item spec handler Thread-Index: AQHSQCZhM86OhCmbAEiJyrXyLiO6B6EJiuuAgABpMgCAALQFEA== Date: Fri, 16 Dec 2016 12:22:52 +0000 Message-ID: <94479800C636CB44BD422CB454846E01315717A4@SHSMSX101.ccr.corp.intel.com> References: <2d37e8ee1c613557e94acef94e381b72b2a535ce.1479309720.git.adrien.mazarguil@6wind.com> <188971FCDA171749BED5DA74ABF3E6F03B6625B9@shsmsx102.ccr.corp.intel.com> <20161216091746.GC10340@6wind.com> In-Reply-To: <20161216091746.GC10340@6wind.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: 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 12/22] app/testpmd: add rte_flow item spec handler 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: Fri, 16 Dec 2016 12:22:59 -0000 Thanks Adrien. I have two questions: 1. when I set " / vlan tci fix 10" with testpmd, I find the mask of tci is= 0xFFFF. Actually tci includes PRI, CFI, and Vlan_id which holds 12 bits, so is= it possible to set the mask to 0xFFF?=20 Our driver will check the mask only covers vlan_id instead of the whol= e tci. 2. When we test destroy function, we find the pointer provided to PMD is NU= LL instead of the pointer PMD returned to RTE during creating flow. Could = you please have double check? Thanks. Best Regards Beilei > -----Original Message----- > From: Adrien Mazarguil [mailto:adrien.mazarguil@6wind.com] > Sent: Friday, December 16, 2016 5:18 PM > To: Pei, Yulong > Cc: dev@dpdk.org; Thomas Monjalon ; De > Lara Guarch, Pablo ; Olivier Matz > ; Xing, Beilei > Subject: Re: [dpdk-dev] [PATCH 12/22] app/testpmd: add rte_flow item spec > handler >=20 > Hi Yulong, >=20 > On Fri, Dec 16, 2016 at 03:01:15AM +0000, Pei, Yulong wrote: > > Hi Adrien, > > > > I try to setup the following rule, but it seems that after set 'spec' = param, > can not set 'mask' param, is it an issue here or am I wrong to use it ? > > > > testpmd> flow create 0 ingress pattern eth dst spec 00:00:00:00:09:00 > > dst [TOKEN]: destination MAC > > src [TOKEN]: source MAC > > type [TOKEN]: EtherType > > / [TOKEN]: specify next pattern item >=20 > You need to re-specify dst with "mask" instead of "spec". You can specify= it > as many times you like to update each structure in turn, e.g.: >=20 > testpmd> flow create 0 ingress pattern eth dst spec 00:00:00:00:09:00 ds= t > mask 00:00:00:00:ff:ff >=20 > If you want to specify both spec and mask at once assuming you want it fu= ll, > these commands yield the same result: >=20 > testpmd> flow create 0 ingress pattern eth dst fix 00:00:00:00:09:00 te= stpmd> > flow create 0 ingress pattern eth dst spec 00:00:00:00:09:00 dst mask > ff:ff:ff:ff:ff:ff testpmd> flow create 0 ingress pattern eth dst spec > 00:00:00:00:09:00 dst prefix 48 >=20 > You are even allowed to change your mind: >=20 > testpmd> flow create 0 ingress pattern eth dst fix 00:00:2a:2a:2a:2a dst= fix > 00:00:00:00:09:00 >=20 > All these will be properly documented in the v2 patchset. Note, this vers= ion > will replace the "fix" keyword with "is" ("fix" made no sense according t= o > feedback). >=20 > -- > Adrien Mazarguil > 6WIND