From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id B31ACA0C43 for ; Mon, 18 Oct 2021 15:19:23 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A668E40141; Mon, 18 Oct 2021 15:19:23 +0200 (CEST) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2055.outbound.protection.outlook.com [40.107.223.55]) by mails.dpdk.org (Postfix) with ESMTP id 980DA40141; Mon, 18 Oct 2021 15:19:22 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=k6DvplCWeG+2/nGnAwVjLyiJjCYrgk137glez5KJBu0XZI8X1nD2/uEcmBPyI/5OVCsfBppUscmijYisveQuwSk5N/+7ikpWnaXn4OSP55eW2gX8PvjFbImqICQGUUh7O1cJShwdyuhOZD+I2rmAp8xlR1rD0+FtZAGBltK7mMLyAmFQJhqh8AvZbL6ANCMOxWGbUlPnMdEOnORs1SI0Ix3KFeC+VxSvzhctlAlY90AU/cEGjvRQDoayuY8Kh6r46IwcwPwf2tnaA/hzdiL6tAUYj3Fw8o7fDSFvM19EpPbzFLc1CKFMw0s1Mc46iv+Sn/xlDnvbsOGDmBlAdOpQyA== 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=WyoRHeYzfwsCu0uzajO97qY9k00o39pTW95g3AmsSpE=; b=Jqek0BWSPlOD7+YuHEPoQKuDDK/K5Jzo98sZysEEb4WM891p8+UOng380wqMK2h0+bFWfkOoqPPkF7qfva2DiK7pgvf3y5Q36gi/PohcgK5AQLdphnaFOdkbJU/nSbdVgpiOZzqO4EL+VGGSxS9pN+r3Sf02uTjFAYAcVcl+be0+IGGQLY7Z/udUUascKRemERGQ/iLGW0Vk98RJ+sptkMlbFBUd6NBx+ET0UMxFV4zwRonVgShjrPTzFUd5eZgFXeZNRx/d+4UfSGUyTP5jWG/ESDfC0cqGk0i1YSePzXLUzBuUi69auO0pkNj+LMkpXUImbK24YYVK7RdcI81c0w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.112.34) smtp.rcpttodomain=dpdk.org smtp.mailfrom=nvidia.com; dmarc=pass (p=quarantine sp=quarantine 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=WyoRHeYzfwsCu0uzajO97qY9k00o39pTW95g3AmsSpE=; b=jUge/IhXaom0tB4Kx2/uP993wL3QW1LCE+PmtYbhPuRQJ9FVagu664cbWrep/I8/UGEfWLdaT2w3iYcVGoFFcrLOOr6uB3c9M69wp7xd6jFeHy43fC7Yrd9zxi5SUYGLCKBGbsCaqQr10r1VIuxph7LoGW837AcZ/tf2s5zVaWr7XtDEKa7SMqmZkHMTISz9uWjcheZ0ZKt2VC5FUc3MtKIffsEn8Y4R57AiMH2CsJB1OVr2Fy/Z8Nu3DcMzcKMwChp8bEcR/JojBmR+cVPjVZfNuvV2XKOkMookIv16Q6Q+vyh3jiRICxFGYOrqK6MRnq1E7d2dfOxDD8ydKiuObg== Received: from DM5PR2001CA0013.namprd20.prod.outlook.com (2603:10b6:4:16::23) by BYAPR12MB3559.namprd12.prod.outlook.com (2603:10b6:a03:d9::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4608.18; Mon, 18 Oct 2021 13:19:20 +0000 Received: from DM6NAM11FT049.eop-nam11.prod.protection.outlook.com (2603:10b6:4:16:cafe::fc) by DM5PR2001CA0013.outlook.office365.com (2603:10b6:4:16::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4608.15 via Frontend Transport; Mon, 18 Oct 2021 13:19:20 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 216.228.112.34) smtp.mailfrom=nvidia.com; dpdk.org; dkim=none (message not signed) header.d=none;dpdk.org; dmarc=pass action=none header.from=nvidia.com; Received-SPF: Pass (protection.outlook.com: domain of nvidia.com designates 216.228.112.34 as permitted sender) receiver=protection.outlook.com; client-ip=216.228.112.34; helo=mail.nvidia.com; Received: from mail.nvidia.com (216.228.112.34) by DM6NAM11FT049.mail.protection.outlook.com (10.13.172.188) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.4608.15 via Frontend Transport; Mon, 18 Oct 2021 13:19:20 +0000 Received: from nvidia.com (172.20.187.5) by HQMAIL107.nvidia.com (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Mon, 18 Oct 2021 13:19:17 +0000 From: Bing Zhao To: , CC: , , Date: Mon, 18 Oct 2021 16:18:55 +0300 Message-ID: <20211018131855.145323-1-bingz@nvidia.com> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [172.20.187.5] X-ClientProxiedBy: HQMAIL107.nvidia.com (172.20.187.13) To HQMAIL107.nvidia.com (172.20.187.13) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a652ad15-7c12-4629-d0dc-08d99239e54b X-MS-TrafficTypeDiagnostic: BYAPR12MB3559: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 4Ad2Hiila3YuJMenOAVNrAuRGPg0EkVsrvMgGv0DbPslZTtkP72FnKY/Q8f32I8v2zXIe1loDURvJdES0tNDEK9LnozdaSoVRogkwMg37VdpV2wUwPMot8ajc3Z9TrRmn64FCMsBk6qJxTmX13YnDz11jCcovtQlPXv1lvxI4C+3g8cA8z5WmafvL/gpbYPiy/fl2vpIW/1ZCIzvEaF/Wy5t9OqeWLM6yBRrOwf/rqn0pWGMsiM1xdpeUY0mGa/Nwhkan11AFh1Dx98znrEhmBknBXwJPmaPyWspDOUOryEBuX73Y3XwVrstWUyNmErevL6aPtnSXAKiuJfbLGU8/mMKZjQq0/C/bnsVARS/gbU+NBBEovYA0r6sdTEFGL2a0kdrDHZHA4Ihicwy3GEoM1DLnTkFeMe0/NeiXylqIz0qbEAvULXh1m1pX3KrEi3SjhArxaKTgY+ep7h1mf9q0OOpUMXxvu3tIa64mugy2NvKPN6LCArmr45QJxwAJiPXsM7p3e6oboY2k0E6M6ZaQLUtElrbKtQnSHxn+odnFdXfrypO4DLszx8P1fyv3WxEHgNt8bNA3F0ZvwL4fkZEQCud1bU9IsG7QtkiW+pEhsp8xP3CMToDdkY0NBfsBrMzOsAwAbkypb/Ud2ybDbpxLFwVnOUV1t6Ycwfr7635D5M3FkTIr2hPeWIbiR6EstQr8t75y+1PXW3uMRRs/ILy+w== X-Forefront-Antispam-Report: CIP:216.228.112.34; CTRY:US; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:mail.nvidia.com; PTR:schybrid03.nvidia.com; CAT:NONE; SFS:(4636009)(36840700001)(46966006)(316002)(4326008)(47076005)(6666004)(1076003)(508600001)(6286002)(5660300002)(70206006)(86362001)(83380400001)(356005)(7636003)(2906002)(26005)(36756003)(36906005)(450100002)(8676002)(70586007)(82310400003)(7696005)(8936002)(54906003)(6636002)(2616005)(110136005)(36860700001)(336012)(426003)(16526019)(186003)(55016002); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2021 13:19:20.2731 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: a652ad15-7c12-4629-d0dc-08d99239e54b X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=43083d15-7273-40c1-b7db-39efd9ccc17a; Ip=[216.228.112.34]; Helo=[mail.nvidia.com] X-MS-Exchange-CrossTenant-AuthSource: DM6NAM11FT049.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR12MB3559 Subject: [dpdk-stable] [PATCH] net/mlx5: fix meter yellow policy with RSS action X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" The RSS configuration in a policy action container was a pointer inside an union, and the pointer area could be used as other fate action. In the current implementation, the RSS of the green color was prior to that of the yellow color. There was a high posibility the pointer was considered as the RSS and result in a error flow expansion when only the yellow color had the RSS action. The check of the fate action type should also be done to get rid of the misjudgment. Fixes: b38a12272b3a ("net/mlx5: split meter color policy handling") Cc: stable@dpdk.org Signed-off-by: Bing Zhao Acked-by: Matan Azrad --- drivers/net/mlx5/mlx5_flow.c | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 509ca01859..4f5211a90d 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -3594,6 +3594,8 @@ flow_get_rss_action(struct rte_eth_dev *dev, { struct mlx5_priv *priv = dev->data->dev_private; const struct rte_flow_action_rss *rss = NULL; + struct mlx5_meter_policy_action_container *acg; + struct mlx5_meter_policy_action_container *acy; for (; actions->type != RTE_FLOW_ACTION_TYPE_END; actions++) { switch (actions->type) { @@ -3629,9 +3631,18 @@ flow_get_rss_action(struct rte_eth_dev *dev, if (!policy) return NULL; } - if (policy->is_rss) - rss = - policy->act_cnt[RTE_COLOR_GREEN].rss->conf; + if (policy->is_rss) { + acg = + &policy->act_cnt[RTE_COLOR_GREEN]; + acy = + &policy->act_cnt[RTE_COLOR_YELLOW]; + if (acg->fate_action == + MLX5_FLOW_FATE_SHARED_RSS) + rss = acg->rss->conf; + else if (acy->fate_action == + MLX5_FLOW_FATE_SHARED_RSS) + rss = acy->rss->conf; + } } break; } -- 2.27.0