From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 93366D30011 for ; Fri, 18 Oct 2024 14:48:52 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 229906B0082; Fri, 18 Oct 2024 10:48:52 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 1B0276B0083; Fri, 18 Oct 2024 10:48:52 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EF6F46B0088; Fri, 18 Oct 2024 10:48:51 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id CC1F06B0082 for ; Fri, 18 Oct 2024 10:48:51 -0400 (EDT) Received: from smtpin26.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 4F9EF12090B for ; Fri, 18 Oct 2024 14:48:40 +0000 (UTC) X-FDA: 82687004484.26.57BFE4A Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) by imf26.hostedemail.com (Postfix) with ESMTP id 9395114000B for ; Fri, 18 Oct 2024 14:48:39 +0000 (UTC) Authentication-Results: imf26.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=NDsx0qJP; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=jgqTspVn; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf26.hostedemail.com: domain of lorenzo.stoakes@oracle.com designates 205.220.165.32 as permitted sender) smtp.mailfrom=lorenzo.stoakes@oracle.com; dmarc=pass (policy=reject) header.from=oracle.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1729262781; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=rgCfDZ7CBl6KlmFw4rLxVAEfXodU7DDfMmfDq1TwuR0=; b=X2iGTGj1m0w2bn/ObPWxjMALxXg+iE9O/nqqm8DRMWzgV4KIHk/yEh5vrpj1HqncZAc55N MIOdOCL2XoucP2LCuYaRvv/xhwoJVmSdh2keYv4ETpNjJQFa0QJKsgs2U/FmQX5vE6Fbq2 NZ7N3l6JrxUR+CLrU2GksTsCngT1N3o= ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1729262781; a=rsa-sha256; cv=pass; b=OffUO8Zo6oXeiGUbuqeD14M5xF5sMMifYxMKshY0tD0oZ03EwOcosJcA7Aowf2COmOwV2F NlFd2qVohKmi5tkzGIY2c8Rhaqc/k95gxmdizBZB3bPqJznyoB5W9Lkzm+dfCZo5XOhGr7 46lKfFRKrUxD/6GDIhC279IKCVQAjyA= ARC-Authentication-Results: i=2; imf26.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=NDsx0qJP; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=jgqTspVn; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf26.hostedemail.com: domain of lorenzo.stoakes@oracle.com designates 205.220.165.32 as permitted sender) smtp.mailfrom=lorenzo.stoakes@oracle.com; dmarc=pass (policy=reject) header.from=oracle.com Received: from pps.filterd (m0246617.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 49IEBcMq017041; Fri, 18 Oct 2024 14:48:30 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=cc :content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s= corp-2023-11-20; bh=rgCfDZ7CBl6KlmFw4rLxVAEfXodU7DDfMmfDq1TwuR0=; b= NDsx0qJPGBKdi7vxpkOsz2OA4yYJVQWRJ765k/y/Jp+r2tGispWEWm6NIV1jr0yZ rIoV6omtTpany2+OvbbG6ZhcfZhXKfs8bkrnYwBk7ui2HFY+faBHVQv5hD6SC9F5 4cwc1+dex/pTXOXCit/yADer/Y6MLZEXAtg9/9pqg5wds19GQfXnOsri9HkRQlRc oAeILZluzX64p4EAtiQROF8HBk6fyWmiZvX9z5I0i7VI6MiFjk5qevuCe0498Bsy Nlk93/4cTxvFvG+FjgC4mWcldDGbZ84eiLs4j2xW5XgrILDsIbW8kEA0mRIu+0lT RibUVRr/HjS8/6lOrzcrNA== Received: from iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (iadpaimrmta01.appoci.oracle.com [130.35.100.223]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 427hnth0gy-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Fri, 18 Oct 2024 14:48:29 +0000 (GMT) Received: from pps.filterd (iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com [127.0.0.1]) by iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (8.18.1.2/8.18.1.2) with ESMTP id 49IDBggZ036092; Fri, 18 Oct 2024 14:48:27 GMT Received: from nam11-dm6-obe.outbound.protection.outlook.com (mail-dm6nam11lp2173.outbound.protection.outlook.com [104.47.57.173]) by iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (PPS) with ESMTPS id 427fjj83tj-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Fri, 18 Oct 2024 14:48:27 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=sdXoaoYfDr/ZEvRAttwmOKGEZmOeNIIz/O6hDh9APf117oDGln9fylcOASHvcDwjxfwcley2M/0bvZS26a98tOz7+AbLL7aIV0C/p2oS87++jm834T0t+qCMADYBXg9UvChV4QQ6AVdQxJKLuPLXYDJNCc6zxGgOecNq1iM+FVRO1WeNIjH78ViuUL+nP8zcC9d3FbkfABYbfIp2bb4tBLVkAEDqJLwim6SV74L66Dyn0IDIbgqPaHsiJdONQe6Vugaz+jCEHYCNO1WUS25+KYUV8uu2IKA2Foi+R5emJG/GPpDTuYv54QDZUZFbA6T5FHSTVSH3ooXbV/hI4epMKA== 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=rgCfDZ7CBl6KlmFw4rLxVAEfXodU7DDfMmfDq1TwuR0=; b=gRTue4sorlwFeGpH7vpKlZhwI9YFgqTzZYJu2bh3Grya5SCDXYQgG1S2nqTMYDMvGqr+Fboyg6lA+Kv59Gu/fo7l9c7aoTt61hpxVUTOEpfM276HcAS/aHpY5R/1roW0onD6Xr+5LwmnR12qqdrySCppg0ij+6qsx7IyvwhkIExbTReiYBoSD36hiGU5F8owuyyFZAsWcv77TpmgFUQUiLZ8c0kFvxILlOK7C8B6Mr0BAxlrO0BApSRTIz5Wiy6xZ6SeA+0VdlLIdMCgdzljMTSavSFYFRoy4qm7LWCUuNaufx/7l9ES0NN+WkVuw6QyxTghbf1ObY5ADcOG73/WWA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=rgCfDZ7CBl6KlmFw4rLxVAEfXodU7DDfMmfDq1TwuR0=; b=jgqTspVn68U2+whE+JYeYNxmavDRJCzGw3bmt/iTa30m4U3lLZF0SvSZN8uKCH/aPLnlEgUbKjPvealsuKlgnDiQhJF+eaNzEnmO0Pi8w3wwnS3SgB1pQOnIJg/ivUpNDmE+Es5H83OtevvASW3GVW+fg0gNGHFsfX1GatJygXc= Received: from SJ0PR10MB5613.namprd10.prod.outlook.com (2603:10b6:a03:3d0::5) by SA2PR10MB4700.namprd10.prod.outlook.com (2603:10b6:806:11c::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8069.20; Fri, 18 Oct 2024 14:48:24 +0000 Received: from SJ0PR10MB5613.namprd10.prod.outlook.com ([fe80::4239:cf6f:9caa:940e]) by SJ0PR10MB5613.namprd10.prod.outlook.com ([fe80::4239:cf6f:9caa:940e%5]) with mapi id 15.20.8069.016; Fri, 18 Oct 2024 14:48:24 +0000 Date: Fri, 18 Oct 2024 15:48:19 +0100 From: Lorenzo Stoakes To: Jann Horn Cc: "Kirill A. Shutemov" , Roberto Sassu , Paul Moore , ebpqwerty472123@gmail.com, kirill.shutemov@linux.intel.com, zohar@linux.ibm.com, dmitry.kasatkin@gmail.com, eric.snowberg@oracle.com, jmorris@namei.org, serge@hallyn.com, linux-integrity@vger.kernel.org, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org, Roberto Sassu , linux-mm@kvack.org, akpm@linux-foundation.org, vbabka@suse.cz, linux-fsdevel@vger.kernel.org, Liam Howlett Subject: Re: [PATCH 1/3] ima: Remove inode lock Message-ID: <5e288d07-5992-4ca4-8ec1-214d2fa6a326@lucifer.local> References: <7358f12d852964d9209492e337d33b8880234b74.camel@huaweicloud.com> <593282dbc9f48673c8f3b8e0f28e100f34141115.camel@huaweicloud.com> <15bb94a306d3432de55c0a12f29e7ed2b5fa3ba1.camel@huaweicloud.com> Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-ClientProxiedBy: LO3P123CA0014.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:ba::19) To SJ0PR10MB5613.namprd10.prod.outlook.com (2603:10b6:a03:3d0::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR10MB5613:EE_|SA2PR10MB4700:EE_ X-MS-Office365-Filtering-Correlation-Id: 367b9f4e-3166-4016-5ae1-08dcef83eb0f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|10070799003|366016|1800799024|376014|7416014; X-Microsoft-Antispam-Message-Info: =?utf-8?B?aHNCY1Z1RXJsOC9uVnpKb3pTLzVQU3R0WFpwbHdsYXpuaG5tYVJscEs0NDU3?= =?utf-8?B?Ykc3Q0pZaEFxNStJU1VkaDE2Zkh2Z0Zodkw2dW1IK0xKSW9yUlRIMk9aMW1q?= =?utf-8?B?c1Mwa0c2cWxhNnhKYUZGNGNNaUk2bVNwTmVpYjNDaXpXMzd6K0VYYjhBVlBi?= =?utf-8?B?S284d3hUTDhUdWhNMHpKSVZHakVDSTNGREEyeDNUYXh3NFdhblRackRYV0FH?= =?utf-8?B?SkRKMzltdkRXbm9ibXpVcFBHcWcwclNpNXdtell1VE44SUh5OGdFN0UyRkhw?= =?utf-8?B?OXJsVmhjc0t4eldobDVkTDNobVdabFhwZ0lmWTFIZGhlYnRnR21GcE5nQVZU?= =?utf-8?B?RERTVmJBY2NaNzJVaUFnNmdjZnRFRE4ra0xDczRSYnNXUlk4N291RG1LYWY3?= =?utf-8?B?TE1zV3dRWVJ0VWpYWXM5RmwvSlhUTDUxRDdqNk1WWXhOb1ZGMVRxc0RZSUVM?= =?utf-8?B?dXhKSlQ5aVRXNU5vQ1JpZFdQdTJVZHR6NmYyRkp0M2d0TWIwK0I3QWsvT2pL?= =?utf-8?B?QXBrQzBTSU0xa2wydEd1V1NEdmFUVkdJUy9NZTF6eUFtK0k0S1llV2hvVGYz?= =?utf-8?B?UFIyakxTc09wS2YyNDVTY3B6dktzdlYySGlOczdEeURCWll5NnFDYTBhcC9C?= =?utf-8?B?dXNyMmY1aDFHKy83MVpTUGtPeERVeEtMaEpDVFliYlV4N0JLNEQ5aEozWkRD?= =?utf-8?B?eENiSkF2SmFmNlZ3S1piNlFGKzQxbC9GbUFURmRaeXhtbWVmYU1wMjBxWVV1?= =?utf-8?B?Rmd1b3FxdzhPb1RXeERoNEUwY2xsT2ExYTBLenJEOHB6S2w3NGtXTklJWlJu?= =?utf-8?B?SXFpeFRwVjhjRkJPYUlTbC9pM0ttM29Gb3dqQVNEYzJoK3hNOUt5RHRZeWRV?= =?utf-8?B?Zjk5NGdkcVZhcEF2K0NIa2NLeDE1WU9OWDNxQ2FlU2V3QnMxUS9pRlVrUHBW?= =?utf-8?B?a0x1ZUVjY0ZKM0hEa0wvdC9qNkdVVmxBcmxweldTWHE0M1lHTHR3ZnlXbGs1?= =?utf-8?B?NjBDTzdReHpBSE11dm5TVmo0OTN6ZjRTWEs3VjUvRTJEMDc2Z0pHUXZ4YXAv?= =?utf-8?B?TEIwTU1rMGZiRUI0eHRCSzdweUI1eEdyQk51cDRMNDJFN2JYZFRBd005WlRr?= =?utf-8?B?TnBqWGg2cGRYZDVkY1U5bHBiemRoMXp4a2kxMDFqTm5VS1NFcWtBOC9IWUdC?= =?utf-8?B?R0RrZ0svNytTdzJrbjJLR0NKTnI1cFlYSzBOQWdsdzNIejg5azRkTWZ2UGEx?= =?utf-8?B?d1VKbjlLZ3dZSlZIYjVXSWx5cENoQzdoMlVscldwdm9SdG9ZZ3JIUElBeDNR?= =?utf-8?B?NERvRk15YjBEc2t4UFhJd1hvekpJbkdsSG1BY2hvZjI4Z1V6eHpuV2Y5bHpo?= =?utf-8?B?ZGZUaVMxa052aElleTBTTHlLZS80WUFHaHlSS0FjS1N5TnFjdDRJdlZOYU1X?= =?utf-8?B?OGVjQStXOWNXcWI2U0dnR2NzWU1KMnRoaTBpYnhSTEtxWkJjdjIvMWQrS2o2?= =?utf-8?B?UW9jekpzUjNBbmh4UnFFVy90YWtxMTlZOUEwWWk2L0JLdjZ3cHJjN3dxQUg1?= =?utf-8?B?Nm1WM2NCM0RNQ05ZQkx3blhIbGtHdm1ZR0hnaDZGNkYyWGpCQ2duUWx5UzNj?= =?utf-8?B?djhFKzFqcU5HR2YxY0x3bmQ4TkNkUUJnRkd5S2xCeFZuZWtReXBEUlJEdUJq?= =?utf-8?B?NGpNV2hqOStvRENDbzBLOHBYZHhyaExtaTF2Vm9iZjBJbnVHYnF2Q1ExOThs?= =?utf-8?Q?wPvO9WsOv55oBcV45mix+48rFGZs2G3x76ruI/9?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SJ0PR10MB5613.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(10070799003)(366016)(1800799024)(376014)(7416014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bUJmS2hxUjJhZlIvblR5K080cnozeFBaL3hxRStsUC9leUR1RmxHY0VmOThR?= =?utf-8?B?bGh6YlBaVUtPMTg3cm9wTG5tVWJsNzlMT3BqNTlkVm94dkZleElVS0I2YXp6?= =?utf-8?B?dE15Nmw2T2RUdnQzWTJQVExqVjNtcjlFeU01cDlZZW9iT2dTNFJFZ1JUalcr?= =?utf-8?B?dWNlaTRZVlFPVnhuSFlVemVIZksvMHV1TFBKaVBxZlNlc3k0L3RXQmQxSUlF?= =?utf-8?B?aThDME5Ja2RzdnhNbTJmSlZnOHJkRmNIcWQ3dHNMTnJpYStscXRJN3VVOGRW?= =?utf-8?B?Z3Bvbm42VVA2Y3lTR1JmR0d5THVuclNiRS96R0FlcjdEbkQ0dk9RSlFWajZ4?= =?utf-8?B?T1lTTzQvdG8yNXpycUtJVDVCU0FIOHVPRTRjVm9RUi9WaG1GeWg2aWo5c1R4?= =?utf-8?B?alIraVZDTEpIL25YY1JtTGhBaW9ZZ2drVVYzVUFleWFkVjlSYi93ZnR5TFg5?= =?utf-8?B?MkliZFBKdEdGTFp3YlpMWkdkeUdnWVdhZ05mSFJhR1B5QWwyMUIyLzJRbWRz?= =?utf-8?B?NFRYQ0NWN2EzUXJjZFMyK0NpeHh0Y2xOYkc5aFNTOGI0UnB2SWFKZDRxY1Bj?= =?utf-8?B?aC9Zdm9BQ1lpY3ptVmlZTWRENFJhWHlNUDd6UW1GZHdybmdveVI0WEJCK3pv?= =?utf-8?B?bjk2OVJhYTM1ZW91dHVnd2swTlFVN2Z4bXgxUWRJcDRNMFlwUW9TdW9ZbXUx?= =?utf-8?B?RUVuTElZSndDbHltQXJWWkN4WVBPS0JWSFNYdXRkOFhLM1RkUGhxK0pHUDUz?= =?utf-8?B?cUNvbThJZklzQStlcTFDN3diazVEVmN0UnpMUy9tQXlzazlBT2ZPb1FUb3VQ?= =?utf-8?B?YWNUMTBUSzQwTTJ0bGJnMTZlR1pvZERjNVRLcW5NTFVhZ2oyN2w0Q2VHNWtT?= =?utf-8?B?NVJwVDhSOXB3enBhVnpoakxPR3Uyc2Q1T3o3WTZOOElvTHBILzU3eGttcmsw?= =?utf-8?B?TmthQ3R5bUtPOHRmMFBYalBJaEdMRkJPeVlZcG9EaVFDNHM0ODBZOG5yZHk1?= =?utf-8?B?YW81OGVZSDdCVlBOdmxHOHdZTFVnNGo2VkR4OEJKVHRDQ0I5OHJVMjEyQlY1?= =?utf-8?B?SFFaWUVaQ1JHN2xaQVpEN0RpNjJrdDRKc3RsQjh3ekZQUFhGTHA2S2VWRlRP?= =?utf-8?B?RDlDZEFjeThlVUx2RHNyWjladW10b1AwT3hCaHVlU28veTQ1V3VTdDJrenJE?= =?utf-8?B?UnBDNi8ybDV6OUVvdkZQOWduMzcyTkJSZFJpWmJ0MTZoWk8rWUNTN3MyNkE1?= =?utf-8?B?QXppTkxEQVg2cUlsNHNGNDBSc056NGZtaGlrZzhib1g2TkVYU3hTZFBlOWpw?= =?utf-8?B?OFhBaWNvaktWRjlXR0l1cDBDL0cxQ1dlYXVPVk5hL1ZoU2R2WVJqTXVqSEEx?= =?utf-8?B?K0x4YklnNFlQNVRSN0tyeFdSTmg1UndIMjdITi9VaGVkUVRvMFQ0QXB4c004?= =?utf-8?B?WkxFdTlLVkduQXprdXVlSHBYL3hpVlJFS1JZSDlXZnk4Uzc1NEFUejdTQ1Mx?= =?utf-8?B?bzZFSmloRmhyYklwcnZCODFCbHBCelFXZkovOU5iaWVTejR3dm5tc2phRWhu?= =?utf-8?B?alZ0Y0FyNit6dVFCbHR1ZzBCZS9sbHJUOElpVDZ2UE44aGhIcUdiRVRlUE9R?= =?utf-8?B?ODJIR3RIdlh6ZkpEaGFJZHo5amowcngxVEs0OWxJU2NJUHU3dGE1K1dtalJY?= =?utf-8?B?WnJxY3R5NEIwdVlRL1l1V2hRd01vbndpS2NhU3VpYkkwbTc3U1JuaHJLRXdX?= =?utf-8?B?bzV6WjEwZmdIVWthNVJPbGJIa21CSnNERzQ3ditocDdLTUVCYytYVU1YT2Ra?= =?utf-8?B?R2VQbkE3MVVvMDVEMHgvUHpFbnQwaU5FMW4zMGFoS2Q5eXdsSXdvODBpQ0xn?= =?utf-8?B?WEJJVzJMN1hHc2FjN2lMSWlGUnU1Rm1yWXkyTzgweVgrb0Z4Q0luaUhEaEk0?= =?utf-8?B?d294R01iNDY3TUZUTWJOZmdBQW9RUlJCUlUzbTdyY0xnSE5GYVdPSlorS1hO?= =?utf-8?B?ekFPdUVFeWtpNHFpbk1oY09zZDh1RTVXcEgwVmdWWTVIaUFjR3ZjbWJuQVNp?= =?utf-8?B?VUo2aG4rMlpSeTV2VUNkQzhoV3ovQldmWFhzRko1L2lUQkhLQll6QnhMcHJk?= =?utf-8?B?a2tPbW5TOTAvSXFQQUxMZHF1ckRrSFdISmdZcm5zRUpkUjluY052UWZJR3cz?= =?utf-8?B?ZTRMMUhTU2pWc1UyQTM0ZHVJaGxPL2NqVDI5dGtoSjEybUxNZTRFQ2Z5V2hB?= =?utf-8?B?NTMreXBZbWVBMUd5U2ZNNzFtV3ZnPT0=?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: BZAMB1fyAvBFDwCQtpnF2vVKuVBSQmMX+z14bYNbPw2BKQXjJGgzyf9+H/ZtEvH6s7cpvh6oiCPHxm96c2IViBsUoGU+ySiwDOKD9sppfi2oXo04Vfz00Yputvz84g8GSpMyf1y14AdmnYIpJFzTukAflt7x/hLlilOt41hOWfrbiPxEvVATWJ/7i+vnnhy/8Eb//MigsaRz75rtm+ChUqm/+NWjIE5C8Lyr3VHR7rHoKh5ukNY7zJ/w63hKbEXawicSOEtzvg6meUF6vBVYpkD0Lm4vlsgUElZeTC/SxdosXWFDoPs6vTGToa4q2YDOittfYyEE1EFMljE+gLhhHLayD+Ak71kBG8PDdVYSCZuZj4qqe6C+WPzVJV/XVO+p5KqvGChR7uPLYXwxG5mY1MBESci+yzjobMfLBPZyo4ylqTPcrUaPs9VQztoiegT5lETTHItvy05xBSEJH567Uy9HpegxEyw1VH8YlkjVN3NHPDYJQjUj4SlhlgzLpZKm4NaBDT9uUOMwRgkw5SIJUj98iItB4h7jttSukLWGNBE/doJ2Z1u4sxr05mmbosLwBWApmbcen+GmsDneD0bm2RTcQds6457XLvqe6b7AqUw= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 367b9f4e-3166-4016-5ae1-08dcef83eb0f X-MS-Exchange-CrossTenant-AuthSource: SJ0PR10MB5613.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2024 14:48:24.6261 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 0zJ4KmQ5Muh8M9gCxS97BMhygzY64ob9/FMr3vRRscm1BurWcyxe/4eSHM5or1jdluw4UgjpntuENL0oZykVYX8VD8KK5TUsgr9KQS0tZFo= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR10MB4700 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1051,Hydra:6.0.680,FMLib:17.12.62.30 definitions=2024-10-18_10,2024-10-17_01,2024-09-30_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 mlxscore=0 bulkscore=0 spamscore=0 phishscore=0 malwarescore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2409260000 definitions=main-2410180094 X-Proofpoint-ORIG-GUID: CjYU5YT9lxAOn73EIJky7tePAz_cUTXX X-Proofpoint-GUID: CjYU5YT9lxAOn73EIJky7tePAz_cUTXX X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 9395114000B X-Stat-Signature: yed1ednnag1owxo6fndhn6m1a8y8sbds X-HE-Tag: 1729262919-230783 X-HE-Meta: U2FsdGVkX19BNEzjUeYgrwGXybKewOkkIzrC1dVfjGcZALACkgZsjPuKWmdxyt9Fo2Q1pb9YDFpBQDoEWt8xjkmm95BO44QipAhXWHkbPgOrI7/bhsRkCh2n0am0lbdshK22uE95q7qQV2RvVQ2JJVBKSG5JzFJlAVUv31au1dac2iQNaTTEuMJYhLuU4aJo7SsOlKvgDYjBmQ7yszWa7i+jsMy3Rif+AhkkDsuyiCbURCTSsPEistuuYdFVgn5wUiZqg9esqX+Hj6JR/bqe7DnpRe8vBgF3TJhMBe6HJPTsBqwolkEAFDAA0iiMHr/qbeNpI91/frilDrlYvsByh+WMgCLwsX9YEC0HO3ttLZOw09bBt2O+tCGe4gotuikFKuqTDZ6EGS4sHiX2pbdBPzRiCuIx8Hp712tuVzVgRLd9+7hHoJjRV5ao7rIZ8QueqRZ8NC4hfWcyF3lM28XQGbeaXpTgwXy4VaPlRcNSVSnUch1TA7mLwffiiVoOQeyxjc3qmGlS9WhoJH3poCuXQAp8N16dHNIZXRuRAWXejZ89CbcLwOkbD/Y7+Mv5l4yo8QTxsGuXuXpWzeAx82Fo6huIwl5NR3ZY2Uuveta7DqFezbgWaz6to/BJaVaujjH5TTo4K58/BoRR7UvjqRu/i5QNHeK84nIHmCS0amg0CkpYwB8hpOqNHSP6+M1/p9CUjuwisl7icz5aOguCCEtcCKg7yYUtwGr1iLPcEzJSCcvlLayHgcH2uRgNATcZgjWdL6D6OT2YrAUwbVIFOVImUw81as1jo5T0iRFP6/0oiDIhc4vW2SNCMFmm57b9Gz4AkRVfw/DGLvVRQd0C2LZNULT3kNeVzrlTbD2FkbDo8ikuU3xZI6hWRo4kigPzdrVMLk1jFG3+dVrdnm6sLaShWPm8Vo9yoFNW0YX3C4b1rlNAgsP2qnH11ocMeaUheXL5OarH5n88W9RdHmwxNQh e5WM+RrQ B16D4HqKWeqTWH90cPCSU79I6nIlPrOfjTZrGywvZUV1qnN8GUNIqpGFaEDeA6a6S1peXMVwPK/BXdb/vpIERRW6czWdpRR5NPQdQ7JpBwTfUGvYjPih2h9WoL3eXaaToR3GGm/E/pvgGlCXj6WyN+3i3lx4njVcDMXBV9wtOlMDdNJfLqtt7/5d5YtyIU6TaA+XAPTktvYXJl770ITagdB1AlI4K/zI8Ff5bc6DTkbnccMMD/G5O/EioLJMUF2//HOLWbI0p5xf2FENIbucr/JWt1Wi7FTVrwi565e3ktzHrLtD+re33Hf7HT+tvNZUkJulGhXVegW5epgMB6Wp9dvMmnSg4KbZyRoSWiEYB2TEi6K1c7e6jFceMHjttkk8tkO/GthXEzChqTpjny/zQPw0h66MrMU2DA9Ikyoq6GZfigl1M7ulHawmZtbwRssr//ajp5QDtc5wvVAadj6XO2DBPObe42MYfI4/a4+TKNPRcKuhXoBi5fsQFejjRqa8jkDUuJbWSkKk/MmdBPUuQrxssME0RXqyD1nFJkXs3BrPEmFeUuK5rk07X6IMCETOXcbIjF3fcxCX0hy45PjDsG4IkLJ7ZjW8kUfUC8Jw/plz0MWJfqGS7qGOhjw== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Fri, Oct 18, 2024 at 04:36:16PM +0200, Jann Horn wrote: > On Fri, Oct 18, 2024 at 1:00 PM Lorenzo Stoakes > wrote: > > On Fri, Oct 18, 2024 at 01:49:06PM +0300, Kirill A. Shutemov wrote: > > > On Fri, Oct 18, 2024 at 11:24:06AM +0200, Roberto Sassu wrote: > > > > Probably it is hard, @Kirill would there be any way to safely move > > > > security_mmap_file() out of the mmap_lock lock? > > > > > > What about something like this (untested): > > > > > > diff --git a/mm/mmap.c b/mm/mmap.c > > > index dd4b35a25aeb..03473e77d356 100644 > > > --- a/mm/mmap.c > > > +++ b/mm/mmap.c > > > @@ -1646,6 +1646,26 @@ SYSCALL_DEFINE5(remap_file_pages, unsigned long, start, unsigned long, size, > > > if (pgoff + (size >> PAGE_SHIFT) < pgoff) > > > return ret; > > > > > > + if (mmap_read_lock_killable(mm)) > > > + return -EINTR; > > > + > > > + vma = vma_lookup(mm, start); > > > + > > > + if (!vma || !(vma->vm_flags & VM_SHARED)) { > > > + mmap_read_unlock(mm); > > > + return -EINVAL; > > > + } > > > + > > > + file = get_file(vma->vm_file); > > > + > > > + mmap_read_unlock(mm); > > > + > > > + ret = security_mmap_file(vma->vm_file, prot, flags); > > > > Accessing VMA fields without any kind of lock is... very much not advised. > > > > I'm guessing you meant to say: > > > > ret = security_mmap_file(file, prot, flags); > > > > Here? :) > > > > I see the original code did this, but obviously was under an mmap lock. > > > > I guess given you check that the file is the same below this.... should be > > fine? Assuming nothing can come in and invalidate the security_mmap_file() > > check in the mean time somehow? > > > > Jann any thoughts? > > The overall approach seems reasonable to me - it aligns this path with > the other security_mmap_file() checks, which also don't happen under > the lock. Yeah equally I find this pattern fine as we check that the file is the same after we reacquire the lock (a common pattern in mm), so if there's no objections on security side I don't really see any issue myself - Roberto if you want to go ahead and post the patch (separately perhaps?) we can toss some tags your way :)