From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2] doc: announce ABI change for rte_eth_dev structure Date: Wed, 27 Jul 2016 01:59:01 -0700 (PDT) Message-ID: <2146153.nVzdynOqdk@xps13> References: <1469024691-58750-1-git-send-email-tomaszx.kulasek@intel.com> <1469114659-66063-1-git-send-email-tomaszx.kulasek@intel.com> <2601191342CEEE43887BDE71AB97725836B80AD8@irsmsx105.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, "Ananyev, Konstantin" To: "Kulasek, TomaszX" Return-path: Received: from mail-wm0-f43.google.com (mail-wm0-f43.google.com [74.125.82.43]) by dpdk.org (Postfix) with ESMTP id 5B98A5594 for ; Wed, 27 Jul 2016 10:59:02 +0200 (CEST) Received: by mail-wm0-f43.google.com with SMTP id q128so29723772wma.1 for ; Wed, 27 Jul 2016 01:59:02 -0700 (PDT) In-Reply-To: <2601191342CEEE43887BDE71AB97725836B80AD8@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" > > Signed-off-by: Tomasz Kulasek > > --- > > +* In 16.11 ABI changes are plained: the ``rte_eth_dev`` structure will be > > + extended with new function pointer ``tx_pkt_prep`` allowing verification > > + and processing of packet burst to meet HW specific requirements before > > + transmit. Also new fields will be added to the ``rte_eth_desc_lim`` structure: > > + ``nb_seg_max`` and ``nb_mtu_seg_max`` provideing information about number of > > + segments limit to be transmitted by device for TSO/non-TSO packets. > > Acked-by: Konstantin Ananyev I think I understand you want to split the TX processing: 1/ modify/write in mbufs 2/ write in HW and let application decide: - where the TX prep is done (which core) - what to do if the TX prep fail So adding some processing in this first part becomes "not too expensive" or "manageable" from the application point of view. If I well understand the intent, Acked-by: Thomas Monjalon (except typos ;)