From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id ADB8BC04AAF for ; Sat, 18 May 2019 15:21:07 +0000 (UTC) Received: from dpdk.org (dpdk.org [92.243.14.124]) by mail.kernel.org (Postfix) with ESMTP id 4FAEA2087E for ; Sat, 18 May 2019 15:21:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4FAEA2087E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=dev-bounces@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1DCCA2B9D; Sat, 18 May 2019 17:21:06 +0200 (CEST) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 5FC8E2082 for ; Sat, 18 May 2019 17:21:04 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 18 May 2019 08:21:03 -0700 X-ExtLoop1: 1 Received: from irsmsx102.ger.corp.intel.com ([163.33.3.155]) by orsmga007.jf.intel.com with ESMTP; 18 May 2019 08:21:02 -0700 Received: from irsmsx105.ger.corp.intel.com ([169.254.7.155]) by IRSMSX102.ger.corp.intel.com ([169.254.2.108]) with mapi id 14.03.0415.000; Sat, 18 May 2019 16:21:01 +0100 From: "Ananyev, Konstantin" To: Sunil Kumar Kori , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH v2 2/3] examples/ip_fragmentation: Enabling IP checksum offload in mbuf Thread-Index: AQHVC9yHnZOx+NpTukme1A79+JRUkaZxAk9w Date: Sat, 18 May 2019 15:21:01 +0000 Message-ID: <2601191342CEEE43887BDE71AB97725801616357D4@irsmsx105.ger.corp.intel.com> References: <1558005895-30340-1-git-send-email-skori@marvell.com> <1558006933-32247-1-git-send-email-skori@marvell.com> <1558006933-32247-3-git-send-email-skori@marvell.com> In-Reply-To: <1558006933-32247-3-git-send-email-skori@marvell.com> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYmViOTMxZTQtMTNiOC00YTk3LWJmMTAtNmVhOWU0YTY0Mzg1IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoicENpbzA0dDhBeFBWbFBtaVdteUVlaGFWa3p5eTljbm1NTlV5WjZCdlRZdjRqSVZjaW4yOWMzU1k3TU9LczVycSJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-originating-ip: [163.33.239.181] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2 2/3] examples/ip_fragmentation: Enabling IP checksum offload in mbuf X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 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" > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Sunil Kumar Kori > Sent: Thursday, May 16, 2019 12:42 PM > To: dev@dpdk.org > Cc: skori@marvell.com > Subject: [dpdk-dev] [PATCH v2 2/3] examples/ip_fragmentation: Enabling IP= checksum offload in mbuf >=20 > As per the documentation to use any IP offload features, application > must set required offload flags into mbuf->ol_flags. >=20 > Signed-off-by: Sunil Kumar Kori > --- > examples/ip_fragmentation/main.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) >=20 > diff --git a/examples/ip_fragmentation/main.c b/examples/ip_fragmentation= /main.c > index e90a61e..d821967 100644 > --- a/examples/ip_fragmentation/main.c > +++ b/examples/ip_fragmentation/main.c > @@ -354,10 +354,13 @@ struct rte_lpm6_config lpm6_config =3D { >=20 > /* src addr */ > ether_addr_copy(&ports_eth_addr[port_out], ð_hdr->s_addr); > - if (ipv6) > + if (ipv6) { > eth_hdr->ether_type =3D rte_be_to_cpu_16(ETHER_TYPE_IPv6); > - else > + m->ol_flags |=3D PKT_TX_IPV6; Is there is any point to do that? This sample app, as I remember doesn't request any extra HW offloads. Same comment for patch #3. Konstantin > + } else { > eth_hdr->ether_type =3D rte_be_to_cpu_16(ETHER_TYPE_IPv4); > + m->ol_flags |=3D (PKT_TX_IPV4 | PKT_TX_IP_CKSUM); > + } > } >=20 > len +=3D len2; > -- > 1.8.3.1