From patchwork Mon Mar 3 16:13:43 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Wojtek Wasko X-Patchwork-Id: 13999155 X-Patchwork-Delegate: kuba@kernel.org Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on2059.outbound.protection.outlook.com [40.107.93.59]) (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 4FBAB22B8C1 for ; Mon, 3 Mar 2025 16:14:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.93.59 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741018441; cv=fail; b=tSwRcjCJuOaV9w15AJv4oYJrL7rynZEwyQEwMe1sKuJa5E2eZqYgVm84nMbgqMcOw6y8kB05/6YSf54o4Vd5WD7stuil2A2UQiX5k4SvLt6IBgaxHyw6HscDHeh4XbCqabypIPG2mQcADAF8getQGvXuoWVhH0F2PuzrM2qldlk= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741018441; c=relaxed/simple; bh=KqWU23so06PZFyLp3WE3tqYxDkHldCy1P3kvJDrQbl8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=bkciB9jyXxfUciXbODTUyhFFwMNwCPQK57mbiomhrUAlmKTQc0YOcQi3fxbtQlboMm1RJwDfn4c2SAvu2djMSpPbRXGfO2RbYcKeoz6CedkHR01My94HlLYr1JL/XwC7PY5YNeioCbXJ84UUYia4DK3Adl4+bbbD9MDTfacnONk= 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=qKvWM2jn; arc=fail smtp.client-ip=40.107.93.59 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="qKvWM2jn" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=RaFOMOU+O+FUYoQdwaWs955To8qEKCdL8HlPfOb9klnYeo5s60FdLZummkC/IjnBPp7LLWtx/h3an2LcGa5FQJiUAXrnK0ywbBUCzAfqPAFMJaY9sNTi5elbC02H7eG8RoG43nNMbCOOoNBoU5khU784T1FKAhQcM3NVBjTycxUjO4rsLRXFra5Gwz2NqAkjdNp3vNilAmDhJtbvWOD49U0e9boGX+kKWLC1zJNKDOH4Rmpvp2zXp+t+wEGfIPU7N8XR/JinAjKvew+QgCntbdBmtNFiktY9U2prDIk0DAv+HK1Ke8pWa0Zf9Dz0oMCLez5yiVcwb+FBGyZO+h0r1g== 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=8D/S7GQPnLgXnKNIubANsV15nNWPvprc+rgWdPTONCA=; b=kDxo00rXZXe94twRXJOXYvIC8KIxoaFUYyNjc/UBVewn544iGt6QNDiGnS8ttGPxFc0t+aaFjRjReTyXxleXMrSvByewQSKbgViFgbiU8l5yehHjeO4GaPTWL5kpesfGv5LCxFNEZQ1G5qQ4TVgmnjpjCWWk/ms6StnMh/oRcrLJn6jH+jT8GLl6BqU1qC9pSa5HMXkBpmyWZSMNLRWn3M7MF6Hkgs1NIH7pjsQRXYcwvk5EcEMsgEvYrxQaAUze5z/Zr6LqZ0hXA7vFbWsZ0h6PKPmo03osJPuZhwoK6yxfiCukYEecfBSbow/IYV9IilJvgGrpdwklxztgCnIu9w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; 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=8D/S7GQPnLgXnKNIubANsV15nNWPvprc+rgWdPTONCA=; b=qKvWM2jn5ZLw2pfbrRGEd1kMl/msAIR9LwvWMHN2CIT+wPiTavmA7Tl1IPUxkCvDB35da32LzKTeUXHAdpNFSeN/kdVwnP/JTqAxJgQ5i1I9FiEqqAXSVlgj11rg+IFmTEkqH6BV++Ub+5onXlUhpl8KHAPKXi/yiv1oHullfdV9jaERcd70t2tI+nSbKMgsRsheEclp/zkVW5AplIS0dvNEZRcQo3hMOY7VBR9iyjuwB6nzYgRNz8P+3RPvrnUC0xnchMCfYb9s8iMT4OTYFveOUZ2VWQOFZjeBftu/8OZZyxFPhb6UcGa3Y1w1pF4q5S7AuT7tvOugpIFnxYeI9A== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from DM4PR12MB8558.namprd12.prod.outlook.com (2603:10b6:8:187::22) by MN2PR12MB4094.namprd12.prod.outlook.com (2603:10b6:208:15f::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8489.25; Mon, 3 Mar 2025 16:13:53 +0000 Received: from DM4PR12MB8558.namprd12.prod.outlook.com ([fe80::5ce:264f:c63c:2703]) by DM4PR12MB8558.namprd12.prod.outlook.com ([fe80::5ce:264f:c63c:2703%5]) with mapi id 15.20.8489.025; Mon, 3 Mar 2025 16:13:52 +0000 From: Wojtek Wasko To: netdev@vger.kernel.org Cc: richardcochran@gmail.com, vadim.fedorenko@linux.dev, kuba@kernel.org, horms@kernel.org, anna-maria@linutronix.de, frederic@kernel.org, pabeni@redhat.com, tglx@linutronix.de Subject: [PATCH net-next v4 1/3] posix-clock: Store file pointer in struct posix_clock_context Date: Mon, 3 Mar 2025 18:13:43 +0200 Message-ID: <20250303161345.3053496-2-wwasko@nvidia.com> X-Mailer: git-send-email 2.47.0 In-Reply-To: <20250303161345.3053496-1-wwasko@nvidia.com> References: <20250303161345.3053496-1-wwasko@nvidia.com> X-ClientProxiedBy: LO2P265CA0123.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:9f::15) To DM4PR12MB8558.namprd12.prod.outlook.com (2603:10b6:8:187::22) Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DM4PR12MB8558:EE_|MN2PR12MB4094:EE_ X-MS-Office365-Filtering-Correlation-Id: 69cfd729-fcb7-4a9c-6e03-08dd5a6e63d7 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|376014; X-Microsoft-Antispam-Message-Info: =?utf-8?q?5qi2DkPPrWYqem+QFG7lRBwZSKgVF6t?= =?utf-8?q?pyAqrc5Ri+FlCLYio7kgW6YN6zTWaRj83MM+wAtY0Oovd8uanR7zYLqBP6mzAGvo9?= =?utf-8?q?cb3Dn8AsEf3amILWuHtgD1hYlXAj16hT9AlN11kHO5Q1zZ1bPS8DrnYR9/Qvl0ulK?= =?utf-8?q?1Aw8kr0C/SRjDfZ5enR//6gNWbvf9p5OFiPOQbGFcyS0q9c0udD4Q+vyWc3x65Tdw?= =?utf-8?q?KO6hrNieumzaiygliqetQyBAxuuSN3X/YiXT+Y0629di7qdhQRnO01qVBM7AsCFg3?= =?utf-8?q?8TOxPiIDx40icIdM1psSIz6nenWV+PTs1QRBArmr0t7LUSPWP1gsjj9VjVj/2EMQi?= =?utf-8?q?LNwK2bPIw9YdJ1zzttPrH4bZcxflDJWbAX9kFjREuJTvfP3JggWqs2ApkiBnBMQGy?= =?utf-8?q?1F4ahOfMNclao/WHzfwEEQX+AHe7hVvlzX0EKzRsSmvHmt92ONN8Dw27Qi/WJeDHC?= =?utf-8?q?u/3Mh3N41CV0iN+s+IrQU+K0+QHbfwJPe1Ru/aD15eOnfOYfBG4V4el3dr5V9U6Fs?= =?utf-8?q?dcNmVSk44Xls2DZyY9YrqeR2sBBdfWwMGNJYVQIDTXmoc0ftlpjdZVvpnwx5l7rn6?= =?utf-8?q?zlyxq1d5ZaTZI/1+TCksuAg45sIEhw4XRzmAk0SNdUTXTHZMkUDGR2cec4R1mH+1x?= =?utf-8?q?6bjD4AX/XyauysfDXJArY7cuLWiFuDKckbbo3aM/A+ZcOOeJtaYHrswOkTy80tRwJ?= =?utf-8?q?8FhZs17/qeRVOEwh4zTbLJ0Sn55nfWU3RTskT74YqdKql9vf5TcDI8S7MWz2iKWsA?= =?utf-8?q?tAwi3q3nIodf8BjUG0UUiB1n47mkh58VfXJgp3u8B2nHFuNoJQ3pcbOmNq7ozOb3m?= =?utf-8?q?mJVlso4opdkqAJaK099aj1Npx3neMqQaiamdtF86/1wAuYGN93uAAhy3esKj3HYNr?= =?utf-8?q?cJrH7DFhtls/RTS+afbq1CFC1lZsiA5bvrVSHWlznwiYKIOBsfkiXX+EP9bQctM4s?= =?utf-8?q?y8bDLSAQf29HOyveDESBhsfOJOXDK3NL7APl9vFxQd+rYaU6zcNqM/TdvMT6YQwD4?= =?utf-8?q?gAuPV9QOm6j+3xUfZJvB/M+M0pI7r+YmRa5hmyGv0dfyyJ3qypDUUGqlf/05Nuj2y?= =?utf-8?q?VEhmTVQFZJkYcixTEq7vudCuaC9rmR1536sJB5WKxzha4aq0A0t1mDtJOBsfDUFTA?= =?utf-8?q?zQOp1Z9XZC9JoDXjeimwAbH5iQAnFZTR35Li0Mu0bmiGd5+jVGNIznEO6PntKODIZ?= =?utf-8?q?Q4fmtmTpT+a+pUfc43L2kV5oxHLFikB4l/zJYpQMhpg+zAuBjTd+Lp5fpTGNJUUf4?= =?utf-8?q?pdGYnoz6PY+PkWIszhQ3cbqrJioRik5179HiU6mY2TzcAyMlejSNJRYNfab6RkFI8?= =?utf-8?q?cnM/drl82z8B?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM4PR12MB8558.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(1800799024)(376014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?2og21j0hRKQqkm4F9bc+IbKoJJew?= =?utf-8?q?/h/CDMAzd8VkkBri0YX93MH8H83ztMmLCJeNasKH1GZ5FjwdFIGJI1rmtTn5IMBbn?= =?utf-8?q?V50Q3wHAEuy17jQO45sM7IlSVsEjzm2yZ1ZXota74TfseBkpHhRGoV2xpADJon5iY?= =?utf-8?q?+897Ytzjpg4PqSAfZtCYg86eMlmlGXAmhVrTR6gfS6czUk2ypO8dK/S95pZv3L2fA?= =?utf-8?q?wEuaZBrFFKmS4+EJWDNHuHug0AWa2sCUatfNUbQdOMvI69UAXuARqDv+EWjhP/pi2?= =?utf-8?q?MhoshH1iKRO5M768wr0YRQQ+VvAt/hgPXnWrzmR2zZX10dfX/T8PN5YSOWfYxcTS/?= =?utf-8?q?kSORLXkonFAcgIP9+yseZtICA7sM2ifsBOghkrFDD1ptryAAuNP6Jt1DQgsjrPLXn?= =?utf-8?q?VtgIANmWPsgzDncFiwRJWGMVamNNGLh1J2dC1yQDOrLvQTCC/Lc86l9CXyzw75IXX?= =?utf-8?q?btS/9LWTN9AcOxa7gKOHajB8dFr1xXHy0ebfGIAUbOLsfsySJNRpHKICVevuv0C0J?= =?utf-8?q?KhawpCwEvyRVMuEnovqzk0QEUhxX6iVgA0F/9E8JvY/0/CTu/Ip0zsYqL+eA+txIU?= =?utf-8?q?nt+vukgO2UYgB59+W2VL99JtIIq0i4Ql22i2SKsLcj9R8o6dF+7w7MLpzKaQKYyjz?= =?utf-8?q?L+hRncUyFIeGElqoJjGfKLLfj4gLN9cl7eWHiZO2IPqsQlcjpWmRkIcEhA0Z2Oyq2?= =?utf-8?q?HQ0E95Z+MlaQUEZyMIZb4i4gpY8N5LQp9hA6kyo1ePI4szS2H/yFQfO4eO0i+x9Qa?= =?utf-8?q?5d5x7bTz5BpOEcu80B+EomjzLleq1ENtffKGH/5ZjHyOdNi5xnfeUr9GumHwuVBYZ?= =?utf-8?q?ZIUOZcX2Qq466LEveIfAMICdaeK3pq73NY5NP1qzBhjS0LbyNxA77YUMZhQ9XZ/nV?= =?utf-8?q?e/vJ+IDzMnN3R3xgWFxAJavWbozo161vun0MgpoStcqkgcxiGTB+w6oAzemVZn3wb?= =?utf-8?q?j3wHPQlY+hRnb1y0w336WBq7gdmh+Mgp/p1QcpIBhjUEpikVV3YU/s3j4Px+FC6yC?= =?utf-8?q?UD6PyboWG4Ph3P9eQiZhwRI/gP3GjIPSzLTulEb7KfRgvhNQB1IkwtiGXVkT6RzV8?= =?utf-8?q?UFKfGUv8nliLegK8AtfBTFpUXbDuDrdqc0gsuklRP7NbGKl/HvkURVFbPHVa2c3Sz?= =?utf-8?q?8fn0kcamFxvnPdV8edAqej26qC6vL0kDxEUxyWCQE8tIoObgNXSsdxbxOyIG4sorV?= =?utf-8?q?go3Bazsor0ukg+v+e+Qq2WnwL5FM0KiwLzjwLaooxmoqHWJwtkY4xSxemQ6FMlecm?= =?utf-8?q?M4FVF1HucXenpWnAvZqSHM7fmOSyZEc675EPVGi+RVw02Of8rLUJiQjM3gD5zUBsu?= =?utf-8?q?KHrflmIdhujlCzFowafLyJJixUzTtI55kiJokrkz8wbzH+PQUjY05JDfLUmsVmAhb?= =?utf-8?q?yGdtaaUaHQZi/0lEzAK525oWyNJsb4upZwbN5h5DRSS5J59TGPiw4V4RRj6V4xNzL?= =?utf-8?q?HCEDSlNG/z6Yfk8IUhX7BCIV84ftzN9c22qp+MkqgVEczkmzfpIbsHyMfxxlODRUp?= =?utf-8?q?ZlaW447yfAWP?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 69cfd729-fcb7-4a9c-6e03-08dd5a6e63d7 X-MS-Exchange-CrossTenant-AuthSource: DM4PR12MB8558.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Mar 2025 16:13:52.8370 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 4mu+mG6YBu3j4E9sEwaB3Ipa94dD+XzNKqIP/yKAAHUTHYwdsA3OLp353vsS8RPG29WzjaW66xpNOutEdElKKw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB4094 X-Patchwork-Delegate: kuba@kernel.org File descriptor based pc_clock_*() operations of dynamic posix clocks have access to the file pointer and implement permission checks in the generic code before invoking the relevant dynamic clock callback. Character device operations (open, read, poll, ioctl) do not implement a generic permission control and the dynamic clock callbacks have no access to the file pointer to implement them. Extend struct posix_clock_context with a struct file pointer and initialize it in posix_clock_open(), so that all dynamic clock callbacks can access it. Acked-by: Richard Cochran Reviewed-by: Vadim Fedorenko Reviewed-by: Thomas Gleixner Signed-off-by: Wojtek Wasko --- include/linux/posix-clock.h | 6 +++++- kernel/time/posix-clock.c | 1 + 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/include/linux/posix-clock.h b/include/linux/posix-clock.h index ef8619f48920..a500d3160fe8 100644 --- a/include/linux/posix-clock.h +++ b/include/linux/posix-clock.h @@ -95,10 +95,13 @@ struct posix_clock { * struct posix_clock_context - represents clock file operations context * * @clk: Pointer to the clock + * @fp: Pointer to the file used to open the clock * @private_clkdata: Pointer to user data * * Drivers should use struct posix_clock_context during specific character - * device file operation methods to access the posix clock. + * device file operation methods to access the posix clock. In particular, + * the file pointer can be used to verify correct access mode for ioctl() + * calls. * * Drivers can store a private data structure during the open operation * if they have specific information that is required in other file @@ -106,6 +109,7 @@ struct posix_clock { */ struct posix_clock_context { struct posix_clock *clk; + struct file *fp; void *private_clkdata; }; diff --git a/kernel/time/posix-clock.c b/kernel/time/posix-clock.c index 1af0bb2cc45c..4e114e34a6e0 100644 --- a/kernel/time/posix-clock.c +++ b/kernel/time/posix-clock.c @@ -129,6 +129,7 @@ static int posix_clock_open(struct inode *inode, struct file *fp) goto out; } pccontext->clk = clk; + pccontext->fp = fp; if (clk->ops.open) { err = clk->ops.open(pccontext, fp->f_mode); if (err) {