From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: Remaining old offloading API usage in PMDs Date: Mon, 21 May 2018 15:33:56 +0100 Message-ID: <2cb2301a-88f5-cfdd-1c38-f2d41ed56c7d@intel.com> References: <9a1b4500-d354-bdef-45d3-7d6eeacfb54b@intel.com> <8fe6b45b-7fb3-0551-39ec-02ba94ef16cb@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: Thomas Monjalon , Andrew Rybchenko , Tetsuya Mukawa , Maxime Coquelin , Hemant Agrawal , Shreyansh Jain , dpdk-dev , Beilei Xing , Qi Zhang To: Alejandro Lucero Return-path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id D08261B1CC for ; Mon, 21 May 2018 16:34:00 +0200 (CEST) In-Reply-To: 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 5/21/2018 3:16 PM, Alejandro Lucero wrote: > I completely missed this last bit in the NFP driver. > > I will send a patch today. > > By the way, will it go into 18.05? I ask because I have another patch fixing a > problem which would be good to have in 18.05 as well. Release is still open for fixes, specially for PMDs since their scope is limited. After rc5 plan is to get only critical fixes. > > On Mon, May 21, 2018 at 1:38 PM, Ferruh Yigit > wrote: > > On 5/11/2018 11:45 AM, Ferruh Yigit wrote: > > Some PMDs are still using old offloading API related variables, thanks to > Andrew > > for reporting it. > > > > Maintainers of related PMDs are cc'ed. > > Please check following list and provide a fix ASAP for it, thanks. > > > > > > # git grep header_split drivers/ > > drivers/net/i40e/i40e_rxtx_vec_common.h:        if (rxmode->header_split == 1) > > > > # git grep hw_vlan_filter drivers/ > > drivers/net/i40e/i40e_vf_representor.c:         if > > (ethdev->data->dev_conf.rxmode.hw_vlan_filter) > > drivers/net/vhost/rte_eth_vhost.c:      if (rxmode->hw_vlan_filter) > > > > # git grep hw_vlan_strip drivers/ > > drivers/net/dpaa2/dpaa2_rxtx.c:         if > > (dev->data->dev_conf.rxmode.hw_vlan_strip) > > drivers/net/i40e/i40e_vf_representor.c:         if > > (ethdev->data->dev_conf.rxmode.hw_vlan_strip)T > > drivers/net/vhost/rte_eth_vhost.c:      internal->vlan_strip = > > rxmode->hw_vlan_strip; > > > > # git grep hw_vlan_extend drivers/ > > drivers/net/i40e/i40e_rxtx_vec_common.h:        if (rxmode->hw_vlan_extend > == 1) > > > > # git grep hw_strip_crc drivers/ > > drivers/net/nfp/nfp_net.c:      rxq->crc_len = (uint8_t) > > ((dev->data->dev_conf.rxmode.hw_strip_crc) ? 0 > > drivers/net/sfc/sfc_rx.c:               rxmode->hw_strip_crc = 1; > > > > Reminder of this one. > >