From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f175.google.com (mail-wr0-f175.google.com [209.85.128.175]) by dpdk.org (Postfix) with ESMTP id B161C7DEC for ; Wed, 25 Apr 2018 17:27:51 +0200 (CEST) Received: by mail-wr0-f175.google.com with SMTP id d1-v6so53711654wrj.13 for ; Wed, 25 Apr 2018 08:27:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=5vd0o5pQl/LlAogKE2SC9Hw6ItL+gedGwYZlvuLMhaM=; b=YJriygqeEcfv7xTcBOnO+IXGtpQ1YLzVtBByscgJeC+iRE0RL9TLuyaS58aDsZ1Exo YZYKH9dH0yVYR5rS4UU0UqSj2/UGJJUNzTzjGw2pX+iIhjcOR5bHBpGQp+EjqrrBZWj9 mk+Ba5BcONRbtI0oJPxYQ2e47dQ7rs+qQ46lzY6ePzdjYXiYtSdWorUgTY841kTy9Lz1 zOF5HcyIA8FDfeN1lRBf4ByTyrTICOBew/PDozoVoDWck87j/yByjZFbzjgqxAb8W+q6 UI5x3wM4/BQ/DG5qHmLjFvpo/8tLpMvH6XjHpIyKqxBrioP34LwuDevqT4Oj7j0zlhe+ 02EA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=5vd0o5pQl/LlAogKE2SC9Hw6ItL+gedGwYZlvuLMhaM=; b=TTkfFGLPKHUnV7Lq6GiHlW5kEjGrNGbHLoYts+6SjAYHroyZ5ygyI2FlReLSRJgYCF 2TN5vnXXXJbM5tPZjURBlEzzVWDZ/xpkd/cf7Fs1QGbMu/a2fy0TOw0bDMhS0PFO4vEG GCIV0bZFlmYi4CQ1u3V4+kNPGeiF9dH1iQ936geKTPHYHK8eTPXQ2+xToe42AN40vfnM 75dUXUX9yBT/w0/2vqUKQIomANfEnd20QEJ+R9JVPouM0BjVE1byceIi4fEvGba1BibD 6qL0pa36w+5DTQf21roMBbp+ig8H5DaMwwdOFgiCQtsH7A1dpouo5W7gQHFCqeaf1gNa AKAA== X-Gm-Message-State: ALQs6tCZL9dWs5btQn5YNF5H3pYuOgRszbt/sQaYXD4PrWmDVajNxv47 FOC3931jTZ130XOkZ73wC+uJjQ== X-Google-Smtp-Source: AIpwx49ooj0W4+A5PT7BEsajrdMRnTOjHMCmm1TxqYow6gmn3McAIqPIR6NEgcSaOkhnQDhm12u3Rw== X-Received: by 2002:adf:c5d1:: with SMTP id v17-v6mr24959948wrg.45.1524670071387; Wed, 25 Apr 2018 08:27:51 -0700 (PDT) Received: from 6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id y45-v6sm19665116wrd.97.2018.04.25.08.27.50 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 25 Apr 2018 08:27:50 -0700 (PDT) Date: Wed, 25 Apr 2018 17:27:36 +0200 From: Adrien Mazarguil To: Thomas Monjalon , Ferruh Yigit , dev@dpdk.org Message-ID: <20180425151852.7676-1-adrien.mazarguil@6wind.com> References: <20180419100848.6178-1-adrien.mazarguil@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180419100848.6178-1-adrien.mazarguil@6wind.com> X-Mailer: git-send-email 2.11.0 Subject: [dpdk-dev] [PATCH v6 00/16] Flow API overhaul for switch offloads 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: , X-List-Received-Date: Wed, 25 Apr 2018 15:27:51 -0000 As summarized in a prior RFC [1], the flow API (rte_flow) was chosen as a means to manage switch offloads supported by many devices (usually going by names such as E-Switch or vSwitch) through user-specified flow rules. Combined with the need to support encap/decap actions, this requires a change in the way flow actions are processed (in order and possibly repeated) which modifies the behavior of some of the existing actions, thus warranting a major ABI breakage. Given this ABI breakage is also required by other work submitted for the current release [2][3], this series addresses various longstanding issues with the flow API and makes minor improvements in preparation for upcoming features. Changes summary: - Additional error types. - Clearer documentation. - Improved C++ compatibility. - Exhaustive RSS action. - Consistent behavior of VLAN pattern item. - New "transfer" attribute bringing consistency to VF/PF pattern items. - Confusing "PORT" pattern item renamed "PHY_PORT", with new action counterpart. - New "PORT_ID" pattern item and action to be used with port representors. This series piggybacks on the major ABI update introduced by a prior commit [4] for DPDK 18.05 and depends on several fixes [5] which must be applied first. [1] "[RFC] Switch device offload with DPDK" http://dpdk.org/ml/archives/dev/2018-March/092513.html [2] commit 676b605182a5 ("doc: announce ethdev API change for RSS configuration") [3] "[PATCH v1 00/21] MLX5 tunnel Rx offloading" http://dpdk.org/ml/archives/dev/2018-March/092264.html [4] commit 653e038efc9b ("ethdev: remove versioning of filter control function") [5] "[PATCH v6 00/11] Bunch of flow API-related fixes" http://dpdk.org/ml/archives/dev/2018-April/098035.html v6 changes: - Fixed mlx5 issue raised by Nelio in "ethdev: flatten RSS configuration in flow API". - Updated release notes (API update / ABI breakage) in relevant patches. - Removed Xueming's deprecation notice in "ethdev: add encap level to RSS flow API action" since it's covered by this series. - Reworded a few patches as fixes since they address API flaws. - Rebased series once again. v5 changes: - Fixed errors reported by GCC and Clang in patch 05/16 ("ethdev: alter behavior of flow API actions"). - Rebased series once again. v4 changes: - No change besides new acked-by lines, rebased series to address conflicts. v3 changes: - Rebased series, fixed latest conflicts. - Addressed Andrew's comments, see affected patches for details: - Empty RSS types in flow rule means PMD-specific RSS instead of no RSS. - RSS hash function now explicitly compared against RTE_ETH_HASH_FUNCTION_DEFAULT instead of 0 in all PMDs. - sfc PMD updated to also accept Toeplitz. - Implicit VLAN TPID matching now removed from all PMDs. - Default mask upate for VLAN TCI now split as separate patch #11. - Ingress/egress definition clarified in patch #12. v2 changes: - Squashed "ethdev: update ABI for flow API functions" in subsequent patches. - Emphasized ABI impact in relevant commit logs. - Modified documentation in "ethdev: alter behavior of flow API actions" to describe how terminating flow rules without any action of the fate kind result in undefined behavior instead of dropping traffic. - Fixed other minor documentation formatting issues. - Modified "ethdev: refine TPID handling in flow API" as follows: - Using standard macro definitions for VLAN, QinQ and E-Tag EtherTypes. - Fixed endian conversion in sfc. - Replaced a condition in VLAN pattern item processing with an assertion check for i40e. Adrien Mazarguil (16): ethdev: add error types to flow API ethdev: clarify flow API pattern items and actions doc: remove flow API migration section ethdev: remove DUP action from flow API ethdev: alter behavior of flow API actions ethdev: fix C99 flexible arrays from flow API ethdev: flatten RSS configuration in flow API ethdev: add hash function to RSS flow API action ethdev: add encap level to RSS flow API action ethdev: fix TPID handling in flow API ethdev: fix default VLAN TCI mask in flow API ethdev: add transfer attribute to flow API ethdev: fix behavior of VF/PF in flow API ethdev: rename physical port item in flow API ethdev: add physical port action to flow API ethdev: add port ID item and action to flow API app/test-pmd/cmdline_flow.c | 394 +++++++++++---- app/test-pmd/config.c | 78 +-- doc/guides/nics/tap.rst | 2 +- doc/guides/prog_guide/rte_flow.rst | 618 ++++++++--------------- doc/guides/rel_notes/deprecation.rst | 4 - doc/guides/rel_notes/release_18_05.rst | 48 ++ doc/guides/testpmd_app_ug/testpmd_funcs.rst | 60 ++- drivers/net/bnxt/bnxt_filter.c | 49 +- drivers/net/e1000/e1000_ethdev.h | 13 +- drivers/net/e1000/igb_ethdev.c | 4 +- drivers/net/e1000/igb_flow.c | 83 ++- drivers/net/e1000/igb_rxtx.c | 55 +- drivers/net/enic/enic_flow.c | 50 +- drivers/net/i40e/i40e_ethdev.c | 57 ++- drivers/net/i40e/i40e_ethdev.h | 15 +- drivers/net/i40e/i40e_flow.c | 130 +++-- drivers/net/ixgbe/ixgbe_ethdev.c | 7 +- drivers/net/ixgbe/ixgbe_ethdev.h | 13 +- drivers/net/ixgbe/ixgbe_flow.c | 91 +++- drivers/net/ixgbe/ixgbe_rxtx.c | 55 +- drivers/net/mlx4/mlx4.c | 2 +- drivers/net/mlx4/mlx4_flow.c | 117 +++-- drivers/net/mlx4/mlx4_flow.h | 2 +- drivers/net/mlx4/mlx4_rxq.c | 2 +- drivers/net/mlx4/mlx4_rxtx.h | 2 +- drivers/net/mlx5/mlx5_flow.c | 316 ++++++------ drivers/net/mlx5/mlx5_rxq.c | 26 +- drivers/net/mlx5/mlx5_rxtx.h | 26 +- drivers/net/mvpp2/mrvl_flow.c | 32 +- drivers/net/sfc/sfc_flow.c | 78 ++- drivers/net/tap/tap_flow.c | 49 +- examples/ipsec-secgw/ipsec.c | 21 +- lib/librte_ether/rte_ethdev_version.map | 16 +- lib/librte_ether/rte_flow.c | 68 +-- lib/librte_ether/rte_flow.h | 339 ++++++++----- lib/librte_net/rte_ether.h | 1 + 36 files changed, 1796 insertions(+), 1127 deletions(-) -- 2.11.0