From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id F1AC5A052A; Tue, 26 Jan 2021 04:36:37 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C5B8E1411EB; Tue, 26 Jan 2021 04:36:36 +0100 (CET) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mails.dpdk.org (Postfix) with ESMTP id D73DC1411E7; Tue, 26 Jan 2021 04:36:33 +0100 (CET) IronPort-SDR: bSu+eCllV05ojyVBn3LhB+pHmSwIXTiiEZaImSkM+EUWyBGRZqfAaHsOB5JiXi4DRzGp6RLJ2Y XN8tolVQsqLg== X-IronPort-AV: E=McAfee;i="6000,8403,9875"; a="179924709" X-IronPort-AV: E=Sophos;i="5.79,375,1602572400"; d="scan'208";a="179924709" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Jan 2021 19:36:32 -0800 IronPort-SDR: HXT5TNfCyUSncBmomrzQ2gXIn/97KN3EDWt4IVfwGa+P3aNM57q7anRvn5L/ff+OsvArpzOzFM VqsDTE/n7QYA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.79,375,1602572400"; d="scan'208";a="393621364" Received: from fmsmsx601.amr.corp.intel.com ([10.18.126.81]) by orsmga007.jf.intel.com with ESMTP; 25 Jan 2021 19:36:32 -0800 Received: from shsmsx606.ccr.corp.intel.com (10.109.6.216) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2106.2; Mon, 25 Jan 2021 19:36:31 -0800 Received: from shsmsx601.ccr.corp.intel.com (10.109.6.141) by SHSMSX606.ccr.corp.intel.com (10.109.6.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Tue, 26 Jan 2021 11:36:29 +0800 Received: from shsmsx601.ccr.corp.intel.com ([10.109.6.141]) by SHSMSX601.ccr.corp.intel.com ([10.109.6.141]) with mapi id 15.01.1713.004; Tue, 26 Jan 2021 11:36:29 +0800 From: "Guo, Jia" To: "Yu, DapengX" , "Wang, Haiyue" , "Yang, Qiming" CC: "dev@dpdk.org" , "Yu, DapengX" , "stable@dpdk.org" Thread-Topic: [PATCH v3] net/ixgbe: disable NFS filtering Thread-Index: AQHW84/MmvMAxy/L9U2SRh3HlLK8l6o5Qa5Q Date: Tue, 26 Jan 2021 03:36:29 +0000 Message-ID: <11023810106f47adbcd2186cc810753a@intel.com> References: <20210125031041.429268-1-dapengx.yu@intel.com> <20210126030308.517216-1-dapengx.yu@intel.com> In-Reply-To: <20210126030308.517216-1-dapengx.yu@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-reaction: no-action dlp-version: 11.5.1.3 x-originating-ip: [10.239.127.36] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v3] net/ixgbe: disable NFS filtering X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Acked-by: Jeff Guo > -----Original Message----- > From: dapengx.yu@intel.com > Sent: Tuesday, January 26, 2021 11:03 AM > To: Guo, Jia ; Wang, Haiyue ; > Yang, Qiming > Cc: dev@dpdk.org; Yu, DapengX ; stable@dpdk.org > Subject: [PATCH v3] net/ixgbe: disable NFS filtering >=20 > From: Dapeng Yu >=20 > Disable NFS header filtering whether NFS packets coalescing are required = or > not, in order to make RSS can work on NFS packets. >=20 > The code without the patch does follow datasheet, but not consistent with > the ixgbe kernel driver. It causes NFS packets to be filtered and make th= em > flow into queue 0, before RSS can work on them. >=20 > Fixes: b826efba6de4 ("net/ixgbe: align register setting when RSC is disab= led") > Fixes: 8eecb3295aed ("ixgbe: add LRO support") > Cc: stable@dpdk.org >=20 > Signed-off-by: Dapeng Yu > --- > drivers/net/ixgbe/ixgbe_rxtx.c | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) >=20 > diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxt= x.c > index cc8f70e6d..2efd054f7 100644 > --- a/drivers/net/ixgbe/ixgbe_rxtx.c > +++ b/drivers/net/ixgbe/ixgbe_rxtx.c > @@ -4923,15 +4923,11 @@ ixgbe_set_rsc(struct rte_eth_dev *dev) > /* RFCTL configuration */ > rfctl =3D IXGBE_READ_REG(hw, IXGBE_RFCTL); > if ((rsc_capable) && (rx_conf->offloads & > DEV_RX_OFFLOAD_TCP_LRO)) > - /* > - * Since NFS packets coalescing is not supported - clear > - * RFCTL.NFSW_DIS and RFCTL.NFSR_DIS when RSC is > - * enabled. > - */ > - rfctl &=3D ~(IXGBE_RFCTL_RSC_DIS | IXGBE_RFCTL_NFSW_DIS > | > - IXGBE_RFCTL_NFSR_DIS); > + rfctl &=3D ~IXGBE_RFCTL_RSC_DIS; > else > rfctl |=3D IXGBE_RFCTL_RSC_DIS; > + /* disable NFS filtering */ > + rfctl |=3D IXGBE_RFCTL_NFSW_DIS | IXGBE_RFCTL_NFSR_DIS; > IXGBE_WRITE_REG(hw, IXGBE_RFCTL, rfctl); >=20 > /* If LRO hasn't been requested - we are done here. */ > -- > 2.27.0