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 B11EFA0A0B for ; Tue, 26 Jan 2021 03:11:27 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A1032141156; Tue, 26 Jan 2021 03:11:27 +0100 (CET) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mails.dpdk.org (Postfix) with ESMTP id 0F3D514103A; Tue, 26 Jan 2021 03:11:23 +0100 (CET) IronPort-SDR: IsxCVtdVN9IxbaNyaceLZh3Ee3Q5Yl+CW3OzNXmRex5o3ADsLzYpgdgxj9NmyISWjfJwzxugeR PtmYtREy62iw== X-IronPort-AV: E=McAfee;i="6000,8403,9875"; a="177268044" X-IronPort-AV: E=Sophos;i="5.79,375,1602572400"; d="scan'208";a="177268044" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Jan 2021 18:11:23 -0800 IronPort-SDR: Dfo8pa8dIIitjTdcq7CH/RSmH+CDw1LBfEpWtpADiRm+8ckw2Toe8D9y+/iYwY3xMgOnDj+DCj q69w9s+Ql23w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.79,375,1602572400"; d="scan'208";a="429524500" Received: from fmsmsx603.amr.corp.intel.com ([10.18.126.83]) by orsmga001.jf.intel.com with ESMTP; 25 Jan 2021 18:11:22 -0800 Received: from shsmsx604.ccr.corp.intel.com (10.109.6.214) by fmsmsx603.amr.corp.intel.com (10.18.126.83) 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 18:11:21 -0800 Received: from shsmsx601.ccr.corp.intel.com (10.109.6.141) by SHSMSX604.ccr.corp.intel.com (10.109.6.214) 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 10:11:20 +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 10:11:20 +0800 From: "Guo, Jia" To: "Yu, DapengX" , "Wang, Haiyue" , "Yang, Qiming" CC: "dev@dpdk.org" , "Yu, DapengX" , "stable@dpdk.org" Thread-Topic: [PATCH v2] net/ixgbe: disable NFS filtering Thread-Index: AQHW8seyVsF0IgQxZUawYLJEdPdCqao5KjUQ Date: Tue, 26 Jan 2021 02:11:19 +0000 Message-ID: <84f0e4cfc8064a5988c0c814f3816c96@intel.com> References: <20210120092700.105207-1-dapengx.yu@intel.com> <20210125031041.429268-1-dapengx.yu@intel.com> In-Reply-To: <20210125031041.429268-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-stable] [PATCH v2] net/ixgbe: disable NFS filtering X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, dapeng > -----Original Message----- > From: dapengx.yu@intel.com > Sent: Monday, January 25, 2021 11:11 AM > To: Guo, Jia ; Wang, Haiyue ; > Yang, Qiming > Cc: dev@dpdk.org; Yu, DapengX ; stable@dpdk.org > Subject: [PATCH v2] net/ixgbe: disable NFS filtering >=20 > From: Dapeng Yu >=20 > Disable NFS header filtering whether NFS packets coalescing are required = or > not. >=20 > This behavior is aligned with ixgbe kernel driver. >=20 I saw your comments about the reason in another email, so could you briefly= add the reason in the commit log? You just said what you do with this patch, but why or say without this patc= h what is the problem, it it better be mentioned if this is a fixing patch. Other is LGTM. > 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