From patchwork Mon Feb 17 09:50:03 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Wojtek Wasko X-Patchwork-Id: 13977394 X-Patchwork-Delegate: kuba@kernel.org Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2088.outbound.protection.outlook.com [40.107.223.88]) (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 E69912163AD for ; Mon, 17 Feb 2025 09:50:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.223.88 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739785818; cv=fail; b=MOt1JhM9LlE66cvbrF+rbteJquaXcVwMzbm4sEDg+dMYh1rpMW0R+TWfwg0IGFZ4Jg1kuBxLPTnNMzVKJInsARyKgQKe9uWFuR0xNDVT9caylwQURWyaOCIIRE2ZQbyTlacCrEVEgL5uzbjgMAc5zLeJgck/QNmHwm8T9tNNhI0= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739785818; c=relaxed/simple; bh=WvOVR3pym8bNDnS8pZuVsw4Foc5qVkKNQXCeTSogaBU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: Content-Type:MIME-Version; b=tbdsfM1DVols9zL0kpdCdS/SutHbhdJNm69GY7T2QvftI0VhAHCGfF4TusTgwdonbBoYZRW/9ophXzH/0iWuMhLpL7czVRPGruE+BgkWMkyEpN463cJ98wXJzbA1o6YYt2VWBKjxkKQ40CSjAPgonlSdUAStxlUGVb8iZ8UI17A= 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=r02LqE1j; arc=fail smtp.client-ip=40.107.223.88 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="r02LqE1j" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Y8dPf4P1xmU4i7tKdolG4/ms+aUL6rz3Gw/fZVOUWewdMbqW4tMvZr/ipWB0hij8BpXIvami0HulhTj/46GHuHfIqYFVgNC4U/fnwRSDq7pRvym75X0weRaDr8QGrzviCRTKLTSrQG/2BBP+pFyjB2Vnd4BiXnvYzXcFX6oDAjVBHu0HMYBRbOLJe876TMfrFvbpdafcqm6PA/uhskujkXoGp6c6ZopIxm80DN2r+Ep7/oBQIQtcShr2yakXnM5+VntNOIeJEO1rO/Ef+mbUfTKI43dlRSntFNH5VstPZ3kaHCysBWR1yQGBTomMJpcKiq8eD5S3Exc4SeVQWW07kQ== 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=ptKJjULvUOtqbRwLT8K+KYy6fNGjjyS99tIa3hDvtMs=; b=AumvN6k4I8Oh312oTPyjj9xppDR66Pymh9qIhjNPLUdttBky3oOExDxtxXp1cHwcQ+GauLZTcpK2CoiQFsv+s8sOCGn8iBUZ5yzB7UOX0MY9NaOrcORIcrB+30gnM8TJykNU/P/5k2zWH5naG30o8KJRrgh4yebxEwDszb9xE7hvb6KvyQWGW/Msu++nXzv5Fl5gFAf6/DhIJN354LOqaLBIluWLoSeWHzq0JGEKdqDdMd2iGgEWMd+7ljPPWFsTMQT+VvVKRBa6CNvxYsYhvxNInUDbUS8gWywLP6DyG4uBRdyiDHeM5gcAwMvM2tUbHxcOC6JKDRZ43Qzk6s23DQ== 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=ptKJjULvUOtqbRwLT8K+KYy6fNGjjyS99tIa3hDvtMs=; b=r02LqE1jgiEzPXayfTGRFvkDo5Jg80DK7h29G5U579voECgvfxjEHfJPkYAidWMDxojXCiD8Z59GXhH3ptVoiMS6/6IbdJm546y2FoohurOO6QNFFXuyqJz2yq4ST2g6gjvtDmrftjsomz878I+k7nDOFiDdnLFG3sSQuu3G6dzxBNL4C059+Hr9Se9XdbDuEkhpxKgC6KoCBjoVM1zJ3CxV3HwWyARpQ8pMN8rQdKw9HVa0WnJjicT/6Tc+2bGr3iR7rK/M1AjVqNYzNXRSGo9Fn3uUorwxfehNxpCTroa9wfnhdX+KofgDWpnYTLfxPNJjhpGYJnZnkGZHOqb+8Q== 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 SA1PR12MB7443.namprd12.prod.outlook.com (2603:10b6:806:2b7::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8445.19; Mon, 17 Feb 2025 09:50:12 +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.8445.017; Mon, 17 Feb 2025 09:50:12 +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 v3 1/3] posix-clock: Store file pointer in struct posix_clock_context Date: Mon, 17 Feb 2025 11:50:03 +0200 Message-ID: <20250217095005.1453413-2-wwasko@nvidia.com> X-Mailer: git-send-email 2.47.0 In-Reply-To: <20250217095005.1453413-1-wwasko@nvidia.com> References: <20250217095005.1453413-1-wwasko@nvidia.com> X-ClientProxiedBy: LO4P123CA0452.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:1aa::7) 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_|SA1PR12MB7443:EE_ X-MS-Office365-Filtering-Correlation-Id: 110d838c-3249-4806-12da-08dd4f387907 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|366016|1800799024; X-Microsoft-Antispam-Message-Info: =?utf-8?q?d1CZYK2JFd5TpixgeYM248sygg5dfDC?= =?utf-8?q?LmwMc4qsgVYpmKlLx2WiJTQ14bonRTKJ05/RUTGFXOMNsdqipZkyBPQ1qI/QsC8NZ?= =?utf-8?q?JGI+ZMSJaWAqXCJjmV1+QQ2zuSUgXWmTiw15fLPjWotZ7usW3CSRKafoehUDO1IMA?= =?utf-8?q?XEN5BzFhJ3dweN/519QyTxZVGAbgsLaMyX/pkVo+8RBZttwba0xGX1S3px+hu+jfg?= =?utf-8?q?mmwucWfgsaE7UHc5WCsqtGnw7gblsJocoXrwvl9+ubO8rJNcRELhyWz1RvdzjoT/a?= =?utf-8?q?ZSM3zRDb1jpVgSu1eBNJizBjV9MJnEk4MsdMzuAjvECRr8FL8aJePVZl0djs1N9/K?= =?utf-8?q?cFFSjtKTDUpogwtt52zYGT5c/KusKk6GGJDXEUAzHkgrFL76EfvSTdlxLLvTUGqmx?= =?utf-8?q?AuATNrbgIECcWqk0m0BLYV5XKW+VuTwVuEPAUKg2oGIAQBOvEjZxyH/SYSVObe0nU?= =?utf-8?q?FSPDaLfeQojKSUH6t1U7L3k2SScPU08glhraac049djR99wMEoD6BNQlxjkVxLvbZ?= =?utf-8?q?HZxcmCnMi9X5dr6e5GkIt1+hqFneAduATnLlvtvMuhTCNwCH3czDmyU+eph6jvTdv?= =?utf-8?q?BnwZwl+elSoeOBzShurhDEJpo1UysyXE93pzrxhPotnlyWUjPT8T2JRJrNFY9biJs?= =?utf-8?q?jpFCneDI9OLdqsHMklzoQPrJmbEnRPFL0qiAuHPEysUtvaj9QURrKdg4YHW2Q/s09?= =?utf-8?q?sMlgbxSe4Qaq/atHbLv9i7fZZ+M2dxhCxIlbE0BJZRydeo7mDnutdpSb3wNC9YgqB?= =?utf-8?q?ZS9tptwwpgRHMlfbHdZsmAs6pEIQo3Zg4eZ4YnxgqyLjEkaayOhBYgQDMjcnkYkT7?= =?utf-8?q?+ECECtsHrSlePsmGUSLFZAGik0YROdBOPDriRknH0G4t+XFaEG6HaOKbz3LYVu7qi?= =?utf-8?q?yqqk7ptvNoAlQJ8m74PnxWuMe8vQCOVL6GqCTtGb5A4mKk+000WobD49jT07PvSNS?= =?utf-8?q?BaxYkJFbPmr0WlLYuKc4pGxlOcnSwMRDGzpP+zo1uhZNMcEW17bosfPTRgIzoVyQ1?= =?utf-8?q?KOIHnp7PxZIfyJYzcl+O3/skQe/uO/jTkVxa2QLLt9/u/TezIYlUPXsWEOVZR2sdA?= =?utf-8?q?Z1nVQotjFsv6mEPELItxX+bKvelrg+a0vsVm+EcWGZzUfKAKP73uDsOxuTzh7rBpk?= =?utf-8?q?+IV5rIiqSkdHCydk7fGdK9Z+8gQPJX4H0tl/5eOHMrudUsw44pJVMgzK+HLNNNJuM?= =?utf-8?q?EZuWz8T4xtPgmf2bqcjUYAr+qPFGHjaVbH9nlpTyuVkC9erBh/Ul/fdV+BYps6XW4?= =?utf-8?q?Ii93hvaNBVqGeQ1dYcTN3SyohNdVfkZQnBsi95WPdgJHfRz1LYa/v0PcOVN3/b4V0?= =?utf-8?q?s32UzP6uIWNV?= 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)(376014)(366016)(1800799024);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?GAtYWmt6spw43/uXchvKymRTg8E0?= =?utf-8?q?v3JeZMqyFEONhFl5FadNrix5jA2dtyD9HghfFwfzDPjA+bDJq7lHZFrOpfb12+xeo?= =?utf-8?q?en8oczBwrqeHSAoI7KG892sLn/DDrpGwmhQD/mV9lLDcZG+uO4pMAxS+B9Hk/iKSL?= =?utf-8?q?ddDxeSJ7iQQtlqz3mtGIkfubI+dOgpBj8H+3DKY6KcpwtR0EUl4Avw9HuRhUr/bSp?= =?utf-8?q?Zf/RnPlvlqjtad2gkO6Q9zW+yW10uTUT29jLpkowtmGsd3Ootwt9vcNHA/sBy6nLG?= =?utf-8?q?16hh0ATFLGs/jr+H7OSkvT2gizW9EiiJ8CpaGnH6TrWLPBDor7bca6ISlLtXF+q4b?= =?utf-8?q?41jlqBftAOhSf4HqjnSgpcHtL0mFFsAKbAgIWOyS+GRiAHC3+R6VFUC9nfGMkMG1p?= =?utf-8?q?/ACyiY5+RKgY+t7enDAp+AUJ81uDDKlmRaB6YZ9Kh24kiWbYC7OWMQou2azAequKm?= =?utf-8?q?pcvxy4Nq8P9G7L7bEyPzwhMll9tpUbPhJUVjaVOIfanOIP5+ho0KwOVBvp0A1iFVi?= =?utf-8?q?hsI5/4ytssE0185A1YaDWtWMPdpXwyOirRotNG9w9ODO36QWCfYvOt6sM1KWPoX2v?= =?utf-8?q?01cc9lAHxhwiNUtEw6lmrYLFVuonlv8NTA7KuEFAyp1yqWPdVVqrAru98cqssz6rI?= =?utf-8?q?oNBVm3gHh9xg9Jcy0TUQNKvZEhcoo1qmYHvEmU9l2b3UhNaUEbTQK6TzKaBDfmUvt?= =?utf-8?q?y7kud/StF3NFG9xVVTdHDQNsOyMjaohWtUZdGV9Jc8jTTgfPN7CR4Hg40UYchYJXQ?= =?utf-8?q?XnjDrWsR7yrSbrHmhQuI5GGX6tVSegWBxO16OTBuJIVpFAmjrDgSwmrLL0m0OfG8+?= =?utf-8?q?kZoKIool81wbe0O4AGRb5GoLxyAgXdFf1HCIHdd1+8fyN5PGIeR4p00meMcytQkNN?= =?utf-8?q?EqtBjqBySDBihU5pXwjuSj3ROi4EiVuZyoka5tKFE7+RKfiHlpnuB1HFDu564g+rP?= =?utf-8?q?CKI9VIZfUC+SIPosJlFqG2gZoIZg1o0LdIH0OCVOi2xJmszhJ7VJSGFKRM9ahZsjy?= =?utf-8?q?q3N8BbRlzdliqRxvYyppHb5HhkUNwgvFmR1IFweDLJCRuzkTNKo1CsNUpRMhYTHRa?= =?utf-8?q?Z4Pt2p1v//b3Of0vF7bAfXrg/f8vgBK2uGOhLc3/Xb5IXqLSdL5Wt1EIQSI87g6MW?= =?utf-8?q?e3S+XJCnKjM79Ps2173t3iBOWy1vS1otALuWSe/FAosCfhGnpI+0F8z9/Aklw4Q+v?= =?utf-8?q?8vSgdG3ShZysUSg2tzaN1JhV8W1UA8NNfnNtrAlyYfokEF/K0ApyaTTeyIriiNEs5?= =?utf-8?q?4k7NOYb4ZqmtR49AwI0vHxbzFD/gHvpGaesjstOlZrwHesfEsaB6GDCLWPubnKe9U?= =?utf-8?q?anPTjKTw3tBPhITR1FB0qL03zPZ3J3FFr+KhhpeeoQu31Z0ttjTW5dHbTB7BfSDwg?= =?utf-8?q?bZwXV2u+qS2v6hbxx+YWjbbbxe3wpDb3jzZZt4Y2i1NghstSiJ2p8drlu6YnlFUeD?= =?utf-8?q?phkr9ugXsRUt/mNl7DjBs+UI+zgQe4DrLW7eVmaP9PMkvqgI9S0N6kECMjiewNTFF?= =?utf-8?q?E3oFFU427kMV?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 110d838c-3249-4806-12da-08dd4f387907 X-MS-Exchange-CrossTenant-AuthSource: DM4PR12MB8558.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Feb 2025 09:50:12.5779 (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: 1phzwT112o6KfgzFTlBQJcaOoANJBB23z5LHwdFn4V/i9d6Q1he6pJs/bXrah8DQIcLhyNNdy9KXvXlkfiVcgA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB7443 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. Signed-off-by: Wojtek Wasko Reviewed-by: Thomas Gleixner --- 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..40fa204baafc 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) {