From patchwork Tue Jul 11 15:20:42 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexander Kozyrev X-Patchwork-Id: 129450 X-Patchwork-Delegate: rasland@nvidia.com 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 6174842E47; Tue, 11 Jul 2023 17:21:17 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A15C1411F3; Tue, 11 Jul 2023 17:21:16 +0200 (CEST) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2067.outbound.protection.outlook.com [40.107.237.67]) by mails.dpdk.org (Postfix) with ESMTP id 01D2F4003C; Tue, 11 Jul 2023 17:21:14 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gtQJK3ahF3cD8Ij47Qaq2PXihogJ6nQGoMZti6XL0TT+Iji5HNy2Bh50x4q7xW/Y05W6kf+v7N7w8GqMgCqiV8pIpeCyTqfXqesbDPEradzQKhxzo8tCQz4Qe6m+FzH/PnC+OpLARjNBxeCHI5+sXiCxkMDf2FgmbnZILKwYJdtuByalCcS6CiPvWOIf3MvOUA1p29r0fNo7mH2Q53Rfssm3wxBk4fN9i7rjp2YYX2z62C3h3i/iyknrmSfYmaKEqUpJNWSqmv2XuHB3n+5AjbIrjgnzAk15L3pVvTmOOJHzdDMeYB+0SL+DKdkZtMn1dNij1IiT2xE/YBwjcNYLYg== 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=RnfRCpX3BZuB1gY5jqd81tsimpBgqqH+3glG5ay26q0=; b=IXdV/0B54BBIqUxbtkyNEVathEILmgAe7yW9frqX4n6FiVyqRXddcISTCaR0Pr0zpFR6L8acXTQkLUI+YqSSYsFqXj0a4SDjYNvXcxEeyifnNnd5RKPbshESiVcOhBwpGgYxSL7QgoeVsbWVNyPmfR1u2nnp/ocJpSSB958p86/Xhs8KXtvX6n6zCKe4wkp4m2ZWHFBAmoXjOYpA5o3jbvFXW90SbKvmf6dJeTyeDXQlj+fUbDySf0ie7b70qv563vh1iBLNo7Xtk9dfNJb+R6I9P8oJKuXKCo3ImYVJ9TTJfERZWo/QgbY0L8uF2VifGu6bqcPWXbA9qHJyOoCUXA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.161) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=RnfRCpX3BZuB1gY5jqd81tsimpBgqqH+3glG5ay26q0=; b=TeHj/8EGkcJTBEWRnm218FNB1WlFy6QLAAJVCGx6y/NRY/iz7qlycGK1Cd+MdTMq6CvA/CmNBcZU3aPb/clIvpYeXYwfqqmCEqkIDazapQ8MJuk9btDWgvzohOg+Hq+Y6/cLSlnQ+rVrkH83XUNi3f6n13lPWuH/AF+60mbYfU8BidLsYWWgLONplGd/cJeNS3Isn+lrUePg8vmKwsQ4UVdn16gxIpAxdoz/H/5AD08gNOOU/xZmCR0QpKxz0WThZ2QCo/H4P7NJh0s95w/8LjbmoANVSQuwy0YFquUgiHZZr2MORF3cc38Nq2SkD3hr3HIawzzdaZpgwhERjb/7Aw== Received: from BN9PR03CA0659.namprd03.prod.outlook.com (2603:10b6:408:13b::34) by MW4PR12MB7384.namprd12.prod.outlook.com (2603:10b6:303:22b::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6565.31; Tue, 11 Jul 2023 15:21:12 +0000 Received: from BN8NAM11FT100.eop-nam11.prod.protection.outlook.com (2603:10b6:408:13b:cafe::b6) by BN9PR03CA0659.outlook.office365.com (2603:10b6:408:13b::34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6565.32 via Frontend Transport; Tue, 11 Jul 2023 15:21:12 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.117.161) smtp.mailfrom=nvidia.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.117.161 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.117.161; helo=mail.nvidia.com; pr=C Received: from mail.nvidia.com (216.228.117.161) by BN8NAM11FT100.mail.protection.outlook.com (10.13.177.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6588.20 via Frontend Transport; Tue, 11 Jul 2023 15:21:12 +0000 Received: from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com (10.129.200.67) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.5; Tue, 11 Jul 2023 08:21:01 -0700 Received: from pegasus01.mtr.labs.mlnx (10.126.230.35) by rnnvmail201.nvidia.com (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.37; Tue, 11 Jul 2023 08:20:59 -0700 From: Alexander Kozyrev To: CC: , , , , Bing Zhao Subject: [PATCH 1/3] net/mlx5: fix validation for conntrack indirect action Date: Tue, 11 Jul 2023 18:20:42 +0300 Message-ID: <20230711152044.3141138-1-akozyrev@nvidia.com> X-Mailer: git-send-email 2.18.2 MIME-Version: 1.0 X-Originating-IP: [10.126.230.35] X-ClientProxiedBy: rnnvmail202.nvidia.com (10.129.68.7) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BN8NAM11FT100:EE_|MW4PR12MB7384:EE_ X-MS-Office365-Filtering-Correlation-Id: 1fe67770-e9ff-457a-5bbd-08db8222761a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: +wEse5/Ag3QFu5MV5Et+FlKfzFrdHW1i6pcLa7+rXz4Zn7LYqoozWI2D1Em9gz8JcgXVQgdCkDDt1/EMIoQjPbE9fUNLmiCPN8h8B0Tkm76aczy+SIHEGATKQvU+lWrq3HDL+giBMBF3y0lxNxukYMmmOL3Or3w+LXsEnc8e0T/dXeVoyijZb8YDFpcxtZjeyF6NkGGk2bpuKhyvG68I0PLFQjMznuohyBen9FYBWCVS566ifBnfcd2cujl8SraSUJJvppQkZzNlMI7pcTG02WoFD1vmbhFRM3PryK0MKt/UaEDEVYkmByWnyr8z6/CgrPDadpdl4ksbfEhXl8CcXjCCZpPAhJiP2nJfYunsXC46XMdTJcYeYDD2udBw7BoNDHG+k7rlo0YtWDeNvltnYID3Ia4cBTqqSmiy0E0Ylwk5g0PHAdDH8OqfrtrCZDTEOesuy8PdAJIJlhHIlvpXwbCKjwBziQpCX6E3KcQ1wSlUefxZtc8mfe4vbWZeCMq2gNyLWGccYIahI4LBm2PZ0v+8SKbbOqhe2XwIqAOesgCp7Ak08du/hvXICQH4CadjzD8gpMI/bSd0mZ2JNuHYvSnFvr8VVVyx/G/V0JOD3k//tG7gmBemBGQ6wOJfpGZR8obGPdt2uhvkEf+3vXYZQ72JDpfctE9wSSzNR4EEBj730Gvg4AghQL0FcEhTgE8MW6GuSQmF1dKD84R8xU193d+8YPGOQF9I8mRW2tr+4qsRoJIAlb3Uod/h9hH2tI3d X-Forefront-Antispam-Report: CIP:216.228.117.161; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:dc6edge2.nvidia.com; CAT:NONE; SFS:(13230028)(4636009)(39860400002)(396003)(346002)(376002)(136003)(451199021)(40470700004)(46966006)(36840700001)(478600001)(54906003)(450100002)(6666004)(47076005)(36860700001)(2616005)(426003)(83380400001)(36756003)(86362001)(40460700003)(40480700001)(2906002)(70206006)(82310400005)(1076003)(107886003)(186003)(16526019)(70586007)(26005)(336012)(356005)(7636003)(82740400003)(41300700001)(316002)(4326008)(8936002)(8676002)(5660300002)(6916009); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jul 2023 15:21:12.2963 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 1fe67770-e9ff-457a-5bbd-08db8222761a X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.117.161]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: BN8NAM11FT100.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB7384 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 From: Bing Zhao After rte_flow_shared_action_* API was replaced with rte_flow_action_handle_* API, one input parameter of the update interface was also changed. A generic pointer was used instead of the "const struct rte_flow_action *" pointer. In the entrance of mlx5 PMD callback for update, the validation is called for all indirect actions. But for conntrack type, the pointer is no longer with rte_flow_action pointer type and it will cause an incorrect casting and error. The content for updating should only be validated when needed. It should skip the validation in the entrance. Right now, the content was already added before updating the hardware by WQE. So the type of the indirect action should be checked before calling the action validate function. When creating a new conntrack object, the validation is still needed since all the content will be used to update the hardware context. Fixes: 40327aab1491 ("ethdev: introduce indirect flow action") Signed-off-by: Bing Zhao Acked-by: Suanming Mou --- drivers/net/mlx5/mlx5_flow.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 4b567f61dd..9bd92f28fd 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -10753,9 +10753,18 @@ mlx5_action_handle_update(struct rte_eth_dev *dev, const struct mlx5_flow_driver_ops *fops = flow_get_drv_ops(flow_get_drv_type(dev, &attr)); int ret; + uint32_t act_idx = (uint32_t)(uintptr_t)handle; + uint32_t type = act_idx >> MLX5_INDIRECT_ACTION_TYPE_OFFSET; - ret = flow_drv_action_validate(dev, NULL, - (const struct rte_flow_action *)update, fops, error); + switch (type) { + case MLX5_INDIRECT_ACTION_TYPE_CT: + ret = 0; + break; + default: + ret = flow_drv_action_validate(dev, NULL, + (const struct rte_flow_action *)update, + fops, error); + } if (ret) return ret; return flow_drv_action_update(dev, handle, update, fops, From patchwork Tue Jul 11 15:20:43 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexander Kozyrev X-Patchwork-Id: 129451 X-Patchwork-Delegate: rasland@nvidia.com 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 35FAB42E47; Tue, 11 Jul 2023 17:21:22 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id ADC9142B8C; Tue, 11 Jul 2023 17:21:20 +0200 (CEST) Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2040.outbound.protection.outlook.com [40.107.236.40]) by mails.dpdk.org (Postfix) with ESMTP id D024340A7D; Tue, 11 Jul 2023 17:21:19 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=HSChGl/p8jQYKXIcpDmHEOL/v4JDcTdfd+DzRqNIPwulPJsbGVvJfAB43kQRDpwkLbBnu21tkjuDDeQ2Nv9NjeDl1cwY6ZJWwYPynNIrKG+/OlKycg34M3dHC5NwJ/aJplET0nkNDaB0IewOf5GjtgzY7nK5DzWHleA3o18nshJtEShpLMwxBeiRIAMrDi9Cn7YiNb/qPZp8md5nZWmShrSiH1XJEfMtjypLM9fBnClP9MVeY7ZF9ehSl6aEIWuYStMBMFCPU54f0J4zmaTGGaNUQrZKQ0pXAX6b+vgYIua10Kk3b2RTHij4BKz8KMM+luslgQSNNonSEzOxyjy78w== 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=X2PudtKe9dSYmvJGsAYewNTU8ReuGKpDJAJ6SQFkeHU=; b=ID4duWD6fBD6fP3qvxBgVAnIqcvM+nTWnJ1ChiEZ+XctbPMeTj0gGcvKeA8vszBtsWcIPIEhlyxYO4B4gcPOIq60bTyHKt7QMOMVn3gjTInMWoC+SXnnDOcUd9s2NII+83Zj8bPLj9G54h52WQW+YzsWgYaEl99SdDONfqGMV9vgvd5UQyJpGm0LQWJ2FuFksQcFXXACm3tdJ7Ozp5YE39vk5KbF3xPIagFei7pYvWp0qA9OJN6yyHu1HEeDLbOPGWoqPrLqVtGRb0FU5XVboNvMTddWorg3aBr1wZl+CAzgf38EWgJSYlnZMTKa/NkaTwq05aEFa3kZaA/s1wY8RA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.160) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=X2PudtKe9dSYmvJGsAYewNTU8ReuGKpDJAJ6SQFkeHU=; b=kRfePC/dcRt3iFSPfqa0Mvzq5sz5/tjZDZ+BbNwkMcd+zKsb3pIuv90mUMLU9CHOEmODasM8IZ7pRIakVArgrBrL6P00/U3jQwvZHNQAQwygjaiS/l6FG8/I1SgpH7r11w55W5JR5XtMPainXJAGoC7U5Cx1cwsdvUPbfomsH5ICl0pIr3HircXs+bi1tl8RjgPufz6j0tI+uQue9cI0cz7n61gcICbPwFFsLrR1MMfcWbtNuJgeTNGFBWbe/IIwJTSeI6byeHcsMqAFPF/LfcxfvRvePKrJifwa4iE2W9zDiuHgFD27tAY/s+hFY2cAJM6UIlk5rQ3JWiONvC6CGQ== Received: from MW4PR03CA0281.namprd03.prod.outlook.com (2603:10b6:303:b5::16) by CH3PR12MB7739.namprd12.prod.outlook.com (2603:10b6:610:151::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6565.32; Tue, 11 Jul 2023 15:21:18 +0000 Received: from CO1NAM11FT029.eop-nam11.prod.protection.outlook.com (2603:10b6:303:b5:cafe::c0) by MW4PR03CA0281.outlook.office365.com (2603:10b6:303:b5::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6565.31 via Frontend Transport; Tue, 11 Jul 2023 15:21:18 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.117.160) smtp.mailfrom=nvidia.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.117.160 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.117.160; helo=mail.nvidia.com; pr=C Received: from mail.nvidia.com (216.228.117.160) by CO1NAM11FT029.mail.protection.outlook.com (10.13.174.214) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6588.20 via Frontend Transport; Tue, 11 Jul 2023 15:21:17 +0000 Received: from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com (10.129.200.66) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.5; Tue, 11 Jul 2023 08:21:03 -0700 Received: from pegasus01.mtr.labs.mlnx (10.126.230.35) by rnnvmail201.nvidia.com (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.37; Tue, 11 Jul 2023 08:21:01 -0700 From: Alexander Kozyrev To: CC: , , , Subject: [PATCH 2/3] net/mlx5: fix handle validation for meter mark Date: Tue, 11 Jul 2023 18:20:43 +0300 Message-ID: <20230711152044.3141138-2-akozyrev@nvidia.com> X-Mailer: git-send-email 2.18.2 In-Reply-To: <20230711152044.3141138-1-akozyrev@nvidia.com> References: <20230711152044.3141138-1-akozyrev@nvidia.com> MIME-Version: 1.0 X-Originating-IP: [10.126.230.35] X-ClientProxiedBy: rnnvmail202.nvidia.com (10.129.68.7) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CO1NAM11FT029:EE_|CH3PR12MB7739:EE_ X-MS-Office365-Filtering-Correlation-Id: 8b601fcb-2635-4036-efe2-08db8222795a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: KHZdSO2w6xycTuiy4IFtvbXhuf5zSg5xEygB9Y0+1JUmRyZsLj/s/56CTpFf/EKkgxmtbube/49/jFtXhNeuNBzzTQTxjHGJYuuLq+AXqCFgnTShLRqNQwoUDkz1P7XM3Tf3UiY+oKJqPanQrhkbSwHzRfxjwvINk3zDj+ojrw1Rn48Ddax+8hlDMps+wxpUUitgQqcVTdvhrcQomJWCnyYuA785amTx4aFzh7ZBj4hnfvs5xsxz03CDoPP0wCiM2UxTbTISPGLS8HXre5cekpHkJIsAi+a3D56vMSnM6514tbvGFY0blO3SzKU/b13Mf5exU/kQwSV3CNtwDM78AdiQShXtX6HzK2Vc6jM9C3kwgGzyCreYJG0YXGn7oBrVE9cs2A4K+ndON7lNgBu3hBMvHOcZhhFHbvFSl5EwdwmADoiNzcKByiL827GulnF7pU+LtZUEn1cobXYnPGcvHtRh2l0TyCKXLetod8yNWOJWCQuSKJWr8Y5OjGVwPucLy49oHldgerLDSQGU/IHijXnNYph+iLUrBdHPqBwsDANjlG0yeVxAioy9cfaYW/ekDtSm62NuNGCiAZNLbv4VnwtHYPcn9vrWOZNo06Rs+L9iEVV8Z6HB/yM428RpJqfOGPmDDUlvowNfTlHbGUZm3sWVWXWGu6S1f4rfoa511hmUtgdd8ueqULygryKvxFPmuj+fAMPe8r0m1rPeJAS/eG5LaO/Q/MwZHpnACgz3COs= X-Forefront-Antispam-Report: CIP:216.228.117.160; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:dc6edge1.nvidia.com; CAT:NONE; SFS:(13230028)(4636009)(346002)(39860400002)(376002)(136003)(396003)(451199021)(46966006)(36840700001)(40470700004)(1076003)(16526019)(336012)(186003)(26005)(41300700001)(478600001)(54906003)(6916009)(316002)(4326008)(40480700001)(70586007)(450100002)(70206006)(107886003)(8936002)(8676002)(5660300002)(6666004)(426003)(47076005)(2616005)(4744005)(2906002)(36860700001)(40460700003)(83380400001)(7636003)(356005)(82740400003)(86362001)(36756003)(82310400005); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jul 2023 15:21:17.8621 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 8b601fcb-2635-4036-efe2-08db8222795a X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.117.160]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: CO1NAM11FT029.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR12MB7739 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 Skip the METER_MARK validation for the indirect action update. The old synchronous indirect action update was left out during the METER_MARK implementation in favor of the async way. Allow the sync method of doing this with relaxed validation. Fixes: 48fbb0e93d06 ("net/mlx5: support flow meter mark indirect action with HWS") Signed-off-by: Alexander Kozyrev Acked-by: Suanming Mou --- drivers/net/mlx5/mlx5_flow.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 9bd92f28fd..e91eb636d0 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -10758,6 +10758,7 @@ mlx5_action_handle_update(struct rte_eth_dev *dev, switch (type) { case MLX5_INDIRECT_ACTION_TYPE_CT: + case MLX5_INDIRECT_ACTION_TYPE_METER_MARK: ret = 0; break; default: From patchwork Tue Jul 11 15:20:44 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexander Kozyrev X-Patchwork-Id: 129452 X-Patchwork-Delegate: rasland@nvidia.com 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 6257A42E47; Tue, 11 Jul 2023 17:21:29 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0D5E142D1A; Tue, 11 Jul 2023 17:21:23 +0200 (CEST) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2046.outbound.protection.outlook.com [40.107.223.46]) by mails.dpdk.org (Postfix) with ESMTP id E324E42D13; Tue, 11 Jul 2023 17:21:21 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=UpjINN+2EUayPjJPC/nNLhAwQDpwbdFzkAVGUiaGuBvafWr6FQBlvtzByCRHFoNo3YZMhHbot+Eu8RaRQNAvc9nBU9Ej/sGGYtKbBlW6uD8rH1jJUOEBrbOIrNAO6VgejS5Kr5DgyfCwqus35NesL9kpHsX3dnlH1+m5XPXan7Xh9BcWcDP3Qo6Vo40AZP26dt+ArXli3E9S3XSztN1cLn3sStBhjM9tj2Ehl2epYFEXuX+3G9zSlPDQH6SXTmYE0DiY7aJkpjmTCyhA6NXVBmCVZ74hXh2aQBlNbLAspWys8zIE9DdHQtFte309FGS/NxupeMs/BFfgDmKoOml8pA== 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=jbNlLe9QIdNhpVkhkpkxxVjSo7r8lVBFyWY2NnQJw6c=; b=R+IMKK9n8N2L95g2bEF7mzaRsHLpxekx/Ck0BOLL0lyr65Z9YwJX72ZfI88shEMZG/8M1wrk2U6grG2RY3W0HdW1qIRg3vxHEDBF1EXCFP/yahhGc66riwUi7HPFf1GheKqEQ/Y5z/TXhwcWRCim2yzYbI1u1Fzui6P5UMo1CXjLLpjzlSchDN3Cav4XRMSLXT17/bVwbxmcnh+vwBg3Lq9UpI/LiB+8gJB1KmXWgXk0W95Agdhol9LXjLvo3QW07CwJQqvr2BGEp7ZuoN91Nwgc//rgdERwqQWgtgPBCaL5d7wV2lnAXEadaQ+q8OsfihyUBGzPy4f9woGUmHAEsQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.161) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=nvidia.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=jbNlLe9QIdNhpVkhkpkxxVjSo7r8lVBFyWY2NnQJw6c=; b=QQoEqQqc+yg7zl3XN28znEoo/lv8ZhymvJGxHigDxqTqPbdxDJwU1ZYXMenxGGxigwYbLRe6uNB9Z7asWhl2oZlWPiijPwGyfV3+mj1i5ofQgn/BPJYBUTMoahDYLiur5tKSfHsCont+EIpnabsxT4bPolCQr4mLladnA7GI34+12k3C49o2J1CdPZY8gPijserQXpkRLePSMTVBPjIlFbN/3PxMKktjQCIwyGRdVVA5Bdb+8EKeOWr44tRj4adeeBbAbhKcYOxM93LyIFat0gvn5CtEQfWXwW/mFrKGJ7jeFZ4iqMEEPfW34x+C76xLLhMYIuCY0TSfVOwEyy69Qw== Received: from BN9PR03CA0111.namprd03.prod.outlook.com (2603:10b6:408:fd::26) by IA0PR12MB8376.namprd12.prod.outlook.com (2603:10b6:208:40b::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6565.31; Tue, 11 Jul 2023 15:21:17 +0000 Received: from BN8NAM11FT090.eop-nam11.prod.protection.outlook.com (2603:10b6:408:fd:cafe::f5) by BN9PR03CA0111.outlook.office365.com (2603:10b6:408:fd::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6588.20 via Frontend Transport; Tue, 11 Jul 2023 15:21:17 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.117.161) smtp.mailfrom=nvidia.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.117.161 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.117.161; helo=mail.nvidia.com; pr=C Received: from mail.nvidia.com (216.228.117.161) by BN8NAM11FT090.mail.protection.outlook.com (10.13.177.105) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6588.20 via Frontend Transport; Tue, 11 Jul 2023 15:21:17 +0000 Received: from rnnvmail201.nvidia.com (10.129.68.8) by mail.nvidia.com (10.129.200.67) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.5; Tue, 11 Jul 2023 08:21:05 -0700 Received: from pegasus01.mtr.labs.mlnx (10.126.230.35) by rnnvmail201.nvidia.com (10.129.68.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.37; Tue, 11 Jul 2023 08:21:03 -0700 From: Alexander Kozyrev To: CC: , , , Subject: [PATCH 3/3] app/testpmd: fix meter mark handle update Date: Tue, 11 Jul 2023 18:20:44 +0300 Message-ID: <20230711152044.3141138-3-akozyrev@nvidia.com> X-Mailer: git-send-email 2.18.2 In-Reply-To: <20230711152044.3141138-1-akozyrev@nvidia.com> References: <20230711152044.3141138-1-akozyrev@nvidia.com> MIME-Version: 1.0 X-Originating-IP: [10.126.230.35] X-ClientProxiedBy: rnnvmail202.nvidia.com (10.129.68.7) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BN8NAM11FT090:EE_|IA0PR12MB8376:EE_ X-MS-Office365-Filtering-Correlation-Id: 270b839e-7911-4ab1-6a1e-08db82227947 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: hxIrtPuaqCPko/6znDtTUh59StY5Y/UHxoZ7OhtZdQKhLCmdgbTMEWFSEFWjzM6fRtyNjUQN9vMmnAGeG5GL4IbAA+FfimCqRkuFoAfoZ18NVJpwcfu0vVMJ/qmdvD/7DHMtLGkd+5kNIySsOmkbmspfGsPtd1SnM8HWJkKn3jSv/AE8HbjaFgMNZ3QAeeYIGrD8OZA5eaM6gd/Z5Z/FsV4LwlacEwHytgYuUUv19PnedOFpsYbYIz7zDM7+WJHr1X+6WO4XpMPW4dlB5vpwD0D/w/L1ld7iOKyWlPEZkOI3UGY6a2LfheSCCZG+x1HGMtLHI3zJVFdyAQDCnSMzXoS+MNFGT7iXsArTd3BkPaDFMlyCE+D7yre/sqFj4Ce66zAAsXoOhjBCpZmfdX0HNypUQw83B2GbPscbzIiBrJKfYKdGkXASDMmF8AdzsxK6wmtRPkWNHRJnKT16osrLJaB8myHX3hiAu8Ajr9XJMYD7IzoQQ7Uo7LJjynUgaitFvQN5J1vpuNwW7SuMCyt2ruwIbGh+M6s3DeLEFoiRnbrSvnOYxyCzYdFMhcBTIFb9dnT0Do3brKG6NMV83r915F0Nf3MBgslsViul39heXpaoa++aTAK/XxMmuX+D7UnUHcpGsf4EjLlbh5UhGG/0vlzhhkmoAVnDQnKKAF7OQDheOLObfJLVykOXBKhcBclVPKGAAwQVCDGljcrBQUm2t9Q5VIY7oV8okya7wWgnKx4= X-Forefront-Antispam-Report: CIP:216.228.117.161; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:dc6edge2.nvidia.com; CAT:NONE; SFS:(13230028)(4636009)(346002)(376002)(39860400002)(396003)(136003)(451199021)(40470700004)(46966006)(36840700001)(426003)(336012)(82310400005)(316002)(6666004)(15650500001)(54906003)(4326008)(40480700001)(450100002)(478600001)(2906002)(36756003)(40460700003)(356005)(7636003)(70586007)(70206006)(6916009)(82740400003)(2616005)(83380400001)(16526019)(186003)(36860700001)(5660300002)(8676002)(8936002)(47076005)(107886003)(1076003)(41300700001)(86362001)(26005); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jul 2023 15:21:17.6404 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 270b839e-7911-4ab1-6a1e-08db82227947 X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.117.161]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: BN8NAM11FT090.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA0PR12MB8376 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 The indirect action handle update for the METER_MERK action was implemented only for the async RTE Flow API. Allow updating the METER_MARK parameters via the old sync method. Fixes: 9c4a0c1859a3 ("ethdev: add meter color mark flow action") Signed-off-by: Alexander Kozyrev --- app/test-pmd/config.c | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index c4784b7f2c..58ac1576b7 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -1943,6 +1943,7 @@ port_action_handle_update(portid_t port_id, uint32_t id, struct rte_flow_error error; struct rte_flow_action_handle *action_handle; struct port_indirect_action *pia; + struct rte_flow_update_meter_mark mtr_update; const void *update; action_handle = port_action_handle_get_by_id(port_id, id); @@ -1956,6 +1957,17 @@ port_action_handle_update(portid_t port_id, uint32_t id, case RTE_FLOW_ACTION_TYPE_CONNTRACK: update = action->conf; break; + case RTE_FLOW_ACTION_TYPE_METER_MARK: + rte_memcpy(&mtr_update.meter_mark, action->conf, + sizeof(struct rte_flow_action_meter_mark)); + if (mtr_update.meter_mark.profile) + mtr_update.profile_valid = 1; + if (mtr_update.meter_mark.policy) + mtr_update.policy_valid = 1; + mtr_update.color_mode_valid = 1; + mtr_update.state_valid = 1; + update = &mtr_update; + break; default: update = action; break; @@ -3162,8 +3174,10 @@ port_queue_action_handle_update(portid_t port_id, case RTE_FLOW_ACTION_TYPE_METER_MARK: rte_memcpy(&mtr_update.meter_mark, action->conf, sizeof(struct rte_flow_action_meter_mark)); - mtr_update.profile_valid = 1; - mtr_update.policy_valid = 1; + if (mtr_update.meter_mark.profile) + mtr_update.profile_valid = 1; + if (mtr_update.meter_mark.policy) + mtr_update.policy_valid = 1; mtr_update.color_mode_valid = 1; mtr_update.init_color_valid = 1; mtr_update.state_valid = 1;