From nobody Wed Oct 8 04:11:08 2025 Received: from NAM10-MW2-obe.outbound.protection.outlook.com (mail-mw2nam10on2050.outbound.protection.outlook.com [40.107.94.50]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9827278F5D; Wed, 2 Jul 2025 10:09:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.94.50 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751450979; cv=fail; b=PSDSjL4RzPjQ6Ok09bJEfnjh5GF8p7kigvAk9cCAURYhxmRzp04aIHvX+CLPjdz/LeW1V++bqAaZddxoZDPQOlftIZlFSkRcy3B1W7AJ0kMcIJsgI4dyCnTVblifvQdvtCVpIZ5Ih3RqGb2A+/+E1jtq0qvR+jiaoh25b9bEfQE= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751450979; c=relaxed/simple; bh=JA8EXN93ApSXWsPWx2pl+JqSLR27hsspDqNfz0PbHg8=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=fUGuDrp2JWa6lMcVmOEMxFhFjx7/ZzD46Gfp8ThqYZ/LVqC8DvHASaMcQwexVCED+4jOzD4F4SJfty6oeN0BecIGoLpnnZN0EtveLolbR0qqeOupJm1vSwrIPouOJyEn3Rz9Dt4kCPZRS/ETaBmMU2JWF2XFl+b8QzCgVRCZQLY= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nvidia.com; spf=fail smtp.mailfrom=nvidia.com; dkim=pass (2048-bit key) header.d=Nvidia.com header.i=@Nvidia.com header.b=BRW85q/0; arc=fail smtp.client-ip=40.107.94.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nvidia.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=nvidia.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=Nvidia.com header.i=@Nvidia.com header.b="BRW85q/0" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=cXkHoBRqS4rXyMm1f8tiEk+CfWLZ0qd3jFXTPxNdLmy0lXE4WIvoRuCEXfx5PB6yWYSF7qV8vfHJCe6fonJqn0xAjfiynW5cpCHjo2SE/HK3MwOILKuQi8NP24hdOeSGb4aBjubB/YEw5xU4mBt1wdWXA7add1gNuUakAF/8tpxQWQipuYsglMPnSoychqFLttiohymL17sScSouCMF/HyRucVn/rCxkvDRjH4tlswRJUE80RhkTvlkvsULEeX3TAJWjEE4nY5k+iKtut8ilDQlyrxPpQqkf4Gk4Edk8G9esw9HVjG1fmOkMUWn4dA27ffNeVITNlsqOUPIfPnOFdQ== 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=e4ikhJdKu2Ln5A99HTzKL8C3VPBtyMze+yDOoYEcyKQ=; b=r458pap6q5KWfhjID+XKM0SoMXy1ydg0lP8I0lgC7hBCSE0FILM3tN+wmovco7Ns74dLYWyrtZlRwxZA89Uj8aedZB7A+BbrjBhJWBaVOsVQeZIlU4VHaNmmeZjdBcqfgOlGU7FwVpLCvJv6Ecwv0meGFwAlbmZVlkTSktmWZS+puKROV/s8rgY2KoV54i+zY2cyIcWS5pr33Pe3ypzqAzpT91YjxAxfLSxZZBqNlKBr2b7ZE6dDjwE4ZLZIUvhQnqUJarqiFBYm2Xvm+gC5VaehCQsMEszYV2guO3bZPoxX4By4oNzE67wAku6LHlkV6N236jRu9TOuw0Ehvkt21g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.161) smtp.rcpttodomain=linux.intel.com 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=e4ikhJdKu2Ln5A99HTzKL8C3VPBtyMze+yDOoYEcyKQ=; b=BRW85q/0r5hbbBNbVXUGDkZibqX93T3KLSVtKuNUGYtv2VUpWs9QyRAZ+hCnrX7pUO77/4cdXvTM923o06eQHPTcie75dFlnmjLqwAnDoyk94aX1IcfTx2cLlOgkamuyhGib5+7hG8kbRmx2mj0YiolG3MHkpK6rU3F+MS2X/fP6FZVRZtDv8mvCMqOUBehrXoikIeYu9GBJKJETWLK0bgmNLK3dIMRKJdRe+ukYQFJAn2wBe0xbx2QaubrLBzjBviXszPr2aqrUR01bojpHEQjfsgF9BKEsP32isP7RqXtHrnCFDVe9SZhUN/yAn9ZLiPJByB0yJY9utZ0UOhOl2g== Received: from DM6PR21CA0004.namprd21.prod.outlook.com (2603:10b6:5:174::14) by DM4PR12MB5915.namprd12.prod.outlook.com (2603:10b6:8:68::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8901.20; Wed, 2 Jul 2025 10:09:35 +0000 Received: from DS2PEPF0000343E.namprd02.prod.outlook.com (2603:10b6:5:174:cafe::bf) by DM6PR21CA0004.outlook.office365.com (2603:10b6:5:174::14) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8922.10 via Frontend Transport; Wed, 2 Jul 2025 10:09:35 +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 DS2PEPF0000343E.mail.protection.outlook.com (10.167.18.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8901.15 via Frontend Transport; Wed, 2 Jul 2025 10:09:34 +0000 Received: from rnnvmail204.nvidia.com (10.129.68.6) 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.1544.4; Wed, 2 Jul 2025 03:09:22 -0700 Received: from rnnvmail205.nvidia.com (10.129.68.10) by rnnvmail204.nvidia.com (10.129.68.6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.14; Wed, 2 Jul 2025 03:09:21 -0700 Received: from vdi.nvidia.com (10.127.8.10) by mail.nvidia.com (10.129.68.10) with Microsoft SMTP Server id 15.2.1544.14 via Frontend Transport; Wed, 2 Jul 2025 03:09:20 -0700 From: Shravan Kumar Ramani To: Ilpo Jarvinen , Vadim Pasternak , David Thompson CC: Shravan Kumar Ramani , , Subject: [PATCH v4 1/2] platform/mellanox: mlxbf-pmc: Remove newline char from event name input Date: Wed, 2 Jul 2025 06:09:01 -0400 Message-ID: <4978c18e33313b48fa2ae7f3aa6dbcfce40877e4.1751380187.git.shravankr@nvidia.com> X-Mailer: git-send-email 2.30.1 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-NV-OnPremToCloud: AnonymousSubmission X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS2PEPF0000343E:EE_|DM4PR12MB5915:EE_ X-MS-Office365-Filtering-Correlation-Id: 6e8c8b4c-f8a4-44f4-7317-08ddb9508bba X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|82310400026|1800799024|36860700013; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?0GXJqGJmzxNJeLakRMGEYUm4bsPtTb5ZCdgwYqMYl7tso0GQUT+yseUMwUW9?= =?us-ascii?Q?NNBsSW509BcBzkIexfnbIkoPR28hLDIWwb82unoyXKU/NUkaObnodhJeor3t?= =?us-ascii?Q?P2rEQ5l1T2GLeXxfIWX5YRRtU13svVJE+JlFiupDQi3VxTEGmNKzuAc7QR46?= =?us-ascii?Q?ZTr3ZI7NZ8b/H8JRgJT7PeMc8f/LMdr7usy900UrV7UF2WVGDiwWr63B4XjX?= =?us-ascii?Q?W3z4nnu4GeD87GvqT7TLgd3Jwa9SSbY/7UtRX8JG2iHTONADreaV2OmpogNQ?= =?us-ascii?Q?2pbnWnNRwbGimG6zMDLblmAs57iUIHTI9cenTxlV1JLndQJperFcWbIuReNo?= =?us-ascii?Q?DIsw7wUzZDxY9XIE+45Emukr3FJ9UZ/FhMkPJfi36jwCqn+2P0aBwhgdivkp?= =?us-ascii?Q?Obcq3be50p/wV+R456CyJYo1R3sEgpYmVQWPZXjROCq2l4qHbgLdUfSjAU9P?= =?us-ascii?Q?SQqB3QZbPBk84D7u+9MZnN9aPbYHDo8txWDbuYoDtSzD34bBAG7dLxLwuqZ4?= =?us-ascii?Q?GstQUdz3aL4/pff4+Pr0Qk8H23PJePmQzh1BiDcc9iMsVNjR6x9LlxOVbcwX?= =?us-ascii?Q?jZUKJ1csNvfL8HtgXLWNXPN5bjelYvTaPmFjGDj6Xb1Lf99GLn8v07iHYYct?= =?us-ascii?Q?j+KBozrGv+uoIfupKj2cVgQTlMeaBUrkUkDz2v0GyoxGoWLEYbnExn/YPavo?= =?us-ascii?Q?MK8doF0qBHIGLvm8r6bkUl/SbBPiBwBObCrtZvV7aEh+09ebV5I5MFEaI5OE?= =?us-ascii?Q?Wy5vXB23FXbFVId/4c/77agBqml25yqdt5XcdNEEhWAZ/5YPbie2mGwt7ixq?= =?us-ascii?Q?yZVscOSlPVRJKb6HnLAPegrZuiEjh2oIaTp3YYBN9CRtdYWtmpDrd8/4Fb1a?= =?us-ascii?Q?UaVbQmnDPPkQ4NkprlCDYAx66lwHoY83uay5UeXH8YqE6eOKNw6F1HI/sX7h?= =?us-ascii?Q?qHA4go7Rp8AjIpJHm6qFp3B3vfFK9AXqCAAqgq8ZywgaUcMBdi2T9k4z/euD?= =?us-ascii?Q?HKswHgBgHkp4jSxFHREzuzRZ3NU2J3hrYkVb7K/aTaZv/HOt5yDgeKztyMFT?= =?us-ascii?Q?XGD+HeMyodk45Ri6JIaOh22REP8FbS0xhAyfrlLED6QROQKwTLwhIMxFS+A0?= =?us-ascii?Q?jRml6mJIwKLOEe9y6Y9ay43aHzBw3WAMmq0ut+4JfOqhSLWje26V/cC8flnN?= =?us-ascii?Q?M/sOyebnO0u5HQibjVVdeu0GQs7+13Ac99QUTHrET0fg++6Ol90FD08GdoL/?= =?us-ascii?Q?a+ueYIsFtB34PNUZZYWWZ7oMLKntEcK9hH/ltoYcfeKigk1wkIZ7liSmjnTw?= =?us-ascii?Q?Z9ezSCFlD7652B0b8lib+m+NDebYHQqw6iJx/Ym2oL2+JCaBHz82LjboX32o?= =?us-ascii?Q?w9hDI2LjeVr+6Y2mjLFzYwGUyvL8WWZKoKnJkQ88gxhbVCVvZPeSqy73UI0q?= =?us-ascii?Q?ENvqfympFhcE6/clioTbEjFVP0nVQroxAhEi1JOxi1h7ULHTTw7erJmELRDm?= =?us-ascii?Q?4UD4Im2yZmd9jmAH6VHYBHOclGnP9ZszK+iz?= 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:(13230040)(376014)(82310400026)(1800799024)(36860700013);DIR:OUT;SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jul 2025 10:09:34.8216 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 6e8c8b4c-f8a4-44f4-7317-08ddb9508bba 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: DS2PEPF0000343E.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5915 Content-Type: text/plain; charset="utf-8" Since the input string passed via the command line appends a newline char, it needs to be removed before comparison with the event_list. Fixes: 1a218d312e65 ("platform/mellanox: mlxbf-pmc: Add Mellanox BlueField = PMC driver") Signed-off-by: Shravan Kumar Ramani Reviewed-by: David Thompson --- drivers/platform/mellanox/mlxbf-pmc.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/platform/mellanox/mlxbf-pmc.c b/drivers/platform/mella= nox/mlxbf-pmc.c index 900069eb186e..9cc3d4ca53cf 100644 --- a/drivers/platform/mellanox/mlxbf-pmc.c +++ b/drivers/platform/mellanox/mlxbf-pmc.c @@ -15,6 +15,7 @@ #include #include #include +#include #include =20 #define MLXBF_PMC_WRITE_REG_32 0x82000009 @@ -1784,6 +1785,7 @@ static ssize_t mlxbf_pmc_event_store(struct device *d= ev, attr, struct mlxbf_pmc_attribute, dev_attr); unsigned int blk_num, cnt_num; bool is_l3 =3D false; + char *evt_name; int evt_num; int err; =20 @@ -1791,8 +1793,14 @@ static ssize_t mlxbf_pmc_event_store(struct device *= dev, cnt_num =3D attr_event->index; =20 if (isalpha(buf[0])) { + /* Remove the trailing newline character if present */ + evt_name =3D kstrdup_and_replace(buf, '\n', '\0', GFP_KERNEL); + if (!evt_name) + return -ENOMEM; + evt_num =3D mlxbf_pmc_get_event_num(pmc->block_name[blk_num], - buf); + evt_name); + kfree(evt_name); if (evt_num < 0) return -EINVAL; } else { --=20 2.43.2 From nobody Wed Oct 8 04:11:08 2025 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2067.outbound.protection.outlook.com [40.107.220.67]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 709D2DF42; Wed, 2 Jul 2025 10:09:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.220.67 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751450990; cv=fail; b=mVi/FdKiHeWSA5qGAsC01aWcguCeT94Eh4lLasdnC2zDzE1aXO8VuwoEPQbTsJ+ONqe9tbayvAzDniXSF9/9r+EuVXltNIMWzxKzrO015Sb0wTKkzWIxTyD2W+jaQ5J+91YTmYlqHF8cx153kZekJL5DLWZJw6+J4P3ic5sel0Y= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751450990; c=relaxed/simple; bh=QsI57GrS1gWCkdyDHeozCh0F89Av06K6oYKc2XbM8x0=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=I915R/yZcPm0bWtNdgX5P+vnFZ0cWUm1d7K5DRl68XQf8VwdjU61j74TKI7cWrk+UitwOT8ETS5q4oi8W4ZL/BUQ8gIwJA9vT2KYWQwwXEGvXauf0L5z9OlyORmS2AOX/Bkq3JXM1Ozv25IRGyIuUGsWnvlOn8Ha++LDeYzjWHc= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nvidia.com; spf=fail smtp.mailfrom=nvidia.com; dkim=pass (2048-bit key) header.d=Nvidia.com header.i=@Nvidia.com header.b=gBUpx0NR; arc=fail smtp.client-ip=40.107.220.67 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nvidia.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=nvidia.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=Nvidia.com header.i=@Nvidia.com header.b="gBUpx0NR" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=e9BJHZ5SErBYsF8ej85JH9MgbKzGEKXL+n8TKymBOp3ejzE2Z9LzTqhdGFY+L1lRW13TFUZ731Xko5eCL9NKZ0hgHmoiG828tMwuz5B+jTq5VY87L2wKL/pXLJtALEwpiwz4E6Nz0En8LYXKPXDG1IWlJa3027hMxyH8JrtsMFeEQms3vVxceuJGlYHJavuN9MyGh6edXQSV12qvd1RbYNJb6WgyU9FtNQl9Gl3lXw4oA4oNe+5V5eH6abbXmEAGszIIj6sZq4n1MMsC5hitVEa9GF8kdKLLHLG//p6ZbjCwtXTOgr4hlhMeqneCIMQz54r45bdtmqrTZyj7mxtz+w== 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=ObfOs9STmGJZmbI71Grb5vPM9rKQw2qCGQTppISr8qI=; b=xu4xCt1llpG/gX9KgLUq5JRPtr2H9WDTgNtrZWiljHUGxJYKfCLT72F4C7Iu5/jU/UJ2DOEN/98LW1rdceJ/cTZHcyky2suV/ZTWcWtBoGiAf5XlhPcE5/nx+HD4jjUWVV7o/RxUySY198EDgMpHmZhPbyH2QrSay5kcGmZlFECzuzXYV8UqJ9Duux8BlLqJDnA2GwBBoDyGzKDiwiaMMTRcxn+WTPxdEWU1bzdgfRnIlJns6PLx2JVnapCZW2aZMUX4gVHu/4rclnupOn7ikirM+O8b7btrej81mJzJT7eDKWnuwpqzNZ1ZhJuWil4mPfIWh//N5+sagcUXy42MJQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 216.228.117.160) smtp.rcpttodomain=linux.intel.com 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=ObfOs9STmGJZmbI71Grb5vPM9rKQw2qCGQTppISr8qI=; b=gBUpx0NRBQTgvm0WuAnh+UOw8Qopt9R/ck5BNQnJRPl67wdJ1+5TR4i5XwNRsUw7BtN6a2w8gmKRmAw2Z+cd7uJx+WxXxN42IQW3B7Ldha13nImenU7gisehPiVP5jhvpr5sW0XK/mP3DNorO/sR8AbF7cxX4sOQYJIjLdlbH4whmIU9WYgwkvLfA/PfMSHdur2tB1qosO8Md64PuBmwpyGi2Ah7rBlXzJwFct9F5Pjwnn10pg6UbwPfWMSCJF/nlzi2pC5rjiF6JXE29qN36ukxoPqA4Ov11pS5VM0RqmDRmdI6CfpOlyHJCs0Fwa8J1jBktkh2z2H1/lPYxeWxrg== Received: from SJ0PR13CA0138.namprd13.prod.outlook.com (2603:10b6:a03:2c6::23) by IA1PR12MB7688.namprd12.prod.outlook.com (2603:10b6:208:420::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8880.23; Wed, 2 Jul 2025 10:09:45 +0000 Received: from SJ1PEPF00002316.namprd03.prod.outlook.com (2603:10b6:a03:2c6:cafe::6e) by SJ0PR13CA0138.outlook.office365.com (2603:10b6:a03:2c6::23) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.8901.19 via Frontend Transport; Wed, 2 Jul 2025 10:09:44 +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 SJ1PEPF00002316.mail.protection.outlook.com (10.167.242.170) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8901.15 via Frontend Transport; Wed, 2 Jul 2025 10:09:44 +0000 Received: from rnnvmail204.nvidia.com (10.129.68.6) 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; Wed, 2 Jul 2025 03:09:27 -0700 Received: from rnnvmail205.nvidia.com (10.129.68.10) by rnnvmail204.nvidia.com (10.129.68.6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.14; Wed, 2 Jul 2025 03:09:26 -0700 Received: from vdi.nvidia.com (10.127.8.10) by mail.nvidia.com (10.129.68.10) with Microsoft SMTP Server id 15.2.1544.14 via Frontend Transport; Wed, 2 Jul 2025 03:09:26 -0700 From: Shravan Kumar Ramani To: Ilpo Jarvinen , Vadim Pasternak , David Thompson CC: Shravan Kumar Ramani , , Subject: [PATCH v4 2/2] platform/mellanox: mlxbf-pmc: Validate event/enable input Date: Wed, 2 Jul 2025 06:09:02 -0400 Message-ID: <2ee618c59976bcf1379d5ddce2fc60ab5014b3a9.1751380187.git.shravankr@nvidia.com> X-Mailer: git-send-email 2.30.1 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-NV-OnPremToCloud: AnonymousSubmission X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ1PEPF00002316:EE_|IA1PR12MB7688:EE_ X-MS-Office365-Filtering-Correlation-Id: 82b948d3-9b28-4c7f-d82f-08ddb950914c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|82310400026|36860700013|376014|1800799024; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?4t1WK336JMln1RpBA5REGCUvubr3xHvMhHU40kmn13APOt5gJ/f7OPXAKXwW?= =?us-ascii?Q?cLucreQRQ75WrxkvTAbLYQBKlWNdkemHgEAMHs+a2cw6xZcLDOkTMOmTMASU?= =?us-ascii?Q?MZ+X9gCSeMC0l372i2nKy+zvzJLcVyFifN0laX09mb1EJCAccfb/0L9WXvmK?= =?us-ascii?Q?/QAlZ0v50wNTVWXnjvUJuaRMGPtdw8Xby7TR/G/4++ULRFlHF/skCW+3FvhF?= =?us-ascii?Q?QTJCl/HOXy4QWuj56O1A/ve0ZfyQ5rNZIFZ+rkDdovBKT0Eb4LYMKjIqfRr2?= =?us-ascii?Q?qtqlqaGPdPJ6BPxIYHZ+j4d+x4macc6f5D8uZJoumGU1gF3/gM7KivQZ0JYQ?= =?us-ascii?Q?PlV7CEXmfc7nz1VllcqhJLntUTAIz9w3PMc2eXG14fsDxHlc7r5xPNKQ11mO?= =?us-ascii?Q?ID7vfb3TrDsyG69SpD8vrsL+6nFwmlMEtwut5S9MFcy8XxshxlNIhBMoAqCE?= =?us-ascii?Q?1mpBIwrkuV51z2clqT+1pL9WJtr2upDHkr7TAPl2UHGaXFgnsvBg71NUAFNA?= =?us-ascii?Q?6jtL20QC2yD8claUQWrbB/B2Cwl3nFSogYwsg0I+HEA49TfwBHKvIbIrGH/v?= =?us-ascii?Q?PkoS5Y+wGnkLTD6TzhPj36bMxq8l8bra8uqIrnOiEzPblHda53mNi9kzENHs?= =?us-ascii?Q?VUMLzjzKHpexsUzTJcR+QgLHddiI1G/pVSiI9MtnHA4TvSCQJjzGe2eg5le/?= =?us-ascii?Q?P21ZPidC2BJtq2S33+YiC0GZDwEXu9sC2C+gWYqalsrHt2rA5tLWpB5zTnml?= =?us-ascii?Q?ewf2EisuarHj/++8NVM2/2wjMIuhVZnUpuy4QEMQ73AwrduFx5a6XsCPamTi?= =?us-ascii?Q?tK7EBgwKoHIEuFGXjs4y8P9plH+Vhn2aY7RtWpYqm1tW2XFCKKh2sKjINn5Z?= =?us-ascii?Q?6M4SXvqHwGl5p1T4qrJKBfFRed/AOyCw06XYesfNJbY16wSdHH2zMekhrNNq?= =?us-ascii?Q?wKF7G7VMsUbfoKap0ITWlC7UdhsabPHc4QL7eHxyQsdFbUI7WolgItYI8MKL?= =?us-ascii?Q?cEBUEru8fm6wBPwI5NqVhp1d8pomCxNo++r8yFiwy2XAhubSrajUhB6c/NW/?= =?us-ascii?Q?ySSck27YvUomB7NvyeiNaT3WBdSPXVVKxqj25d3Di5LHHNSqmkWQ9yvE0hhh?= =?us-ascii?Q?xMOAq+Jx8b7a1o+WI07rIh8rtocuHDfAWQAOhTuk8X9NMy6eMBBbljTILMfr?= =?us-ascii?Q?0hhp2fFoJ+6fkp0dLfOa1S9kowQK9wSccwR4rVa+DtjAvopA+V8XTAncJD69?= =?us-ascii?Q?y/SnNjsGizfmmdyi9RRFFTj82meeYDKS1HyDJ/aqaBDSUJ9Q/ibQnA8XdkzE?= =?us-ascii?Q?ZGRV8HIh71ID6weOTrx/4+it/Tp/CHK+U5daXYsgTElgpsVuMzmJ0m7XsNT9?= =?us-ascii?Q?dWe32u+AN21c8sWmDzuCK9hQdLYDGnks1bNZfYeaMMRdecNjSfaW6dWlLF0Y?= =?us-ascii?Q?CvokZ5hqmx3yg+q8P45ITnDwWeV82FoPdcm+w+6+D36hg3SyBqMjx6giL/k7?= =?us-ascii?Q?4E661htQY3VQjaLCb66sMdi7/u2YUKJwWDEP?= 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)(82310400026)(36860700013)(376014)(1800799024);DIR:OUT;SFP:1101; X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jul 2025 10:09:44.1421 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 82b948d3-9b28-4c7f-d82f-08ddb950914c 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: SJ1PEPF00002316.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB7688 Content-Type: text/plain; charset="utf-8" Before programming the event info, validate the event number received as in= put by checking if it exists in the event_list. Also fix a typo in the comment = for mlxbf_pmc_get_event_name() to correctly mention that it returns the event n= ame when taking the event number as input, and not the other way round. For set= ting the enable value, the input should be 0 or 1 only. Use kstrtobool() in plac= e of kstrtoint() in mlxbf_pmc_enable_store() to accept only valid input. Fixes: 423c3361855c ("platform/mellanox: mlxbf-pmc: Add support for BlueFie= ld-3") Signed-off-by: Shravan Kumar Ramani Reviewed-by: David Thompson --- drivers/platform/mellanox/mlxbf-pmc.c | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/platform/mellanox/mlxbf-pmc.c b/drivers/platform/mella= nox/mlxbf-pmc.c index 9cc3d4ca53cf..9aa8de1122e5 100644 --- a/drivers/platform/mellanox/mlxbf-pmc.c +++ b/drivers/platform/mellanox/mlxbf-pmc.c @@ -1223,7 +1223,7 @@ static int mlxbf_pmc_get_event_num(const char *blk, c= onst char *evt) return -ENODEV; } =20 -/* Get the event number given the name */ +/* Get the event name given the number */ static char *mlxbf_pmc_get_event_name(const char *blk, u32 evt) { const struct mlxbf_pmc_events *events; @@ -1807,6 +1807,9 @@ static ssize_t mlxbf_pmc_event_store(struct device *d= ev, err =3D kstrtouint(buf, 0, &evt_num); if (err < 0) return err; + + if (!mlxbf_pmc_get_event_name(pmc->block_name[blk_num], evt_num)) + return -EINVAL; } =20 if (strstr(pmc->block_name[blk_num], "l3cache")) @@ -1887,13 +1890,14 @@ static ssize_t mlxbf_pmc_enable_store(struct device= *dev, { struct mlxbf_pmc_attribute *attr_enable =3D container_of( attr, struct mlxbf_pmc_attribute, dev_attr); - unsigned int en, blk_num; + unsigned int blk_num; u32 word; int err; + bool en; =20 blk_num =3D attr_enable->nr; =20 - err =3D kstrtouint(buf, 0, &en); + err =3D kstrtobool(buf, &en); if (err < 0) return err; =20 @@ -1913,14 +1917,11 @@ static ssize_t mlxbf_pmc_enable_store(struct device= *dev, MLXBF_PMC_CRSPACE_PERFMON_CTL(pmc->block[blk_num].counters), MLXBF_PMC_WRITE_REG_32, word); } else { - if (en && en !=3D 1) - return -EINVAL; - err =3D mlxbf_pmc_config_l3_counters(blk_num, false, !!en); if (err) return err; =20 - if (en =3D=3D 1) { + if (en) { err =3D mlxbf_pmc_config_l3_counters(blk_num, true, false); if (err) return err; --=20 2.43.2