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 678A8CFA452 for ; Wed, 23 Oct 2024 16:58:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0068A6B00A3; Wed, 23 Oct 2024 12:58:41 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id EF8656B00A4; Wed, 23 Oct 2024 12:58:40 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D99AC6B00A6; Wed, 23 Oct 2024 12:58: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 BD0FD6B00A3 for ; Wed, 23 Oct 2024 12:58:40 -0400 (EDT) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id D577C40DAE for ; Wed, 23 Oct 2024 16:58:30 +0000 (UTC) X-FDA: 82705475244.19.C6D7AE7 Received: from mx0b-00069f02.pphosted.com (mx0b-00069f02.pphosted.com [205.220.177.32]) by imf10.hostedemail.com (Postfix) with ESMTP id 92D3AC0018 for ; Wed, 23 Oct 2024 16:58:29 +0000 (UTC) Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=RQceBIcQ; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=kyVj7yS7; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf10.hostedemail.com: domain of lorenzo.stoakes@oracle.com designates 205.220.177.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=1729702566; 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=Y+o8PVjl7S+01VjmEjLnTEaCUw4NOCksjV7SynMkTVw=; b=1MbBY3y6S+ST4uByQUxAi2H/l+BABQt1PlsV7ffjyuq3huD+GTXYyaSfjjxl+rGXwRGjc1 J+LQFu1LmM/Fubbx1q8aTvlkVhF9PduoElqwfSiB6FYBMIuErY89JXRDLMHpn1b6QwgD2l QCduNVTubqZcWl0on60nEHLJ5iJ4d3A= ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1729702566; a=rsa-sha256; cv=pass; b=ffHgLTfpheUKbdUmXU7YLol7/L26qMw3tFNL2UANOQ0kqZrvKnVI81wvWPLVyWhqZmRNTL r9At2H2lqPXr9TuNW/hqvo59WAU1LKt/IGmpDbqRwh6jxUqdW/KAjdphF/tAvfZkSvge+k qg+4yM3jnV5KFk5aYce/fw/w/nb/u44= ARC-Authentication-Results: i=2; imf10.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=RQceBIcQ; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=kyVj7yS7; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf10.hostedemail.com: domain of lorenzo.stoakes@oracle.com designates 205.220.177.32 as permitted sender) smtp.mailfrom=lorenzo.stoakes@oracle.com; dmarc=pass (policy=reject) header.from=oracle.com Received: from pps.filterd (m0246632.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 49NFfeMx031470; Wed, 23 Oct 2024 16:58:35 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=cc :content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=corp-2023-11-20; bh=Y+o8PVjl7S+01VjmEj LnTEaCUw4NOCksjV7SynMkTVw=; b=RQceBIcQirck/a0qvbwG3Dnth3470Lg1bP hYgX3JnwmoTQIQ+wh/+7SSM+MG7tHg+LcWKjmoTj2LgZCDGOk/IcGvQ+fE1saLdH UqT1Ipc53c4ASE/wiVhPLFrwIz194Db0ppoHBOG0QgUsBCu0Kl82ViDeEzCTUGm4 8v8ZG7RlyZc0+NNomswJ68b0zetu5NymsFr8ZANgQ+0e1UmoSGJ7tXtmGKXmCsG2 66bHpcJ+UJerTv2WfImxUcC5QM08m8ocAtnuUHOvqXDGD+NCbRtahhI7Ai6CBl0c if9H5k552RUxNu/mc+5aA+76p/PJyvG/RgFq/bxSTbrg0McRVwfQ== Received: from phxpaimrmta03.imrmtpd1.prodappphxaev1.oraclevcn.com (phxpaimrmta03.appoci.oracle.com [138.1.37.129]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 42ckkqyy38-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 23 Oct 2024 16:58:34 +0000 (GMT) Received: from pps.filterd (phxpaimrmta03.imrmtpd1.prodappphxaev1.oraclevcn.com [127.0.0.1]) by phxpaimrmta03.imrmtpd1.prodappphxaev1.oraclevcn.com (8.18.1.2/8.18.1.2) with ESMTP id 49NGPBuW036077; Wed, 23 Oct 2024 16:58:34 GMT Received: from nam11-co1-obe.outbound.protection.outlook.com (mail-co1nam11lp2170.outbound.protection.outlook.com [104.47.56.170]) by phxpaimrmta03.imrmtpd1.prodappphxaev1.oraclevcn.com (PPS) with ESMTPS id 42emh2sknm-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 23 Oct 2024 16:58:34 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=b1RZJkUVUtstE3kH6J5nAK2hiqCu/jxMo0rF3HAwGjco5RlBiY4PifwdqJ6y9TOhwkhtK/obpc6AVDWPohuqEJCo6b2HjW9a79kc+0+KPgc05DjyJskDS4ss15QhpcK0sQsQZYP2YiR93kGnF4DltutBtHkV3P1OeS92j8uQiiRgJPQ5OiUPCIuJ51ax1XTspjLqLv0hkfRX24Ih2+087niNSh1jZgOybnXdYlGjlGRMLlTMgL+zkIHH9HXpxkhmUYzWk4+j8wU89pT1juTM1YHUal22WV60BADQqS3mPVN7t0HG5Ze08v/eUTf7HtvRJ0ShV3Nm4U0FEKYDogr6dw== 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=Y+o8PVjl7S+01VjmEjLnTEaCUw4NOCksjV7SynMkTVw=; b=dWznvHg/2Hmxu0HGiku3cgxmvLZSdVmr/eVp0D37uZEQl9/oASdxfbezz1arAyEkSJEvs3Qu11zY+Ekku54LWaDJZ1+2p9aMVy0ARRyHwVxUqpXSzJSsLCeTwO27OffsjKbJQl7hAMnybGOpmO2iGKuMii3lKXtRRCQPYr9N8m7pVHFzUtTYc2oF62gYPG+uCvid4306kyGgTYAN/qlGzfTBucVKiAAxUgUHK6X41NKPUQZ7eKc+7oG2VumjcLrp7y+agvgtloxD4gu3nbNHGoMZqWTBRNQnk6bKXc2DwXJHJkSyKaOjwrELqeug2IA30sgQUH/DPYDj+F2gY59DmQ== 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=Y+o8PVjl7S+01VjmEjLnTEaCUw4NOCksjV7SynMkTVw=; b=kyVj7yS7WwRvCQr54UPPcQK+eqnPPoOF65nB6uSRRNR+2XUtsKmneYFH+zJDc3u6X/vtK9LhFSR78Au1/f1jd224r0U1PV6vN25r+RUEZsw0BzEjkuz2Jf4vGmI7CSzjIDEi8E/+vCqh5JpiVy5PmeFbzsSMRCuq0N9BmXi8LVI= Received: from BYAPR10MB3366.namprd10.prod.outlook.com (2603:10b6:a03:14f::25) by DS0PR10MB6727.namprd10.prod.outlook.com (2603:10b6:8:13a::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8069.28; Wed, 23 Oct 2024 16:58:09 +0000 Received: from BYAPR10MB3366.namprd10.prod.outlook.com ([fe80::baf2:dff1:d471:1c9]) by BYAPR10MB3366.namprd10.prod.outlook.com ([fe80::baf2:dff1:d471:1c9%6]) with mapi id 15.20.8069.024; Wed, 23 Oct 2024 16:58:09 +0000 Date: Wed, 23 Oct 2024 17:58:05 +0100 From: Lorenzo Stoakes To: Vlastimil Babka Cc: Andrew Morton , "Liam R . Howlett" , Jann Horn , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Linus Torvalds , Peter Xu Subject: Re: [PATCH hotfix 6.12 2/8] mm: unconditionally close VMAs on error Message-ID: References: <9a84bad9fdebbdb0adca2b5b43ed63afceb5bacc.1729628198.git.lorenzo.stoakes@oracle.com> <27a022e9-dd74-4269-b98e-c4d78bb5339e@suse.cz> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <27a022e9-dd74-4269-b98e-c4d78bb5339e@suse.cz> X-ClientProxiedBy: LO2P265CA0414.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:a0::18) To BYAPR10MB3366.namprd10.prod.outlook.com (2603:10b6:a03:14f::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BYAPR10MB3366:EE_|DS0PR10MB6727:EE_ X-MS-Office365-Filtering-Correlation-Id: 2ece34a7-7536-4702-8bd4-08dcf383df80 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|366016|1800799024|10070799003; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?f1RmQwhyufseKD1JZvbOjBFVkOsfCXWrq/kAT71DSMkpPJd6FGFCOg6F0rHx?= =?us-ascii?Q?mR2MpDKLeug3hKsrLhEywvcQ2z5ygNIESY08FirWbeZXpnpzxyh1DqMXa1qq?= =?us-ascii?Q?Ac9L/ACZ1rLlMxJn1384x4a1zL/dgp651xcOPuKjmbbqTCfH9RZ6ZM5GysQ0?= =?us-ascii?Q?chCRAD6eUvWsxQ9v+rL092o4o0bSqYRvls/4Ljf9b2LxjZSfLOBalj9BkuYw?= =?us-ascii?Q?bOaDxPB6KgUhoXx32ZYINFdpsavTEnwUdvuS6OXc4eap/lYsFaMbFfMoiTFf?= =?us-ascii?Q?ixUoMRhy3j6f7wTnGZgkND4lu/yDGFWXtL1MqYKGtM1J9hY94w+DvInFqoFu?= =?us-ascii?Q?X3NqB+gWptbQ3+/K1DSySLQydxfBLg/nba/lCF8ajmYTB0Lvx2tQ3NTUj9xC?= =?us-ascii?Q?AfNjtHozKOfbrpuixL75bVGsDjTbXOYP6ga0W1j6v7lmgf/XYzxcb14U8/98?= =?us-ascii?Q?vs4CuwAyKcqS8YUIVJc7CNwQYai64tQhKFh1vlAhv9s/iQXY5IhtsqoEPtUj?= =?us-ascii?Q?lHPojAzmY83jpQn1eAuenD3i7AhR1cVuLkCz1nae1p77v+kByRp2PpzArLyj?= =?us-ascii?Q?sxz4X9+Okh1gPsyDkxYfacpzrv7GuoEKQ0aVspZ26zyE1SPyZUAC+q/QkJLJ?= =?us-ascii?Q?c0RJIJRB1KWJ0WAG7LunY+L/MiDPYwjI545NHZeD3PUxLWZ1rX9ffWOwHFAA?= =?us-ascii?Q?DtaUF3Mcpn/tOyQ3GxK2Ul5hDcrbYErVoPtvjXouBWx9XMrMHR6g6VE7hmYB?= =?us-ascii?Q?yOyOUWVGdEqAOM9rkGukDA6OVg7x86Bz2y/D3e0krdZejt/ySqI/SOSAVTnh?= =?us-ascii?Q?QMVElTJxJuc5k8+FwXS2o7fmOReOoPDYtLENN1hWN+DJzlQQ37YaGGeLWc6L?= =?us-ascii?Q?Mwx6XbniEL6g+De0sGbnlzjNwd5Lh2XDFT2zVs9PRlhHEST3mbXepsQKPOxL?= =?us-ascii?Q?ZE5VKR7nJ/SEdqQZErJBIo5/nRLYGiup9JMe1alcNwPB9UoQMRRmi40ETBrk?= =?us-ascii?Q?fGhTouXgF696oUtpUPqO7OQuHlr/NuawOXXAciKLFlm2kZ7NMf8eauLpkrhG?= =?us-ascii?Q?W40vQ5URJGdWG3VLAmSn8i3NeFesw2UDkX0YviffSkF7bmW/HOwMKnZ87/od?= =?us-ascii?Q?favy8+xi4XOHddp6TebEAqy+jtKdDktZo+dsEVZ8o0lFcXDFgLaXpy1uvDj/?= =?us-ascii?Q?zd6YmrPXu6vVDGja2tMSEg8wjI+cWl2L+L4kX/0tbt70OxvzXwuJ1E8JIyDi?= =?us-ascii?Q?nNBpPmiNgYBusYYmefHUxdg4nwk5VyE839InypinVAhL5MyiJvPf0w/jIl1V?= =?us-ascii?Q?v+cBKJTN/mlPXdt+Kx2ngZDX?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BYAPR10MB3366.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(366016)(1800799024)(10070799003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?bjd8eQ3C7JnnF2Wx0NYET1+k3O/JJeNYP4Sn7hfidficmlsIXE+7lxV4k2NQ?= =?us-ascii?Q?06PrvDPNpKU3DcELKWFKciuqhew9UkHaKfR2FYtyhMdoNju9S4DhtPDVc+xR?= =?us-ascii?Q?s/G9Pg46HqmgXNW8RvGMuKMTXQJCBhlX/KkbQMLgTk7g/QSZ/Wu2dZxpfYig?= =?us-ascii?Q?giayu+0S4WnFl1a4N79+mC2xYmh69hQYKh9wRRhgRESpV/1iO3aszfYROqaA?= =?us-ascii?Q?5Aml/L3QpWvl3+lgoguAA42DQavQoxiG9jBbOXS+bcRa+qhldjz3FK0+3LDi?= =?us-ascii?Q?M/6f3vsxt3Uwppe7Qj9oFAO5WbJ8lFAigqGHaNSeTC4vMuNr6ZdcbGlVS3D9?= =?us-ascii?Q?G6siDVLjvZ+B2n9ukqcYZXUQg/la1pVO6Gn5YuXLjuYrcRuJDq2zbPf/5Bql?= =?us-ascii?Q?ATgzui34x74G0Y8EK9o7gBG3lE/5Vihr0gfILtI9l3iqyDNVRFMV5Zp/0Aoa?= =?us-ascii?Q?T5Cz8k21RAXfYhXygjIyCZVWVxoipazrYRu52Mrq1/Tj0Aiay2t+z+njG8IL?= =?us-ascii?Q?xKDtlane5pKlai9UJuEqPPPkDxjRUAEfGQmOZVR4wGtYxgcocmXpr9BHE4si?= =?us-ascii?Q?aNSVj1b1qH74ibM4luJ/UNrG1HKjFjO+inXvcqf1O0kKkVIOOMZs5/+YNB/h?= =?us-ascii?Q?yqpq0jgvu8mQ5zLRb1hu8dtwJy0QK8se7Jg9cLLvRhClKqFE7GgUIkGFyN1/?= =?us-ascii?Q?6IKrgxxVCDZ3wEuR3CcF3sxdUZBv41G8uDkcHOHnNGD8LPD0Jd5GoMbZa5VW?= =?us-ascii?Q?zIh+Zp7ZZLbg3c1w3uz3BGkzgCcguFId7GMDMK9gykKemE0iudX5ABgcKkaf?= =?us-ascii?Q?7gBjOTaXlspO0uigpvz8+sqcuQ4gggLOZe4qUbClok9pqLYaBqhsKSW7iX1p?= =?us-ascii?Q?1XaSS5VlEvS//XR4UNaIWMI6yb+JFx2zJ3IloENpAoJTp/svWL3BZjZyep+D?= =?us-ascii?Q?ySdAPGcztDdKB5pqwSBY9IvxGZDB+aJWpXqnEbzDjFzqKPsG4rnK5b/N38At?= =?us-ascii?Q?cEpuJxSW3NrTXE7NLMEX7AX7IfqNIZuOeMhZIJ9roqAFprvc6xLeRHACkd0k?= =?us-ascii?Q?Mwo1KDajT0EjyxO4iAzBNq263ZqPbNwMkARU+h0w2s5kBlN0Z5XfZegpHoOQ?= =?us-ascii?Q?3OE3Dyk/eh5BCLgxhkDE54ula6txYOpjBiIGrrR6KM0N1A3/gdVW0P39w6CO?= =?us-ascii?Q?jS/EO7aXIXwbPdjTug9/NFD12HoSxrmCHEZhy59JPKiyZ55ebdxmSHEBchFy?= =?us-ascii?Q?me0fYUOsu2U4puFISphToJZq6tPCVGqm+mp4XvuXAaEIcmuDtHLhLbZc0Ulv?= =?us-ascii?Q?yk2wz5wVrfKodoED/qzW02vHwivmDrvZoyvoZ5mJBDwESbY5Hk63zxZnSimI?= =?us-ascii?Q?s2e6yz6ZoyaXgVH2MebnGzJC2Y9eCtA5YyDkLY56kTRtiCCcmAfO+ZcOMEqm?= =?us-ascii?Q?w6iC7GpxVKsPxdtIOUg9wBrUvUguGjA2KnohLitppi/e1FoK9Ck9jlctfISc?= =?us-ascii?Q?rWmBgtLeMOPsqLyKHjNHHn3IdXpusZd9srsYYVYgp41oNdIzliriwczEDOTa?= =?us-ascii?Q?fEosT+44VaXRb2nMYoBMe/6QVVf2ZuJT1QB59Vg5tIfWqvdArVOo0lZy8Xww?= =?us-ascii?Q?oHjwW+im8b9rFVCa4y/+Q7f8ZRt9wsPHNhf18NZYsTZGcaCmYcRSz0S7e/4S?= =?us-ascii?Q?Cv62bw=3D=3D?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: rI4ly2TNCYvdiyE3Hc8oCtGEQrRb3pYNwBG8fabNRCGzqcXW0fp1y2oiuR3x8I3LFW9j90lc0oASP9DZ7li24zLGcOG+GFJa8+UuGMfnYhiiOE+ibZ6pwXgheK6h8A9/arMMoFSa6e63jTEsAguFUWAj/+yHD6Sp2y4w0vgBxjToBbIEKSFVdBoWuXXoFBHO/2MHNZlckgeH3IzqOM6uJyihpHomXprEx1ecHl7LD3bzhpKaQEFk8BS9s2ZBK90TN1KKsr6bisoWkcIIevJBYmFsowobnBvjgp3f46B8qXc6DGgZDEh8vxy6ZzLvkX+2YJ8Gp587xVEPV6lR9hvB2h+nm2Fid365eD/LVJcVycYfyUPzQQPT1QgmpXTonxWbRmt5GJWVtsdJN/sJ6e6gN0/0H8kgz+aIUNB5eQP+kDUS9DGbbELKaOSC9o4a534L4YSAH3f5qlAPZUuTktQPFLTQsL3YoPMJRlS17fmExmIT5q/5o9SHq0p80R1Xrqu9qU1qjfFP39znKrnUAXdRwJzJN7FMHtMOFImcM6wGDbybkUfB5zE3N8TKanaBHrCmlx0xpG58qDjy4AmlgxE8VQQnTn1lwbhBXKh/4jXmEiM= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2ece34a7-7536-4702-8bd4-08dcf383df80 X-MS-Exchange-CrossTenant-AuthSource: BYAPR10MB3366.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Oct 2024 16:58:09.8223 (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: 7kzXrWoL9mK1Dx0nvdRQ3ApqO7lF+ttKovBx2Kmyo3SFRjmxwqqh3Nkx000YhVNmJ9gdFCcXNi06uAw63D2/SGjaRRHTLYVpvCHpYVRXD2g= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR10MB6727 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-23_15,2024-10-23_01,2024-09-30_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 adultscore=0 mlxlogscore=999 mlxscore=0 bulkscore=0 spamscore=0 phishscore=0 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2409260000 definitions=main-2410230106 X-Proofpoint-GUID: DQYH5xMKQEg0mgnm48k15b0fIS9a0sqZ X-Proofpoint-ORIG-GUID: DQYH5xMKQEg0mgnm48k15b0fIS9a0sqZ X-Rspamd-Queue-Id: 92D3AC0018 X-Stat-Signature: wq35k14rzn9gsprmbtc8qfydtji4ueeg X-Rspamd-Server: rspam09 X-Rspam-User: X-HE-Tag: 1729702709-403796 X-HE-Meta: U2FsdGVkX18fAPQaXOQNKLpaq8roUEH3Qkuu0FnWn3jwdzanEZGVXPVCzSTunD9ofrL0NnmOnu4CJLSC8b+9nrFsOiAO0avvg2aEFKu53NipGVmzp4aIRTnLN5AJwDqQx1r+xLsECER0uPMRxueDx5mpxc1qfmDHRMzdyv9bfQWKuNz/LC4I+9Pp2k06z+nVQ5+jlwBrFYAI0SVdf9k96xpWloIpnBgYYqtVG97UzIIA+Fmg6eJsyhdlWi8blfPHCa8tHPUsch1/Vmh9yo9/+kZT1+dqDygFSZbAGZhb2jkryZHqO8muhfaLMcjCfxbl1WuUBofQE5D66s+sENXAGEdPI7B6pH2XkNmojh/CGMLnSWE14zfUAZkBE+u3sLI9dRHIMFjV7oSUCOdwEjK1tpKnOeo5KIZbNA7KLwmb37vRm42+w5Yw0mORuEEQl2Kh+XnRo1Z9UG/X4KHOWxkLYFJ8jJzBtmvwpjI4c5NkOc4FRd12tLbFgMtQMo+jJU4eDoAZ1wGJOOjmJq/smTBxv2o4e0iBvhl10ywTpTTqljZBkOM61qIRbkC9Ks/T3wgrcjHTrRr2TOJFcaXE/fw+1qCVImOUMCLoiCw9azeFfYSBD5Q8v8iOAFUYn1/0Ad/VtLbLhCRFFLGP5iOHfEOFfKspaEqXwn2ygHx8ohYaDxPuah/Lt2mvBGeS8zx3iL7zFcJhOmrUIFlXDjmozE8og4tmrqth94qL59rxFeFmYPqTFkZ4IzfjgWtIDOgZhCoaAYgt2sWiMqqvT6mSxQuuk6NjKKeYFzG9H+glqy8NR3d2zdqBslGlzk0AOXQTPcO+bCT7dH7r35WaCaXbgnYNAB9xZ1hADttlAaeufYpUlI9C9i9gD68QWeZ+k8feIKN9A7DGkBdip9Yi/eLl4Pqql4RwI3wLnxbguD3L0QKKVPz9J7IP4zoJaj1/LS6i0C6zj9TmM9MaGn/D4Pk6pJI P9k6Abml IG3Z/72NfvBIxjCrHhWAPyQcKAR+4ITGcga8NEu4i3cMwKb4hOUW653Oac0B8PIV5Qilna8VLo7Lcx1mobd6c/F/WdrUdXR389AjRutm0iIKbQthMtyYHVcVFoHhEUOMP5RRbxuxRbH8IaBhx3qO8X+6pt3XLO6B8rEphiNNpyPh9bP/Tb7TlHm0KVV2fR+caUf0q4neNmTLFyPLWEeTD4R+5qbmUz8jt8YXkTHoZ0dbv89fFiNzVrbw/u2ikbPkpAH82LIRaRSrUF9s30fuBZFo+cS7cSgqeobOB8vuUffQtHjNXBWx3/lN5MtMOS0HYucJ39t+vs5Y86wkA5mEzF7wtrt9GMF9fDdQIFhdJlCNJRm1bpIuGnyQndCX3QBMvoAINlPVvcsQ0PtSdqdUHwpuXePslJnH4qpSa81E8SmWxMqyshH9N1BghnxQBNFmrhhZjgIenK75KWw4Igde3WMz7+ZKUvEHQpV0ATSTntv9SbjZei2eOoxWlGrlEPdPIzK5OUazHRrh7E96E+COjGJvv60V1ALwuwdIBvnsxXc6OvtK56XX+5ZOQGQ== 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 Wed, Oct 23, 2024 at 11:24:40AM +0200, Vlastimil Babka wrote: > On 10/22/24 22:40, Lorenzo Stoakes wrote: > > Incorrect invocation of VMA callbacks when the VMA is no longer in a > > consistent state is bug prone and risky to perform. > > > > With regards to the important vm_ops->close() callback We have gone to > > great lengths to try to track whether or not we ought to close VMAs. > > > > Rather than doing so and risking making a mistake somewhere, instead > > unconditionally close and reset vma->vm_ops to an empty dummy operations > > set with a NULL .close operator. > > > > We introduce a new function to do so - vma_close() - and simplify existing > > vms logic which tracked whether we needed to close or not. > > > > This simplifies the logic, avoids incorrect double-calling of the .close() > > callback and allows us to update error paths to simply call vma_close() > > unconditionally - making VMA closure idempotent. > > > > Reported-by: Jann Horn > > Fixes: deb0f6562884 ("mm/mmap: undo ->mmap() when arch_validate_flags() fails") > > Cc: stable > > Signed-off-by: Lorenzo Stoakes > > Nice simplification. Nit below. Thanks! > > Reviewed-by: Vlastimil Babka > > > +/* > > + * Unconditionally close the VMA if it has a close hook and prevent hooks from > > + * being invoked after close. VMA hooks are mutated. > > + */ > > +static inline void vma_close(struct vm_area_struct *vma) > > +{ > > + if (vma->vm_ops && vma->vm_ops->close) { > > + vma->vm_ops->close(vma); > > + > > + /* > > + * The mapping is in an inconsistent state, and no further hooks > > + * may be invoked upon it. > > + */ > > + vma->vm_ops = &vma_dummy_vm_ops; > > + } > > Nit: if we want to "prevent hooks" as in "any hooks" then we should be > replacing existing vm_ops even if it has no close hook? If it's enough to > prevent further close() hooks (as commit log suggests) then the > implementation is fine but the comment might be misleading. We prevent hooks _after close_, if it has no close, then no, but I'll update the comment to be crystal clear. > > > +} > > + > > #ifdef CONFIG_MMU > > > > /* Flags for folio_pte_batch(). */ > > diff --git a/mm/mmap.c b/mm/mmap.c > > index 10f4ccaf491b..d55c58e99a54 100644