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 52AFB45A16; Tue, 24 Sep 2024 12:53:42 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3E94F402E2; Tue, 24 Sep 2024 12:53:42 +0200 (CEST) Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on2045.outbound.protection.outlook.com [40.107.93.45]) by mails.dpdk.org (Postfix) with ESMTP id D85EC4028E; Tue, 24 Sep 2024 12:53:36 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=MjTGpCAXoOYhUKp2XKl7o0S3rTWNY7zxInGaAulaulzc/atFRYR38EtRa6XljHdsnnsBDHOHXyX6IgKHx15dycEirFFkGS/LR3lO21P5YU2bsUg3WS1nmAgh94YO3S3NQONyiLIHdR8IDj/GB9oNBvq6IVAuTsjLrr6iISyJhGfutR0v0HsEGRyq9fO+RbfCDR5/dVJRrkSF4cL57K30bdvHodgtVLQho9F8X05l9lLv6D8A+h6QAoPg+TNJlHBvDDNMuvgtr1IfYe4P3WNaLwRpA6RzxfuSav0HhN19/EnTLMl+f9GfF3S7+grJVP0lV7eClTnj4/mAq6cLo/WnZQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=aKyuQaC51tP+FD3nlKTijM2LT3ZeIC8y7uHB724M77c=; b=ZaCzz3GslsyIUpK7q9ajjk7Vy7LfBKyRqhADUlZ0vGxpKMpADxIBLF0h6sAHEJ8pSVWoOE38R1BcTrbjZXuYPTwvQ1NifENwxdNAJydFfSCCoNyrWGcvS8BTOSyi95KPWd+6fapGKLS+ibqPwnffv4/fflPa15jsxeIZusWvchj82lYU3oEJ/Vv/ZFodgAKbsUcv6oPWbZQDvLDFN4xTjAUqUnKqvb1bbwJbuvu6CRHzth8Ti2FgP+6OJOjJNuAqIo1Sx409SMBcPsdz7a+BZ1jIYp7IywjGlE1jyHAtdLu+azE7cq6gHAag/UQo8tw27HvscSkOpXLc+1+NhOg65w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.160) smtp.rcpttodomain=monjalon.net 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 (0) 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=aKyuQaC51tP+FD3nlKTijM2LT3ZeIC8y7uHB724M77c=; b=njDE4Q4sJBhrwm+6zJgdkr3/DzMW84T6KunPzXGMEvdgoAdDCl/pgh9PAPHhWhvpa+OwyzVdiq0dZ6XKI021UC35RrhIUHoGsdoVl8tJQOM2RXJ1zeqBA1IXlYSWwSerweKAo3xrod3NE0Rr/Tu+e7Qt2MqHh8H4yq2rKzVqQM5i88dYXh0505NprNaFaq7tPPhO1IMZ57T+ajJ125OOunO7QOgUPAcw6ZmE9fMa5mpbZokTSoOpomHCDwEZT4hSQubcf0TvEonsVb2RKCdN8CdajHKIr8O6P0KmMP7dIttwHTvTXwtijQPWBuNiSGOYxk/rqr3RJv5UxKwxSJh0vA== Received: from BL1PR13CA0187.namprd13.prod.outlook.com (2603:10b6:208:2be::12) by DM6PR12MB4108.namprd12.prod.outlook.com (2603:10b6:5:220::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7982.28; Tue, 24 Sep 2024 10:53:32 +0000 Received: from BL6PEPF0001AB73.namprd02.prod.outlook.com (2603:10b6:208:2be:cafe::92) by BL1PR13CA0187.outlook.office365.com (2603:10b6:208:2be::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8005.17 via Frontend Transport; Tue, 24 Sep 2024 10:53:32 +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 BL6PEPF0001AB73.mail.protection.outlook.com (10.167.242.166) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7918.13 via Frontend Transport; Tue, 24 Sep 2024 10:53:31 +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.1544.4; Tue, 24 Sep 2024 03:53:17 -0700 Received: from nvidia.com (10.126.231.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.1544.4; Tue, 24 Sep 2024 03:53:14 -0700 From: "Minggang Li(Gavin)" To: , , , , Dariusz Sosnowski , Bing Zhao , Suanming Mou , Alexander Kozyrev CC: , , Subject: [PATCH V3 1/3] net/mlx5: set rte errno if malloc failed Date: Tue, 24 Sep 2024 13:52:56 +0300 Message-ID: <20240924105258.58037-2-gavinl@nvidia.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240924105258.58037-1-gavinl@nvidia.com> References: <20240924055938.48374-4-gavinl@nvidia.com> <20240924105258.58037-1-gavinl@nvidia.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [10.126.231.35] X-ClientProxiedBy: rnnvmail203.nvidia.com (10.129.68.9) To rnnvmail201.nvidia.com (10.129.68.8) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL6PEPF0001AB73:EE_|DM6PR12MB4108:EE_ X-MS-Office365-Filtering-Correlation-Id: 8acb129d-9048-4d14-0ebc-08dcdc87216a X-LD-Processed: 43083d15-7273-40c1-b7db-39efd9ccc17a,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230040|36860700013|376014|1800799024|82310400026; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?8bCN1NLVfC2L801WVtWw5mzzExX4sgBykYaM72fmTN2MysoN8hoyDOOb8UfW?= =?us-ascii?Q?RR2RzYMtLI9OdfOYcJezGO/99TACn9GpPazlmyr6+3W9YGjHEvEA5UpWXFFh?= =?us-ascii?Q?sx4EjqnOLydOYgS2sLn+Cz0FuW4ONuJUl4jH/GAUWVPPcOQ6i38m7m6aUhtA?= =?us-ascii?Q?mN2sqO0RCCMbDE6vtULAYYBCKz4j9jQu9S1bsjRv79JrLzsXbaQcW4BD9rkg?= =?us-ascii?Q?dPSVrmAGxxtUv3+WPZNSgVn20RvcfvS9mJjhpd0YsYIDccXTs0LrniBfGhx1?= =?us-ascii?Q?OrW2vakb7xwEww9VRLCkPRgdZmYVkcj1PBpQwUrlQ6wlB8/miiIME6yJtgPM?= =?us-ascii?Q?bNmo4B1xk93hnr81DhqOKhlI/2kdt1xIzckmkAQwk4P7/CCEWYm8bbEHthjT?= =?us-ascii?Q?YhIl3xa2A09KfORiFdxlkTWmqXzoWqm/RnvtxPA9b81Lc2eJ/G2gm++37I9q?= =?us-ascii?Q?9yek2HMEFjfF7P6pJfErKYcvFFIOrr8lSsJlVQgNjHALfigth4PMmfEwqhEt?= =?us-ascii?Q?6tv+wxg3R6KRi67fCNAzoAlxYy1Kg1TlZUk2Mid22Ev3mcX2qk0CEQZJOd2H?= =?us-ascii?Q?UoohwUzOKhr3riWslCUOxF/ehueRg6ZixuSwNjcmQHZfQSBPG15lLis9XADT?= =?us-ascii?Q?ZmKs4qXmOXNrt71ZTnYR+tFUjVMALp9Dyn2CzfC1lu4nK+X3ofJpeUOZM5q7?= =?us-ascii?Q?gRXKZZd6Lmp7RyUKMlQGMCgMij0CIu0FPf6tfK03JeXOcHKwrpK3LJDW0Q0u?= =?us-ascii?Q?2AArKT45Za2S45HwbW5n50is+techrucnStvvjjclP4dwzF9iZs883nYN6WZ?= =?us-ascii?Q?53zjGuFqGVq/PYjbDvHcGsrSLzAY7rWvHvfsXNs1osvqUAbexx/IVs1FbpfE?= =?us-ascii?Q?FMq2sIEUXFg+GoJ2Wyr/0kKaC3Lzv2ePm1vYONbFJhmhjeCfX5e0oSbLc6Zb?= =?us-ascii?Q?XXhLmtsln+ZvIXijTsJzvpmxG3fMAUByAAM7CelKWwwCRQCPa2V2SLdH0Gzh?= =?us-ascii?Q?57fCuf7iutP3sub6D70Fa/Ln6+iByvUJ/26U78H3JIM1/WoXRoCGcm2oB21d?= =?us-ascii?Q?/v4cKqWL8XpLGKn+MyUuXvfdsjJFn3H/bRfw+wVf0HbMN6A3fcUp+01/HAM+?= =?us-ascii?Q?qx9Vvqg4Oei28LWrt/MtLEhSTMLihDTPYTd4SStzzF6rWQ9hERNzRlp8sMtq?= =?us-ascii?Q?uRuNX+RdixqnndFpFBYujeO+OSVsaEr9+MY9TpiNsF19su40YIUkI+Ou6yJ3?= =?us-ascii?Q?RCujEDnhcsp6uFDfvbIroP1mItBn2/j/+be2HBvnpfoQVpTTXTqPOasKAqZ5?= =?us-ascii?Q?U+tLIo9DW+t1IH98iYG3/GXgpD8JtzHgxV089Bn2zw9oyJqhYGmzoolUuV7u?= =?us-ascii?Q?FSPQVG6sbcB8AWxR79qdvUzk/As2c820VFY++V1r5bCnM9X8/jQjQviT92C+?= =?us-ascii?Q?Dx2gxOb3SM7M1FURxuI840Nx9mxK3YFC?= 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:(13230040)(36860700013)(376014)(1800799024)(82310400026); DIR:OUT; SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Sep 2024 10:53:31.7187 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 8acb129d-9048-4d14-0ebc-08dcdc87216a 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: BL6PEPF0001AB73.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB4108 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 rte_errno should be set if anything wrong happened in under layer so that user can figure out what's going on. There were some cases that did not set it when ipool allocation failed. To fix the issue, set rte_errno to ENOMEM if mlx5_ipool_malloc failed to allocate ID. Fixes: c40c061a022e ("net/mlx5: add basic flow queue operation") Fixes: 48fbb0e93d06 ("net/mlx5: support flow meter mark indirect action with HWS") cc: stable@dpdk.org Signed-off-by: Minggang Li(Gavin) Acked-by: Bing Zhao --- drivers/net/mlx5/mlx5_flow_hw.c | 31 ++++++++++++++++++++++++------- 1 file changed, 24 insertions(+), 7 deletions(-) diff --git a/drivers/net/mlx5/mlx5_flow_hw.c b/drivers/net/mlx5/mlx5_flow_hw.c index a275154d4b..f34670b3ec 100644 --- a/drivers/net/mlx5/mlx5_flow_hw.c +++ b/drivers/net/mlx5/mlx5_flow_hw.c @@ -1905,7 +1905,7 @@ flow_hw_meter_mark_alloc(struct rte_eth_dev *dev, uint32_t queue, const struct rte_flow_action_meter_mark *meter_mark = action->conf; struct mlx5_aso_mtr *aso_mtr; struct mlx5_flow_meter_info *fm; - uint32_t mtr_id; + uint32_t mtr_id = 0; uintptr_t handle = (uintptr_t)MLX5_INDIRECT_ACTION_TYPE_METER_MARK << MLX5_INDIRECT_ACTION_TYPE_OFFSET; @@ -1917,8 +1917,15 @@ flow_hw_meter_mark_alloc(struct rte_eth_dev *dev, uint32_t queue, if (meter_mark->profile == NULL) return NULL; aso_mtr = mlx5_ipool_malloc(pool->idx_pool, &mtr_id); - if (!aso_mtr) + if (!aso_mtr) { + rte_flow_error_set(error, ENOMEM, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, + NULL, + "failed to allocate aso meter entry"); + if (mtr_id) + mlx5_ipool_free(pool->idx_pool, mtr_id); return NULL; + } /* Fill the flow meter parameters. */ aso_mtr->type = ASO_METER_INDIRECT; fm = &aso_mtr->fm; @@ -3926,8 +3933,10 @@ flow_hw_async_flow_create(struct rte_eth_dev *dev, return NULL; } flow = mlx5_ipool_malloc(table->flow, &flow_idx); - if (!flow) + if (!flow) { + rte_errno = ENOMEM; goto error; + } rule_acts = flow_hw_get_dr_action_buffer(priv, table, action_template_index, queue); /* * Set the table here in order to know the destination table @@ -3938,8 +3947,10 @@ flow_hw_async_flow_create(struct rte_eth_dev *dev, flow->idx = flow_idx; if (table->resource) { mlx5_ipool_malloc(table->resource, &res_idx); - if (!res_idx) + if (!res_idx) { + rte_errno = ENOMEM; goto error; + } flow->res_idx = res_idx; } else { flow->res_idx = flow_idx; @@ -4070,8 +4081,10 @@ flow_hw_async_flow_create_by_index(struct rte_eth_dev *dev, return NULL; } flow = mlx5_ipool_malloc(table->flow, &flow_idx); - if (!flow) + if (!flow) { + rte_errno = ENOMEM; goto error; + } rule_acts = flow_hw_get_dr_action_buffer(priv, table, action_template_index, queue); /* * Set the table here in order to know the destination table @@ -4082,8 +4095,10 @@ flow_hw_async_flow_create_by_index(struct rte_eth_dev *dev, flow->idx = flow_idx; if (table->resource) { mlx5_ipool_malloc(table->resource, &res_idx); - if (!res_idx) + if (!res_idx) { + rte_errno = ENOMEM; goto error; + } flow->res_idx = res_idx; } else { flow->res_idx = flow_idx; @@ -4218,8 +4233,10 @@ flow_hw_async_flow_update(struct rte_eth_dev *dev, nf->idx = of->idx; if (table->resource) { mlx5_ipool_malloc(table->resource, &res_idx); - if (!res_idx) + if (!res_idx) { + rte_errno = ENOMEM; goto error; + } nf->res_idx = res_idx; } else { nf->res_idx = of->res_idx; -- 2.34.1