From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v8 2/3] net/i40e: configurable PTYPE mapping Date: Wed, 12 Apr 2017 09:36:42 +0100 Message-ID: References: <20170404015516.7784-1-qi.z.zhang@intel.com> <20170407032505.61691-1-qi.z.zhang@intel.com> <20170407032505.61691-3-qi.z.zhang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: jianbo.liu@linaro.org, chaozhu@linux.vnet.ibm.com, dev@dpdk.org To: Qi Zhang , jingjing.wu@intel.com, helin.zhang@intel.com Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 05DB52C6B for ; Wed, 12 Apr 2017 10:36:45 +0200 (CEST) In-Reply-To: <20170407032505.61691-3-qi.z.zhang@intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 4/7/2017 4:25 AM, Qi Zhang wrote: > The patch adds 4 APIs to support configurable > PTYPE mapping for i40e device. > rte_pmd_i40e_ptype_mapping_get. > rte_pmd_i40e_ptype_mapping_replace. > rte_pmd_i40e_ptype_mapping_reset. > rte_pmd_i40e_ptype_mapping_update. > The mapping from hardware defined packet type to software defined packet > type can be updated/reset/read out with these APIs. > Also a software ptype with the most significent bit set will be regarded > as a user defined ptype (RTE_PMD_I40E_PTYPE_USER_DEFINE_MASK) so > application can use it to defined its own PTYPE naming system. > > Signed-off-by: Qi Zhang > --- > > v5: > > - Rebase to dpdk-next-net > > v4: > > - Add comment to explain the "user defined" ptype. > > v2: > > - Rename APIs to rte_pmd_i40e_ptype_mapping_xxx > > - Add missing API declaration in rte_pmd_i40e_version.map > > - Add missing PTYPE in check_invalid_pkt_type > > - Fix couple typo error > > drivers/net/i40e/i40e_ethdev.c | 229 ++++++++++++++++++++++++++++++ Hi Qi, These functions now needs to be moved to "rte_pmd_i40e.c". Can you please rebase patch on top of latest next-net? Thanks, ferruh