From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v3 0/3] ethdev: add IP address and TCP/UDP port rewrite actions to flow API Date: Tue, 9 Oct 2018 14:04:39 +0100 Message-ID: <45232613-fa5a-2353-2327-1b0a15ffce6d@intel.com> References: <1538840520-10369-1-git-send-email-rahul.lakkireddy@chelsio.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: indranil@chelsio.com, nirranjan@chelsio.com To: Rahul Lakkireddy , dev@dpdk.org Return-path: Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by dpdk.org (Postfix) with ESMTP id AD1A15F24 for ; Tue, 9 Oct 2018 15:04:47 +0200 (CEST) In-Reply-To: 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 10/9/2018 9:44 AM, Rahul Lakkireddy wrote: > This series of patches add support for actions: > - SET_IPV4_SRC - set a new IPv4 source address. > - SET_IPV4_DST - set a new IPv4 destination address. > - SET_IPV6_SRC - set a new IPv6 source address. > - SET_IPV6_DST - set a new IPv6 destination address. > - SET_TP_SRC - set a new TCP/UDP source port number. > - SET_TP_DST - set a new TCP/UDP destination port number. > > These actions are useful in Network Address Translation use case > to edit IP address and TCP/UDP port numbers before switching > the packets out to the destination device port. > > Patch 1 adds support for IP address rewrite to rte_flow and testpmd. > > Patch 2 adds support for TCP/UDP port rewrite to rte_flow and testpmd. > > Patch 3 shows CXGBE PMD example to offload these actions to hardware. > > Feedback and suggestions will be much appreciated. > > Thanks, > Rahul > > RFC v1: http://mails.dpdk.org/archives/dev/2018-June/104913.html > RFC v2: http://mails.dpdk.org/archives/dev/2018-August/109672.html > > --- > v3: > - Replaced uint32_t with rte_be32_t to reflect the byte order of > the new IPv4 addresses to rewrite. > - Replaced uint16_t with rte_be16_t to reflect the byte order of > the new TCP/UDP port addresses to rewrite. > > v2: > - Rebased to tip. > - Removed adding actions to app/test-pmd/config.c, to sync with > rte_flow_conv() changes. > > v1 changes since RFC v2: > - Updated comments, help messages, and doc to indicate that IP/TCP/UDP > of the outermost headers are modified. > - Updated comments and doc to indicate that a corresponding valid flow > pattern item must be specified to offload corresponding header rewrite > actions. > - Re-based CXGBE PMD changes in patch 3 to tip. > - Updated all instances of fw_filter_wr to new fw_filter2_wr and removed > fw_filter_wr. > - Ensure correct ULP type is set when offloading NAT actions. > - Returning appropriate RTE_FLOW_ERROR_TYPE_ACTION error if a corresponding > valid flow pattern item is not found. > - Updated release notes. > > > Rahul Lakkireddy (3): > ethdev: add flow api actions to modify IP addresses > ethdev: add flow api actions to modify TCP/UDP port numbers > net/cxgbe: add flow actions to modify IP and TCP/UDP port address Series applied to dpdk-next-net/master, thanks. (modification mentioned on v3 3/3 has been applied while merging)