From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Iremonger, Bernard" Subject: Re: [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition Date: Wed, 3 Oct 2018 13:00:20 +0000 Message-ID: <8CEF83825BEC744B83065625E567D7C260D0C16E@IRSMSX107.ger.corp.intel.com> References: <20180913134707.23698-1-jerin.jacob@caviumnetworks.com> <20181002192451.19119-1-jerin.jacob@caviumnetworks.com> <8CEF83825BEC744B83065625E567D7C260D0C0C7@IRSMSX107.ger.corp.intel.com> <20181003111901.GB9248@jerin> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "Lu, Wenzhuo" , "Wu, Jingjing" , "Mcnamara, John" , "Kovacevic, Marko" , Thomas Monjalon , "Yigit, Ferruh" , "Andrew Rybchenko" , Olivier Matz , "dev@dpdk.org" , "shahafs@mellanox.com" To: Jerin Jacob Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 0B4231B16A for ; Wed, 3 Oct 2018 15:00:23 +0200 (CEST) In-Reply-To: <20181003111901.GB9248@jerin> 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" Hi Jerin, > > > Subject: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > > > checksum definition > > > > > > Introduced DEV_RX_OFFLOAD_OUTER_UDP_CKSUM Rx offload flag and > > > PKT_RX_EL4_CKSUM_BAD mbuf ol_flags to detect outer UDP checksum > failure. > > > > > > - To use hardware Rx outer UDP checksum offload, the user needs to > > > configure DEV_RX_OFFLOAD_OUTER_UDP_CKSUM offload flags in > slowpath. > > > > > > - Driver updates the PKT_RX_EL4_CKSUM_BAD mbuf ol_flag on checksum > > > failure similar to the outer L3 PKT_RX_EIP_CKSUM_BAD flag. > > > > > > Signed-off-by: Jerin Jacob > > > --- > > > > > > v2: > > > - Removed DEV_RX_OFFLOAD_OUTER_TCP_CKSUM and > > > DEV_RX_OFFLOAD_OUTER_SCTP_CKSUM as there is no realworld use case > > > for it. > > > See: http://patches.dpdk.org/patch/44692/ > > > > > > This patch series is depended on > > > http://patches.dpdk.org/patch/45840/ > > > > > > -- > > > app/test-pmd/config.c | 9 +++++++++ > > > doc/guides/nics/features.rst | 3 +++ > > > lib/librte_ethdev/rte_ethdev.c | 1 + > > > lib/librte_ethdev/rte_ethdev.h | 1 + > > > lib/librte_mbuf/rte_mbuf.c | 2 ++ > > > lib/librte_mbuf/rte_mbuf.h | 3 +++ > > > 6 files changed, 19 insertions(+) > > > > > > > > > > > This patch fails to apply to the latest master branch, a rebase may be = needed. >=20 >=20 > Have you applies the depended series http://patches.dpdk.org/patch/45840/ > as mentioned above? If yes and then still has issue then let me know. >=20 The patches apply and build fine with the above patch applied first. Regards, Bernard.