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 B6DA9C3271E for ; Mon, 8 Jul 2024 12:53:40 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 2E8F26B008C; Mon, 8 Jul 2024 08:53:40 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 299076B0098; Mon, 8 Jul 2024 08:53:40 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 0C4BD6B0099; Mon, 8 Jul 2024 08:53:40 -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 D90C76B008C for ; Mon, 8 Jul 2024 08:53:39 -0400 (EDT) Received: from smtpin22.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 777DAA1310 for ; Mon, 8 Jul 2024 12:53:39 +0000 (UTC) X-FDA: 82316576958.22.198969B Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) by imf24.hostedemail.com (Postfix) with ESMTP id 0F4FC18001D for ; Mon, 8 Jul 2024 12:53:35 +0000 (UTC) Authentication-Results: imf24.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=JjzGlcQl; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=W6xtI3gF; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf24.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=1720443186; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=WihLDam2zpygRFBV8/MyTA2mGPsy8creOeaWUz9QekA=; b=xacNvqo9pYZfKtRzMw2+FCWD6g5etGZKM9k+SZ9w7UNNYnrGNVMixRByKJ2hWNA0rSSxLk xGRYM/g5YAGCMa8mstQhekDOnURaea0gpAE7KM9E/jFnMJQg4AMWKfBpXFhYtM7PCXAXmg esI1Yw+STE1j+wo4FsPBLdreH+FOKNs= ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1720443186; a=rsa-sha256; cv=pass; b=u4g1fo+5NoNi1XifXn8LzwT9bXU6xbMh1+OO/updfHtYHMxk5bN32AH0Z4y7ebcLx3XQt5 UQ/VLTeLUkKQgfQuBjp+6qQ99K8+pO3kzW6fKYncVKQS0Tfv/maqffs1MGJF4fSkvb8Qft gfgVEbKVwUwYT/dPtjmXYNRWdwNkPCk= ARC-Authentication-Results: i=2; imf24.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=JjzGlcQl; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=W6xtI3gF; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf24.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 4687fccW004442; Mon, 8 Jul 2024 12:53:27 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h= date:from:to:cc:subject:message-id:references:content-type :in-reply-to:mime-version; s=corp-2023-11-20; bh=WihLDam2zpygRFB V8/MyTA2mGPsy8creOeaWUz9QekA=; b=JjzGlcQlM/zprcWOuwEUuiya4lXqV3+ 7203v3ihRevD3Ta3345xZhL14Poybq7Vxc/+bXkPFvUfzdLrT9OoxJoGrIP/ernd Jb1WqcmnXkPBjcLcqL2MlfD5hC9kiQG2Vp2XXLvHbM1xzQLkiHNTv37LXQ4f9c7C +v5yhLfgNgHDeykTPyy+wQ3Vz64389y1htDmbsS2pS+d9Iy93qrO7ZBS5vUs/Ra+ yefDpHPVo4j2I2/E51zJX2MLQy5yhMHnS4l5Jb1zmq6ygMKzYxRugICGm97GCnXl If7VcksJS2uck4vxYtJMWnuzFaSOvRcyIDHZvLXaHYAOEFtl0dV9OsQ== Received: from iadpaimrmta03.imrmtpd1.prodappiadaev1.oraclevcn.com (iadpaimrmta03.appoci.oracle.com [130.35.103.27]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 406xfsjhvm-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 08 Jul 2024 12:53:27 +0000 (GMT) Received: from pps.filterd (iadpaimrmta03.imrmtpd1.prodappiadaev1.oraclevcn.com [127.0.0.1]) by iadpaimrmta03.imrmtpd1.prodappiadaev1.oraclevcn.com (8.17.1.19/8.17.1.19) with ESMTP id 468BZFMg007191; Mon, 8 Jul 2024 12:53:25 GMT Received: from nam11-bn8-obe.outbound.protection.outlook.com (mail-bn8nam11lp2177.outbound.protection.outlook.com [104.47.58.177]) by iadpaimrmta03.imrmtpd1.prodappiadaev1.oraclevcn.com (PPS) with ESMTPS id 407tu1kt85-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 08 Jul 2024 12:53:25 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=CYySQWqkDB8pDKlLcpC7n/vgByvxgFxzQwYmw/yRf7a4cX/aT+repK/j/cePg89z65gAiq+01dDW7qqMmozn+qIhv+Y/OShSmRAXSpgDAuxNYKPKStRllTrfB1LJMmxwyfFWDjakYjiYPGmlru3/76Pu+XPItboAin/lC9OAmuE5se6mAzw02M7aBW+Vk3lxieAppgq4XI8yAdICMGSNLNv3b4PPQLeQvO2AqCYnLwIceNeJkzdy5TmuXCq0mbktK0MKJQocMc5sov+2I9BQfc2gZQwtjQNR1Te27YQ3MzIOs7Wx6EldxEpwyF8kF5OiUFN/AWNv7elsU1EzOX2SSg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=WihLDam2zpygRFBV8/MyTA2mGPsy8creOeaWUz9QekA=; b=jSJbG+ttwk7HTlWNq/rP5lbEEJQqcDfMIxJ12ZcVfzZWacgvPo86OoYg+mlUa1g5aPEEMiXmyWaHN01AVNeOQl1VI6uAjuLLQplmB4c8xtexVfUxMrunCpHvXiXCXWFORWbcRuFAvDYZ/JZfozQ+UW/nzK36d7Hr2XCVy89yVLc2KKoN8BU0lv4fPjm88afGSeznns1UFSc/h9uHJAyrxNYm+B7COcf8UULERjsYIm7tATcLe3kZD4ZxVSl6KzjmpzE1yITDsG9rV7opy7jVR9Lt6IdxtSydYancJXE3y9B3sjhE8AHjEHpII0VcH7VDhW5OzjK8wtact/qdG1U9mw== 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=WihLDam2zpygRFBV8/MyTA2mGPsy8creOeaWUz9QekA=; b=W6xtI3gFxqf48LuaQk8BVcR8pw9u4HSTxSLVNhgxbblT1AHML0+Yg55cgxfRE7Hn8Kx5n8XKhQgVmon4pjruG9oLmoe0ywHHxvCECy1C5aZgAdXW4woZ3D7hmWBY5ZXspry/ZOq92d5Hf9fJaZBdzQ0rASrFkZUtXdydlUVarbQ= Received: from SJ0PR10MB5613.namprd10.prod.outlook.com (2603:10b6:a03:3d0::5) by PH0PR10MB7025.namprd10.prod.outlook.com (2603:10b6:510:283::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7741.35; Mon, 8 Jul 2024 12:53:04 +0000 Received: from SJ0PR10MB5613.namprd10.prod.outlook.com ([fe80::4239:cf6f:9caa:940e]) by SJ0PR10MB5613.namprd10.prod.outlook.com ([fe80::4239:cf6f:9caa:940e%6]) with mapi id 15.20.7741.033; Mon, 8 Jul 2024 12:53:03 +0000 Date: Mon, 8 Jul 2024 13:52:59 +0100 From: Lorenzo Stoakes To: "Liam R. Howlett" Cc: linux-mm@kvack.org, Andrew Morton , Suren Baghdasaryan , Vlastimil Babka , Lorenzo Stoakes , Matthew Wilcox , sidhartha.kumar@oracle.com, "Paul E . McKenney" , Bert Karwatzki , Jiri Olsa , linux-kernel@vger.kernel.org, Kees Cook Subject: Re: [PATCH v3 16/16] mm/mmap: Move may_expand_vm() check in mmap_region() Message-ID: <8fbb424d-a781-4e61-af7a-904e281eba8c@lucifer.local> References: <20240704182718.2653918-1-Liam.Howlett@oracle.com> <20240704182718.2653918-17-Liam.Howlett@oracle.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240704182718.2653918-17-Liam.Howlett@oracle.com> X-ClientProxiedBy: LO4P123CA0177.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:18a::20) To SJ0PR10MB5613.namprd10.prod.outlook.com (2603:10b6:a03:3d0::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR10MB5613:EE_|PH0PR10MB7025:EE_ X-MS-Office365-Filtering-Correlation-Id: c828d01f-a84a-47ed-2e56-08dc9f4ce7f8 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|7416014|1800799024|366016|376014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?J1lnrW/xG/fTlLhSbVVXbWxpbknuXw1IbmVfwNjHgyEDRlE83FnAC4ddl3Jt?= =?us-ascii?Q?2iVclLmHNPyBpC0zJ+JEXUcqHmrHU4g/sqtblJ0brbc4WasJvRp0MdPlg68Q?= =?us-ascii?Q?1EEKg6SMNfqr7/hEvHZ5B3muPyZ7lWspHo0R1SA1FkP9WO4C5mOfzHnhzrhR?= =?us-ascii?Q?gHhwqqJ837JgxGQAfv2hLCiESSXEEFxpwvqWNOc5xJd4ny+BMHsWIbjaht6I?= =?us-ascii?Q?gih5Acvj4jd6hX77Ko9FD6elM7UW1IXh1Dkr4EziskW5Gc/jJAaLgBDmYbw2?= =?us-ascii?Q?I6UwZNbln6Wds2UzMTDDjnFkNbHGmoFwkcmUHXJCQgbQ/rrmfZ59iTBJi6i7?= =?us-ascii?Q?o+Ku2qqPOPpY2BgR8Aohj4CTeUbMV/EqNLGXBtF1d47nljD2XVRq1KhVcW+Y?= =?us-ascii?Q?S6s2GNxKtyJ/Zs+USXvxkhJrD/ku2kpRR+t+xVRjB3nSaS+vNrDVJ9tKORXf?= =?us-ascii?Q?xOeCqOB46/71IjokxDmyxGbp795Pqga+GeIjigyCIUtdolLVgRcRJWxbTP1i?= =?us-ascii?Q?GGSn/CU2Ls6CycXbpPjvWC9A2WBcyQML2eSYGcx45croLJvOfRnUJY+F0m4P?= =?us-ascii?Q?sfTKf1aLnnYaXxPUFc8ZoNIGFqAjen96vNkslZvgapiZC9KFAFPIucfGzLO0?= =?us-ascii?Q?OVT57dJnWP874a8OCciGJQ2m/YVirQ9Y6Rx98idLdXUprQXmnr/wYDBaPDfS?= =?us-ascii?Q?/wOebJ6pvu8vV6uax6vlaIM9JpGz7TxdqQ2B4Lpp/J53BV/9DwIwcAWp73mG?= =?us-ascii?Q?Hxniz+ws19v16aM9NZZBa22IlWCbTpMuprTR3P7MLu5L93XXK3hs/NAQwp/U?= =?us-ascii?Q?Ea1lhyitXMHdnDnXR8x6TOxv4N09h2zHcBgGQ3aSDtBVYl2E/38pSr4GSTYB?= =?us-ascii?Q?S4YEaHPr6kzL0jzxZMvwhU9JxSl/2XpOKJOPh1aJ2YDYOS5PPQUGYkqMGGYh?= =?us-ascii?Q?r1IZrvS5pgXOGT+81ydLjVr7CgeSnCsh6+9JAG3vrRUh7tU453pT6y7cw5OH?= =?us-ascii?Q?kk0obaDmMpWiRtFt3XxSI2XYCb2bYbIoCe67nBwY6n5DeSp27DDkCBeEP0yK?= =?us-ascii?Q?KushTVgmqt3fsy3Hsw3MwnuLVf9r1+gfHifZz26hUofgAQsPwY9OqdW0A9yU?= =?us-ascii?Q?uOV4fUlBSDUWWtlyQtHg2T97H75MKPlqd1lK+BtFu/a/ufCCSd1rWydbOYw1?= =?us-ascii?Q?WM+DBEGj822lfop55oEY7+iq/Sz1yXBzky9vJhCRrYtIUn980ioOntSOUllq?= =?us-ascii?Q?GcjljKoT9U34pV2tBHLOulVcY/FwHCHyBuaAbrCFQiHOABKcFvifC15IbWEl?= =?us-ascii?Q?yDgW33yIN0U3gufJ4638U3KHyngbKgasVYjFz8xurAP6qA=3D=3D?= 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)(7416014)(1800799024)(366016)(376014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?/ltuz6b5SsJXjXRbtddFOstQVwAe8xHAoKRiii9aXYvtAon6YRVmmJVlGpsE?= =?us-ascii?Q?zwlIDvGvBP+PxerluW/qR6KQkLZIO8NhTMo9SiryTcBKvODnM8Co3sc1lT6E?= =?us-ascii?Q?SRL/u52q97z+oCxVyoELU4H4rgKnI1dtP4SkEx095bFPLNasS9x2QlpUl4s/?= =?us-ascii?Q?PX8IRgQkdrnrfw8TKYMxlvSICumIPgwtlGNth87o/ODm5wVXd//kTBfD9QdJ?= =?us-ascii?Q?reI9lMr/WhdL88XMFZfTDzENJZiPtemswQbx4Goi8DgHNa8yGlXbGv0lD0N0?= =?us-ascii?Q?J+0IRLPk4EenkW1/LU/hddUZTh94Bra4xYcw16l8QkQJ75vD//bA0ZVQ+aEs?= =?us-ascii?Q?xqCWamq2Sf9K6kXKC1FUUJEjAJVttVsBk7KTpZiFBwTKPZx+ELhlQB/234rB?= =?us-ascii?Q?0CPfBcP6P+u/K1tWXHEV2CrA5cvV/xvK6ziPRnQFxsg+3bC1LHf2TQvpFMiv?= =?us-ascii?Q?xk85c7x/84TVKU1LUgz3CFpe4ED/yQnEXnWEPO/7iLS1aSt00MZzIg6S5tRp?= =?us-ascii?Q?ItZ0Y98T1ClyXIbxXrHCTITFZGwaQ3OG65cnE0Fe1RHuYNW2x5ccfnQGTgLk?= =?us-ascii?Q?dIWfwmLyWjGblcR4GMnt3AH2qV5XqPiom0fkEGeARzA8oQBXTJEV3AZ9xViv?= =?us-ascii?Q?WmSb+5dHL6CY5Ym/GREImo4PvWN8Gzf4xlFVEFF1fbIYQEAYhhP4MD12LQtA?= =?us-ascii?Q?RyAQcS4ipwK5TPuGjwpPCW+IEHxQKu9tl7RcO2ygqcVOsAKHZU0mYjBHs+Kd?= =?us-ascii?Q?tDFuPxB9abd7HmzzHaQl2H+j/35aGgj3u+bfcmyNRWH8Oh51xjUoT+M2VJiV?= =?us-ascii?Q?5w2EKpi14JOuN4kzJwioEw7W5yZ3PKBVE4hLuwekwrCvaJFbYsJbxsZfarIa?= =?us-ascii?Q?yRHQOoRL0sKm7wEu3yXLQXJGSlWKqFY0WNUu07NewTVQtrY2js1ksH5oKyxv?= =?us-ascii?Q?dmAbXY5vUcH0TmEOOtOcfigAiotpSgTtdfMd1DYyaodstd2MheIIzI/SARgl?= =?us-ascii?Q?Y1Ks+va8N9IKOOh1uY2xuXfGg5lgJdqVh758g119DboYni+5taBBh6C13ZGH?= =?us-ascii?Q?rsgkp/ukwk343W7DL/WJcDvVE6Iq8CmFnwCzfPktQw3rTMyDFUGzMjRdReem?= =?us-ascii?Q?u47bx4Abb95KPvk7mRB0TB/eHk1YZAcG2zZ6J4nccBKFXigpD3zXMX/in5ep?= =?us-ascii?Q?gD/2gOxVQcuxR/mWTfu1UzcryGeLzrIHt4oX1s5ukasSso7GJ+QHAnaS2s9s?= =?us-ascii?Q?BF7+anO7XIGM9bszJRvkrnSOTUOE2b/ZxcwddjWGxt/ZT7O00Zdml9XwU4x1?= =?us-ascii?Q?vqoV5AsslGP2MpW/kqYkCIT7j1wrNudJJnqacW8Iqh/1BHbGhl9regZTDY/5?= =?us-ascii?Q?t3DH+du8Okp5lF/Z7FoieaUIKxDZ/rjXS9ojuoonYJ+Mf8yBZlbjdA/OFaG7?= =?us-ascii?Q?gO1ApvziJiU+oHpJBpy164JQEI0SCAY8seZQ3DDEcqGL4Con1F41Bvz/HnAg?= =?us-ascii?Q?Q8f6mbRFKyqmyOQ7OIe9CdZXnxeQmSfv19M5gsLrjG/meo4jVCLWaEPesAmB?= =?us-ascii?Q?TCwxpA1zCPo201S3iIuj39Dqd2Hd/DftFRVT1LsJ3niET7wjV/u2BbQvgIMi?= =?us-ascii?Q?hw=3D=3D?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: QFdQ78h6QvipvyfzbImbP8GU/00sxxPDhyUEiGe3ECk8xMkSMIs5b99iSJxXrS49lQA5EuXjJXtv568c/p0G51RwyK2g3iNK1fVFXcZa65FLwGAPRGgkXthhcHPdjOlw49srLnTttwQjKGQaegCMECKoFBvFcov7i4It4FRCaPwBmrZe6ULc4kGbbpSXqTmB8/nYZDyZSNVPSsrBDpFBcpppJBRMn1aWY3j3Jlx3o8l1j3JuH3DS2xeFm565IahMwGzXMAwRvErlubpF15EyPV3lGlxBqfvqEFUZC81680E4QogGWrdOtEcc7x+H/7JtaDIGTfBbQ54/+dEb5uRBAma4Hx7swax3UJVSOShvAXuErWcROaOpRslqIyeW+qyATtZNeyS5j26YqwFWvcXOjLjxwE1k0ohiCDAY7/HgHuTrCjuiuOJc4AHveNC05CaVWpcL20cFgwsa0xayRT8xmlTEP+GwRVQ7YLI7sEUCrF0mefbWFMMh/W/dmuk4FabZV54tfio7S7TerOd6gI01uGjqaOQ+mTuZgDqhRlsyxGZW6fdfr0miaPAEKIVLiFFQxc+rbiLuo4BPLrASXu8gj9xu9EAutwhoBXfJW0tx1XU= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: c828d01f-a84a-47ed-2e56-08dc9f4ce7f8 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR10MB5613.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jul 2024 12:53:03.8871 (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: d3Ly7b3MoNUpzWFV7FQ9juxUl13PNuXqubKdmxVPKTyEr9TxRYXgOEY3LwCx7OYwFc4KjZ72qZeGJCcxWs+kG6vdj2A8RITADWDMdE3+sbs= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR10MB7025 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.680,FMLib:17.12.28.16 definitions=2024-07-08_08,2024-07-05_01,2024-05-17_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 mlxlogscore=999 mlxscore=0 phishscore=0 spamscore=0 adultscore=0 malwarescore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2406180000 definitions=main-2407080097 X-Proofpoint-ORIG-GUID: Agjzl9OC6fMPSsQvpa-FH_Wes0gSKzcW X-Proofpoint-GUID: Agjzl9OC6fMPSsQvpa-FH_Wes0gSKzcW X-Stat-Signature: qfmbkus4oced4gnsa3sp75xia876dpwe X-Rspamd-Queue-Id: 0F4FC18001D X-Rspam-User: X-Rspamd-Server: rspam08 X-HE-Tag: 1720443215-473296 X-HE-Meta: U2FsdGVkX18HGmw7DDJ8/Wvl/Y0qI29LV9PARuiALYDxELEvdgRCP3I2dLhn33O2D28QCNZmvu0b5abBm+gpHoNbPG+KpD9d8xPF6x4FzuLtxuPuIs45bKtLwepoaKvQI+WDRuGbMcvo9n/+yRa2mEoYVHJ3nJrZ8gsnAJJY2dbIFjNbmXOax981/3E7JF/H2Am5YxL+k6X95oJyAjTqWEbbGBWsKYaM5qJWD6lK8jwtSBN38h2wdaZqhuGiAiOrboyFBZUPqw1uPd2SOsQgJ2sYSF1TdNbgsXDSH4v+AgN/mLPc44V6OWkNQ3uVcHQFP1nhd9Aq61J8f7z0MIYl/nbswtzCzHQmQTiZkoTm1xRbXnRBSA9UkmtZzul3JqZT5f1f8aoccJy0bnkf4wHZiS1Zmmr/jb+m+WJ9Pz9cavHsdrg4JVNLJEIjJs/So3KOa34YHuZ8F1qv+IeTs4n8vqa3MiBQG/yeLxclakKdCc75JsAWkh2RV5aPtoE4GMJ5Y3ymr044UIF4Pe0OxANFih5tDEK6t9EHCI1S2b9gfhWys6vXyhakA1iRApir0xXfPwaRTbyY8j4KEjwQopnUhMdxsosdLQNeuuYnD4qmm5BVD3ICECe1Lw9JEyiqjxavVaHB2Op4ScYjXaRv5UzrM12+OGm/L/fn3fBqumWZh6XmcwVzeDV+2C/6FhD54Fx2sJvG/RZCJQokKPuW6mafU7YFfvfAkCZgng+FN9I2mmcu4IL5u409EE8TpAJ/REo7jf0BoyiMYw6TBw/S4LWkKpuEwGmo1+3EJTE/+t0svxQR37UHsXGo5Lr+nMw5g5tJrubB6JO9ayESvmL278sg015Da9hIuuXFtVMdV1kpBhBfHQ64E4fVr3w9oN+lotNc9ZmiljpSiWfMzSU9LbpKoReCnn/wbfO+PlDBNEX3CREFJaKbSAIzD4bBxZ4VCa6Y9rzkrERDtZvBNN6EGma cilkMKtp wlKGsjNYUMGYcU7WnI9TpXjCXqIafhGS1tL1oFy11qXAp2EU0OY9fuY4nVD9jW1Mu9RG52vmDyh6uz5P/7FTmohRXQH5M8YkOZFe0yVTVpyfHaShO/gFKGVwYj4zinQrLBCOCxj1TDX90vECP7z12O+BFlwHqnAjNUNLr78e17wl8xmenW6uaZS5Rc3U6KRuJ7wLdsTHHk3FCbmljBsT3zhFjmXUEu72A320BHW04hlGipoZEFV6XUNgUVHqdb3vfdeI1eV8e47zSoLltA7QHlRYv5OxJ0KUOYXl5LCgA7pEDr2XWU95Vpx4g2Q4UrMekAXzDlQ7lcePvYMCYk7xvOXVjEy8vMcgbJwtdzHmdEf66MK95G/8zo1bw04TU2q2/Q7GslSw9/SB5xaqJYcI3MoqZl5vnzEv2KyUylmaGU3Sd72C4ogWTrV6zRfwbGSxYkyaOTvclRTRAFC9ZuB8U3S9xVYEvpvR9eJUU6sA146QUQawYkIiPycCEr/EvIFiH7bm+fIqyqEfcCoRAR2d6++yXeQilg4M7cUrVTppsP/UGQCGBFRPO2bFRWw== 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 Thu, Jul 04, 2024 at 02:27:18PM GMT, Liam R. Howlett wrote: > From: "Liam R. Howlett" > > The MAP_FIXED page count is available after the vms_gather_munmap_vmas() > call, so use it instead of looping over the vmas twice. Predictably indeed you removed the thing I commented on in the last patch ;) but at least this time I predicted it! ;) > > Signed-off-by: Liam R. Howlett > --- > mm/mmap.c | 36 ++++-------------------------------- > 1 file changed, 4 insertions(+), 32 deletions(-) > > diff --git a/mm/mmap.c b/mm/mmap.c > index b2de26683903..62edaabf3987 100644 > --- a/mm/mmap.c > +++ b/mm/mmap.c > @@ -400,27 +400,6 @@ anon_vma_interval_tree_post_update_vma(struct vm_area_struct *vma) > anon_vma_interval_tree_insert(avc, &avc->anon_vma->rb_root); > } > > -static unsigned long count_vma_pages_range(struct mm_struct *mm, > - unsigned long addr, unsigned long end, > - unsigned long *nr_accounted) > -{ > - VMA_ITERATOR(vmi, mm, addr); > - struct vm_area_struct *vma; > - unsigned long nr_pages = 0; > - > - *nr_accounted = 0; > - for_each_vma_range(vmi, vma, end) { > - unsigned long vm_start = max(addr, vma->vm_start); > - unsigned long vm_end = min(end, vma->vm_end); > - > - nr_pages += PHYS_PFN(vm_end - vm_start); > - if (vma->vm_flags & VM_ACCOUNT) > - *nr_accounted += PHYS_PFN(vm_end - vm_start); > - } > - > - return nr_pages; > -} > - > static void __vma_link_file(struct vm_area_struct *vma, > struct address_space *mapping) > { > @@ -2946,17 +2925,6 @@ unsigned long mmap_region(struct file *file, unsigned long addr, > pgoff_t vm_pgoff; > int error = -ENOMEM; > VMA_ITERATOR(vmi, mm, addr); > - unsigned long nr_pages, nr_accounted; > - > - nr_pages = count_vma_pages_range(mm, addr, end, &nr_accounted); > - > - /* Check against address space limit. */ > - /* > - * MAP_FIXED may remove pages of mappings that intersects with requested > - * mapping. Account for the pages it would unmap. > - */ > - if (!may_expand_vm(mm, vm_flags, pglen - nr_pages)) > - return -ENOMEM; > > if (unlikely(!can_modify_mm(mm, addr, end))) > return -EPERM; > @@ -2987,6 +2955,10 @@ unsigned long mmap_region(struct file *file, unsigned long addr, > vma_iter_next_range(&vmi); > } > > + /* Check against address space limit. */ > + if (!may_expand_vm(mm, vm_flags, pglen - vms.nr_pages)) > + goto abort_munmap; > + I know you can literally only do this after the vms_gather_munmap_vmas(), but this does change where we check this, so for instance we do arch_unmap() without having checked may_expand_vm(). However I assume this is fine? > /* > * Private writable mapping: check memory availability > */ > -- > 2.43.0 > Looks good to me generally, Reviewed-by: Lorenzo Stoakes