From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v2 0/3] ethdev: add generic L2/L3 tunnel encapsulation actions Date: Fri, 5 Oct 2018 13:57:28 +0100 Message-ID: <8b986b80-9320-2e25-6f2e-a7f3305cced2@intel.com> References: <1537116824-191205-1-git-send-email-orika@mellanox.com> <1537995646-95260-1-git-send-email-orika@mellanox.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org, dekelp@mellanox.com, thomas@monjalon.net, nelio.laranjeiro@6wind.com, yskoh@mellanox.com, shahafs@mellanox.com To: Ori Kam , arybchenko@solarflare.com, stephen@networkplumber.org, adrien.mazarguil@6wind.com Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 42D3E3977 for ; Fri, 5 Oct 2018 14:57:49 +0200 (CEST) In-Reply-To: <1537995646-95260-1-git-send-email-orika@mellanox.com> 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" On 9/26/2018 10:00 PM, Ori Kam wrote: > This series implement the generic L2/L3 tunnel encapsulation actions > and is based on rfc [1] "add generic L2/L3 tunnel encapsulation actions" > > Currenlty the encap/decap actions only support encapsulation > of VXLAN and NVGRE L2 packets (L2 encapsulation is where > the inner packet has a valid Ethernet header, while L3 encapsulation > is where the inner packet doesn't have the Ethernet header). > In addtion the parameter to to the encap action is a list of rte items, > this results in 2 extra translation, between the application to the action > and from the action to the NIC. This results in negetive impact on the > insertion performance. > > Looking forward there are going to be a need to support many more tunnel > encapsulations. For example MPLSoGRE, MPLSoUDP. > Adding the new encapsulation will result in duplication of code. > For example the code for handling NVGRE and VXLAN are exactly the same, > and each new tunnel will have the same exact structure. > > This series introduce a generic encapsulation for L2 tunnel types, and > generic encapsulation for L3 tunnel types. In addtion the new > encapsulations commands are using raw buffer inorder to save the > converstion time, both for the application and the PMD. > > [1]https://mails.dpdk.org/archives/dev/2018-August/109944.html > > Changes in v2: > > * add missing decap_l3 structure. > * fix typo. > > > > Ori Kam (3): > ethdev: add generic L2/L3 tunnel encapsulation actions > app/testpmd: convert testpmd encap commands to new API > ethdev: remove vxlan and nvgre encapsulation commands Hi Ori, Can you please rebase the patch on top of latest next-net? Thanks, ferruh