From patchwork Thu Jan 26 16:18:56 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ferruh Yigit X-Patchwork-Id: 122572 Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id D443042493; Thu, 26 Jan 2023 17:19:18 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 862ED40223; Thu, 26 Jan 2023 17:19:18 +0100 (CET) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2075.outbound.protection.outlook.com [40.107.223.75]) by mails.dpdk.org (Postfix) with ESMTP id 52D2E400D7 for ; Thu, 26 Jan 2023 17:19:17 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZTN5vQ1UqcDBcrrDn2jwojyRT4nVG3lBzJQ0pKoq39tRBJEoVlCBIQeCDOu13BoAcf+3QPEa+hpMg6jNhpzW9x7Sc5kFFEHixO4KAuxtlDb284AhfM4yM4rpOwj39aHcJRkderqic2NieGjKSaM+Zn/dY/ian3VIiE7gyoZ18fUIWqunXeATVoVrXJ99qgup5xJ0DE85s+upUPrCxDMQgNAQrsewPWgvPNaPNnjrXFJyoBDTpCMf0LBCMFQv0R9IvZhzXUnA1+y8u4h9kYwI5NcGjRIqdJC8sGvcBEhoypZpRzGgLz4jOgMO8PDBX9RiV+OpFQNkTx/tOJvxrdknGw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=GJADZP4WfomcW/N+9FXhZrfUG2FN+8MsN5rdl5q6nak=; b=ecyIWSXYRtMqu4DF9DTWQlG5XIn6yBcJB3dxZ0ePxLLordH2UheKVyHq9wL/zly0vxeSRhWcWSfaP08Rf/HnV93gmhYdsAVG4PEBbl1ksoVRWt5qztVSDoBbsDAc7XNkgBteikr/iFjA/WKm4KYr3685I3GPrGKoyU5XWOFowKee2k0KxHLF5U6c/DAhSYwoMeG4NtEcBPjRilcFLWfMj3txHp3iU/eA7pDO6WE6Ens0WPxY9RVNVh1v6uBbKI6J93DRIMh4ovteU8zkzW6Tea8df2Xa2e6BKYmL5BNqFQEMPkJrxK45ZKwVap8IE5MnwcXSljJUiObsltkfD5KQVA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=monjalon.net smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=GJADZP4WfomcW/N+9FXhZrfUG2FN+8MsN5rdl5q6nak=; b=BcP+FBSJYyYHfHBZLurLC96jwZwBAtEsJLp90xjcjMS82tlKVo0Y+9yVNBJAM+iueBw8qCZSHIS3+D1OyEJGYszf0EzEHfZWM2IcqWgAiRacJu8cMjgjpFjW5URX1O3hiPYY6nj9i3jP1XDzMCrs0S+lkVUL81Gt+yo/vYsX6yA= Received: from BN9PR03CA0114.namprd03.prod.outlook.com (2603:10b6:408:fd::29) by DS7PR12MB5767.namprd12.prod.outlook.com (2603:10b6:8:76::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6002.33; Thu, 26 Jan 2023 16:19:15 +0000 Received: from BN8NAM11FT064.eop-nam11.prod.protection.outlook.com (2603:10b6:408:fd:cafe::1f) by BN9PR03CA0114.outlook.office365.com (2603:10b6:408:fd::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6043.22 via Frontend Transport; Thu, 26 Jan 2023 16:19:15 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by BN8NAM11FT064.mail.protection.outlook.com (10.13.176.160) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.6043.17 via Frontend Transport; Thu, 26 Jan 2023 16:19:14 +0000 Received: from telcodpdk.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.34; Thu, 26 Jan 2023 10:19:13 -0600 From: Ferruh Yigit To: Thomas Monjalon CC: David Marchand , Subject: [PATCH v5 0/8] start cleanup of rte_flow_item_* Date: Thu, 26 Jan 2023 16:18:56 +0000 Message-ID: <20230126161904.864517-1-ferruh.yigit@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20221025214410.715864-1-thomas@monjalon.net> References: <20221025214410.715864-1-thomas@monjalon.net> MIME-Version: 1.0 X-Originating-IP: [10.180.168.240] X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BN8NAM11FT064:EE_|DS7PR12MB5767:EE_ X-MS-Office365-Filtering-Correlation-Id: 164aa6ec-09db-41a7-04d3-08daffb91129 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Egd3fIw9Iebv99qPypaOiT7bUtf0iM/eQksLJQvwTXbslOKAzIM4BeDsChav0OadH7qbTpkGcMKR0BSGL539x1DmS+f1XYyeXAEyypfkq4Gw90I67VMivKCqkd3WlIHN3Q1NspB7eOM3ArqJIzLlgP1Ss3DNfh3yUAusjT1pJramMA71XUZZLCNiIU2J9Z86R0WaogSMUUPfIthBA6JQgOt1c+60wNRtS2sAxJ09Z5BYQCs80oXsUa53uw9t+hqk23Lxcju6umMC7YxR1NgFGVzz2KiGKhkQIJBBjZqSE1mufcjANgKQ4xFE2fO3JwkV/JAEWP6bErqA65Z0zrl9626xIxD86f1Gg6bjXJoTB3D2/fSqDcMS52HREYCKD3rW/plievbm++VY9I5C6yXyYmbDZczZo1Gsa8+JcaEqa7RaDgw+0HkPV0XJf8GTv2g3iz0iPgP2S92iXONThwWk31YH3uY7qOEHQ3/kqd8NLVxmtns+NPxpmYyBaVvC/BCoo+SX4DzQf4Dw8rCPWv/5C2WPpL6/vaL3W0RHET9Hm+v1MQ4yizD2aMnyz792nl4hMZXvtoiKFMgsXpAyl0B7Be8zwgFT+JP+uOcxsV88Afj104de6MoRK5itdtN5mEIsGaBa88KC9iHpxxqSxeZW8rpsZJ+D63Yt3C1WKjkkH/BhOSL/i8Hgkesr4jgXRMJBe02tULfHeijEsCXS8xO0kgZ8+HhbmvbGfU0u280EU0c= X-Forefront-Antispam-Report: CIP:165.204.84.17; CTRY:US; LANG:en; SCL:1; SRV:; IPV:CAL; SFV:NSPM; H:SATLEXMB04.amd.com; PTR:InfoDomainNonexistent; CAT:NONE; SFS:(13230025)(4636009)(39860400002)(136003)(376002)(346002)(396003)(451199018)(40470700004)(46966006)(36840700001)(8676002)(40480700001)(82740400003)(81166007)(4326008)(356005)(36756003)(82310400005)(478600001)(86362001)(336012)(54906003)(7696005)(316002)(426003)(2616005)(47076005)(70206006)(1076003)(6916009)(40460700003)(2906002)(70586007)(36860700001)(186003)(6666004)(16526019)(44832011)(26005)(41300700001)(5660300002)(83380400001)(8936002)(36900700001); DIR:OUT; SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jan 2023 16:19:14.7811 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 164aa6ec-09db-41a7-04d3-08daffb91129 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BN8NAM11FT064.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB5767 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org There was a plan to have structures from lib/net/ at the beginning of corresponding flow item structures. Unfortunately this plan has not been followed up so far. This series is a step to make the most used items, compliant with the inheritance design explained above. The old API is kept in anonymous union for compatibility, but the code in drivers and apps is updated to use the new API. v5: * Fix more RHEL7 build error v4: * Fix build error for RHEL7 (gcc 4.8.5) caused by nested struct initialization. v3: * Updated Higig2 protocol flow item assignment taking into account endianness annotations. v2: (by Ferruh) * Rebased on latest next-net for v23.03 * 'struct rte_gre_hdr' endianness annotation added to protocol field * more driver code updated for rte_flow_item_eth & rte_flow_item_vlan * 'struct rte_gre_hdr' updated to have a combined "rte_be16_t c_rsvd0_ver" field and updated drivers accordingly * more driver code updated for rte_flow_item_gre * more driver code updated for rte_flow_item_gtp Cc: David Marchand Thomas Monjalon (8): ethdev: use Ethernet protocol struct for flow matching net: add smaller fields for VXLAN ethdev: use VXLAN protocol struct for flow matching ethdev: use GRE protocol struct for flow matching ethdev: use GTP protocol struct for flow matching ethdev: use ARP protocol struct for flow matching doc: fix description of L2TPV2 flow item net: mark all big endian types app/test-flow-perf/actions_gen.c | 2 +- app/test-flow-perf/items_gen.c | 24 +-- app/test-pmd/cmdline_flow.c | 180 +++++++++++----------- doc/guides/prog_guide/rte_flow.rst | 57 ++----- doc/guides/rel_notes/deprecation.rst | 6 +- drivers/net/bnxt/bnxt_flow.c | 54 +++---- drivers/net/bnxt/tf_ulp/ulp_rte_parser.c | 112 +++++++------- drivers/net/bonding/rte_eth_bond_pmd.c | 12 +- drivers/net/cxgbe/cxgbe_flow.c | 44 +++--- drivers/net/dpaa2/dpaa2_flow.c | 60 ++++---- drivers/net/dpaa2/dpaa2_mux.c | 2 +- drivers/net/e1000/igb_flow.c | 14 +- drivers/net/enic/enic_flow.c | 24 +-- drivers/net/enic/enic_fm_flow.c | 16 +- drivers/net/hinic/hinic_pmd_flow.c | 14 +- drivers/net/hns3/hns3_flow.c | 40 ++--- drivers/net/i40e/i40e_fdir.c | 14 +- drivers/net/i40e/i40e_flow.c | 124 +++++++-------- drivers/net/i40e/i40e_hash.c | 4 +- drivers/net/iavf/iavf_fdir.c | 18 +-- drivers/net/iavf/iavf_fsub.c | 10 +- drivers/net/iavf/iavf_ipsec_crypto.c | 4 +- drivers/net/ice/ice_acl_filter.c | 20 +-- drivers/net/ice/ice_fdir_filter.c | 24 +-- drivers/net/ice/ice_switch_filter.c | 64 ++++---- drivers/net/igc/igc_flow.c | 8 +- drivers/net/ipn3ke/ipn3ke_flow.c | 12 +- drivers/net/ixgbe/ixgbe_flow.c | 58 +++---- drivers/net/mlx4/mlx4_flow.c | 38 ++--- drivers/net/mlx5/hws/mlx5dr_definer.c | 48 +++--- drivers/net/mlx5/mlx5_flow.c | 62 ++++---- drivers/net/mlx5/mlx5_flow_dv.c | 184 ++++++++++++----------- drivers/net/mlx5/mlx5_flow_hw.c | 80 +++++----- drivers/net/mlx5/mlx5_flow_verbs.c | 48 +++--- drivers/net/mlx5/mlx5_trigger.c | 28 ++-- drivers/net/mvpp2/mrvl_flow.c | 28 ++-- drivers/net/nfp/nfp_flow.c | 21 +-- drivers/net/sfc/sfc_flow.c | 52 +++---- drivers/net/sfc/sfc_mae.c | 46 +++--- drivers/net/tap/tap_flow.c | 58 +++---- drivers/net/txgbe/txgbe_flow.c | 28 ++-- lib/ethdev/rte_flow.h | 121 ++++++++++----- lib/net/rte_arp.h | 28 ++-- lib/net/rte_gre.h | 7 +- lib/net/rte_higig.h | 6 +- lib/net/rte_mpls.h | 2 +- lib/net/rte_vxlan.h | 35 ++++- 47 files changed, 988 insertions(+), 953 deletions(-)