From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 9F494594D for ; Thu, 24 Jul 2014 10:22:30 +0200 (CEST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP; 24 Jul 2014 01:23:56 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.01,722,1400050800"; d="scan'208";a="574675312" Received: from fmsmsx107.amr.corp.intel.com ([10.19.9.54]) by fmsmga002.fm.intel.com with ESMTP; 24 Jul 2014 01:23:48 -0700 Received: from fmsmsx152.amr.corp.intel.com (10.19.17.221) by FMSMSX107.amr.corp.intel.com (10.19.9.54) with Microsoft SMTP Server (TLS) id 14.3.123.3; Thu, 24 Jul 2014 01:23:48 -0700 Received: from shsmsx102.ccr.corp.intel.com (10.239.4.154) by fmsmsx152.amr.corp.intel.com (10.19.17.221) with Microsoft SMTP Server (TLS) id 14.3.123.3; Thu, 24 Jul 2014 01:23:48 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.204]) by shsmsx102.ccr.corp.intel.com ([169.254.2.120]) with mapi id 14.03.0123.003; Thu, 24 Jul 2014 16:23:46 +0800 From: "Zhang, Helin" To: Thomas Monjalon Thread-Topic: [dpdk-dev] [PATCH 1/5] ethdev: Rename macros of packet classification type Thread-Index: AQHPpxO7hZbuZpxe6Uy2+vEfk1yc85uu3+ng//974ICAAIZdYA== Date: Thu, 24 Jul 2014 08:23:46 +0000 Message-ID: References: <1406184149-11531-1-git-send-email-helin.zhang@intel.com> <2490690.EmSZ5vBN3i@xps13> <5763244.xFk97hyylc@xps13> In-Reply-To: <5763244.xFk97hyylc@xps13> 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 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH 1/5] ethdev: Rename macros of packet classification type X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Jul 2014 08:22:31 -0000 > -----Original Message----- > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > Sent: Thursday, July 24, 2014 4:20 PM > To: Zhang, Helin > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/5] ethdev: Rename macros of packet > classification type >=20 > 2014-07-24 08:14, Zhang, Helin: > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > > 2014-07-24 14:42, Helin Zhang: > > > > For better understanding, 'PCTYPE' was added to the name of i40e > > > > RSS shift macros. > > > > > > > > Signed-off-by: Helin Zhang > > > > > > > -#define ETH_RSS_NONF_IPV4_UDP_SHIFT 31 > > > > +#define ETH_PCTYPE_NONF_IPV4_UDP 31 > > > > > > Why is it clearer? I don't understand what means PCTYPE. > > > > It is 'packet classification type' defined in data sheet, and widely us= ed. >=20 > Widely used? No it seems to be an i40e naming. > Which datasheet are you pointing? Sorry, it is widely used in i40e. That might not so widely for the whole DP= DK. The datasheet I mentioned is the Intel(r) 40G NIC datasheet. >=20 > > It is not just for RSS only, it can be used for flow director possibly. > > That's why I think it would be better to rename it with PCTYPE names. >=20 > At least, you should add a comment in the code to explain the meaning. OK, good comments! >=20 > Thank you > -- > Thomas