From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v12 0/6] add Tx preparation Date: Wed, 30 Nov 2016 19:26:36 +0100 Message-ID: <3517413.XL3bTbAyaC@xps13> References: <1477486575-25148-1-git-send-email-tomaszx.kulasek@intel.com> <2601191342CEEE43887BDE71AB9772583F0E2444@irsmsx105.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Cc: Harish Patil , dev@dpdk.org, Rahul Lakkireddy , Stephen Hurd , Jan Medala , Jakub Palider , John Daley , Adrien Mazarguil , Alejandro Lucero , Rasesh Mody , "Jacob, Jerin" , Yuanhan Liu , Yong Wang , "Kulasek, TomaszX" , olivier.matz@6wind.com To: "Ananyev, Konstantin" Return-path: Received: from mail-wm0-f49.google.com (mail-wm0-f49.google.com [74.125.82.49]) by dpdk.org (Postfix) with ESMTP id B27EF3989 for ; Wed, 30 Nov 2016 19:26:38 +0100 (CET) Received: by mail-wm0-f49.google.com with SMTP id t79so225739907wmt.0 for ; Wed, 30 Nov 2016 10:26:38 -0800 (PST) In-Reply-To: <2601191342CEEE43887BDE71AB9772583F0E2444@irsmsx105.ger.corp.intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-11-30 17:42, Ananyev, Konstantin: > > >Please, we need a comment for each driver saying > > >"it is OK, we do not need any checksum preparation for TSO" > > >or > > >"yes we have to implement tx_prepare or TSO will not work in this = mode" > > > > >=20 > > qede PMD doesn=E2=80=99t currently support TSO yet, it only support= s Tx TCP/UDP/IP > > csum offloads. > > So Tx preparation isn=E2=80=99t applicable. So as of now - > > "it is OK, we do not need any checksum preparation for TSO" >=20 > Thanks for the answer. > Though please note that it not only for TSO. Oh yes, sorry, my wording was incorrect. We need to know if any checksum preparation is needed prior offloading its final computation to the hardware or driver. So the question applies to TSO and simple checksum offload. We are still waiting answers for =09bnxt, cxgbe, ena, nfp, thunderx, virtio and vmxnet3. > This is for any TX offload for which the upper layer SW would have > to modify the contents of the packet. > Though as I can see for qede neither PKT_TX_IP_CKSUM or PKT_TX_TCP_CK= SUM > exhibits any extra requirements for the user. > Is that correct?