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 1791CD49219 for ; Mon, 18 Nov 2024 12:53:03 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 691B86B00BC; Mon, 18 Nov 2024 07:53:03 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 641556B00BD; Mon, 18 Nov 2024 07:53:03 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 493476B00BE; Mon, 18 Nov 2024 07:53:03 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 268D06B00BC for ; Mon, 18 Nov 2024 07:53:03 -0500 (EST) Received: from smtpin30.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 81A7480150 for ; Mon, 18 Nov 2024 12:53:02 +0000 (UTC) X-FDA: 82799204334.30.7EE7C1D Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) by imf08.hostedemail.com (Postfix) with ESMTP id 7393D16001A for ; Mon, 18 Nov 2024 12:52:26 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=EBumZbYo; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=RUxfdFFM; arc=pass ("microsoft.com:s=arcselector10001:i=1"); dmarc=pass (policy=reject) header.from=oracle.com; spf=pass (imf08.hostedemail.com: domain of lorenzo.stoakes@oracle.com designates 205.220.165.32 as permitted sender) smtp.mailfrom=lorenzo.stoakes@oracle.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1731934197; 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=z2fLwLXRkVqkdlScOlsvBwwRwHkdtIZPtxYsEp0OKRg=; b=sJFDpGIEFQlBwQX63fway4WEw/QjWeq91sim9jRqNXJRNhcnwXzX6VWrYdI1xXg1Y8eNEm R8ikBAahzB2ZntDatSTa0L/5msi12jPdPhuis401QKZ+tpmDsPt3AnErBG9KoeZNBFrCo4 959YSMUbfWNtE1r3xaDzYLBIJ4i/IkM= ARC-Authentication-Results: i=2; imf08.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=EBumZbYo; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=RUxfdFFM; arc=pass ("microsoft.com:s=arcselector10001:i=1"); dmarc=pass (policy=reject) header.from=oracle.com; spf=pass (imf08.hostedemail.com: domain of lorenzo.stoakes@oracle.com designates 205.220.165.32 as permitted sender) smtp.mailfrom=lorenzo.stoakes@oracle.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1731934197; a=rsa-sha256; cv=pass; b=CMboV94noS1qTx5gpYqvkb17mRF9hmxUpoV0mLALiYUFcIiUkz9sHx+s0PES2IAIR43lOI x6kODQizh4whezIn9I6ot8BDK9nP5AFKuKxp3LzkXWziZTvBKdiGa2vlcydy9MQuBHEyGE AscRDhk+/TsKqhcAWrmknv4XYHQFe8o= Received: from pps.filterd (m0246627.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 4AI8QY4t001059; Mon, 18 Nov 2024 12:52:56 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=z2fLwLXRkVqkdlScOl svBwwRwHkdtIZPtxYsEp0OKRg=; b=EBumZbYoDEYpJdJ4yXd5JRgWI2u26EKSOo 1uB3LWsPxP0dR6dcFHmUtj7JyM6JznN7iBVcGQX3E5vT6IEWlQX02PJQTWab4IPE dYbJCBYb4G1yCpidbO7ms/FMykXlhLNmMcjrNm83/cskt+CVtFWbAYRCbFtlQGOO bkQ6+bnrH4L3tJhNgtitjsCQsUfqT0m0Zb4XuAHRfhK0ZA1bPdWQmGujchegq/Ul dn4YP36e9TOvqWi21acb9kZWIBSFVBtyLnRpSV6IYikuGyr2wk5/3PH3xzZ/IrMQ qdcwKiyOT6UAwVBXctYHrmThurHqdEDaZ2MMetKuqff9UKikMjYg== Received: from phxpaimrmta01.imrmtpd1.prodappphxaev1.oraclevcn.com (phxpaimrmta01.appoci.oracle.com [138.1.114.2]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 42xhyyakw1-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 18 Nov 2024 12:52:55 +0000 (GMT) Received: from pps.filterd (phxpaimrmta01.imrmtpd1.prodappphxaev1.oraclevcn.com [127.0.0.1]) by phxpaimrmta01.imrmtpd1.prodappphxaev1.oraclevcn.com (8.18.1.2/8.18.1.2) with ESMTP id 4AIB0Pf0009022; Mon, 18 Nov 2024 12:52:55 GMT Received: from nam10-mw2-obe.outbound.protection.outlook.com (mail-mw2nam10lp2046.outbound.protection.outlook.com [104.47.55.46]) by phxpaimrmta01.imrmtpd1.prodappphxaev1.oraclevcn.com (PPS) with ESMTPS id 42xhu79c32-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 18 Nov 2024 12:52:55 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=cRuTNn7pm5u84Rs/hFrUnOkeW08/H6Cdv8gE/a/kHxbDd/5W/demhdGXJeccwRvEWwxVaGHx26PXCOB6jaD/5KdZQ18aJeSz2iuVmiEXXqrD8RnzGWWjOTR9aA4kTjwzPJKHTi6hEwAxJnjgUGVHVQWEVECciu+2OArWKi7Z9VpcpMsxJNkSyt3pcvV6lSSRfa5LrcOEupfUM/pARW4EgRES44D2RKKOdVCvg4sngaOAsfnyGLXXPi0DaBNzpz2pDAxdkUDDln7d/5xo+YzdedbazO96naUs3JmrzYjWPzB+NesKpKFgIhQD//6CAELZYwimRvBNVsj4rmXV9IOKBA== 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=z2fLwLXRkVqkdlScOlsvBwwRwHkdtIZPtxYsEp0OKRg=; b=rd824Oj/gs3PblIZ1fIAseaqVZv5AR7NOkLVp7zz7ZsHQF64giestEcuACsM6RWK80xfMJi3ty+y1i2ndCym0ZbSlhS+WoPhOZKWysv0jn/E49xpeFGhmU2hWOHeiWpM5yIib7PTgN/w4aQDddaeXn+xIAtWyHAzJHG0DOhbkPdTVSLsSQuOmwQF8SQUPqQkTc1Z17+36fAqjWZd7P+HXtg10tEyY696AHITY/zK1LXX0fV4Zfg4gCM9nspuFx28+hBbSmOSXxIuZYMTFETtajrya8D1B9e1p2SC8aICWkoy3SGlqUtSEESXn1Mgg4waLVKy2f+UB82MpEvW76ACUw== 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=z2fLwLXRkVqkdlScOlsvBwwRwHkdtIZPtxYsEp0OKRg=; b=RUxfdFFMgKWvcijkvREvqRhdnKBwIrBffkYeKRawhNdHrasIUVT7bDG5LaZV1gyyJbDy6j0UucBB63FbumsTm9BBHAr/s1QIBN7URHCINAlDSjf2KB1DHOMFYwoZHeXkEG4Z++JL5TXwZ5xfnyRmDEvK5i9ilXhu+lO38+uoy6c= Received: from BYAPR10MB3366.namprd10.prod.outlook.com (2603:10b6:a03:14f::25) by SJ2PR10MB7559.namprd10.prod.outlook.com (2603:10b6:a03:546::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8158.22; Mon, 18 Nov 2024 12:52:52 +0000 Received: from BYAPR10MB3366.namprd10.prod.outlook.com ([fe80::baf2:dff1:d471:1c9]) by BYAPR10MB3366.namprd10.prod.outlook.com ([fe80::baf2:dff1:d471:1c9%7]) with mapi id 15.20.8158.021; Mon, 18 Nov 2024 12:52:52 +0000 Date: Mon, 18 Nov 2024 12:52:48 +0000 From: Lorenzo Stoakes To: Wei Yang Cc: akpm@linux-foundation.org, Liam.Howlett@oracle.com, vbabka@suse.cz, jannh@google.com, linux-mm@kvack.org Subject: Re: [PATCH] mm/vma: check retry_merge only for new vma case Message-ID: <7e550e5d-88f7-4407-91e5-2b4ae177b05e@lucifer.local> References: <20241118021823.17386-1-richard.weiyang@gmail.com> <20241118123439.4zi7ebdelwg6e6wh@master> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241118123439.4zi7ebdelwg6e6wh@master> X-ClientProxiedBy: LNXP265CA0001.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:5e::13) To BYAPR10MB3366.namprd10.prod.outlook.com (2603:10b6:a03:14f::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BYAPR10MB3366:EE_|SJ2PR10MB7559:EE_ X-MS-Office365-Filtering-Correlation-Id: 980da9ef-25dc-4dff-35ed-08dd07cfe9d6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|366016|376014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?+BmC18JjpvwUwUhjVzxdPI7xjQMkQjPa0nMRMzM9Kap5MfSOo/N5QHRL2xQr?= =?us-ascii?Q?PSyaLfAVDL4NQ3Kf7k6wAA89QEiQvxtJavbsc1mRn/BKuBueiyXOegQAO2Pl?= =?us-ascii?Q?gP/gfLn+0BnZBl2RSVhjcX7S2grznnqaQ6velqzyELfq12mYiakPg4K0OP4+?= =?us-ascii?Q?ToDwtyO9XVUNgVmBVqru+L61yjbE7iGyUcaXQkB5SNuaEgThybly2aItR04v?= =?us-ascii?Q?PB8kZhtAABM6yUKkSH8s6eebEaM74cPBC1WyhSFd2NFuMlo0N+nDG+8GQ/o+?= =?us-ascii?Q?zAnkqzliWt/ERNxhvKSPWS+GiDqyQJzVYLPkmL6MVVFcus9drwpKqaPiNhBo?= =?us-ascii?Q?byrLWWW9qdWkNkxI4v6jhI/ApFy7aGag3PGEQAVhTTtoFQeYtmDn1YamY9wC?= =?us-ascii?Q?v+sEjV97vdyL61XCgRfdn49heoXXkfRAAERLgw0g2h4gfIDfN/9agjDKWPJR?= =?us-ascii?Q?KVXCXbmmF3kK/Rdnkugzr3dlP1bibAXwv8VG+wUJ1JIYkofZoKONun8EskO6?= =?us-ascii?Q?Ckhh4fAFo86sK6xZv7WQgIpMW97N08lTSf0pOF2GI6PwVkod3HCp81vkXiPn?= =?us-ascii?Q?sPVjk7W9NxVyEfhx6vjZubYj+bmEduFbeDes2xrdGdzqV98HMXVZaTq0YaWr?= =?us-ascii?Q?X9Sfrn3WV3zy1vMgNlRjFISv5jrOzWwaBfbrnEYm91VWuhKtY4ZxyFwt6XXf?= =?us-ascii?Q?VgKPc0w7Gae/9i4St7Vj3zyAqR5G2qNnerxTwn2gllDha8xrWVaiojmy8WG8?= =?us-ascii?Q?og7h2pN6ArGccblMywJ7oyRwCYgj2mcKk3Y3pcikEwn/7EGzVHQz1dkjgAjL?= =?us-ascii?Q?K/gnQjXXTtTfNZSKvPX4KG076JH+03tJKZr/fNIPPc/lSmiue5NuFkT9COWs?= =?us-ascii?Q?SZB9DHT3gKQfPwJnBUYj/cQukQyG1aTURH01Fb9NSWIn6cd0YU0e7W3qfg37?= =?us-ascii?Q?djdQmkH+yKgWX+I4lvqr8E+5D30AtbtWgKE+3xP7zpxP7wFx8nxQuCL+sUAc?= =?us-ascii?Q?NPOhPwNFGAOuNeY2SoLLs9K+Z1IkG+oC/C0orHOdhLbBpj2ORL2OS7NMmdUy?= =?us-ascii?Q?F3EFCQ4cCTIrKG0N32w1jlvCTdy5bhldFKOyn5omWDazkudD/9RxvluQhxhS?= =?us-ascii?Q?yFML687zX7+vicXBzYXYjVosJOt3u20da8Di54aqlEe1/A+JLz13KOZTqiRe?= =?us-ascii?Q?TxAbYOEIU3QCdSPOouoeFab1jGjCE/9hgXIdIioJyBpyPVu8XacSXlZ0E0gg?= =?us-ascii?Q?8rV7YfMUtTIcMAF7Ttz+rMsIvfMGWGVSTeznvpNARqh9nR1eEOtgqjY/8jNk?= =?us-ascii?Q?TrHqRQVPGJycmGTaj+fjTG1e?= 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)(1800799024)(366016)(376014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ioBEoBxOmVRXK+ai2D5tHvPguG2qldbw9Fe/8f3crcuxD68S3AT9ojzo+Mpp?= =?us-ascii?Q?oQvZY4PiLtq1JFccBkPvKWDrSxdysLZtT6GYRlvaUA3cEYCuxAA6Z2nDa2ZJ?= =?us-ascii?Q?ngT1WRjcdhgBFEbBAhY50FeQy5Yn/ox8WjLpKLw0JpcYZR8qAlWjL1KwtgVt?= =?us-ascii?Q?OCLvLj7UsZg/uioLqTIam9uCGXNgaWJ9orKzXfZnikItRpRpqXvps3W/yLf8?= =?us-ascii?Q?b2pner27KTT6nTsChnTMh/1Ug3xQd+QBnZDiu0NpLFJ0OOw7myu7UoS4XoQP?= =?us-ascii?Q?TfjANXFOf61wQYLb3B1OzgW68AlnlmKkFDMRxiSsG7YkZpmzp2l1oQy4ngPk?= =?us-ascii?Q?lZ2IETQLXJuYbmIvsEL4XClJcKGeDxcNnzc10+NcldETAb2067QcK68ti3W2?= =?us-ascii?Q?gYhv/gp/t9rmoqN6vaSpUl6gQ140brIiEmAnjM/nEhIQtCXE7pB78M8LE2X+?= =?us-ascii?Q?T6hEVi1xMfHPrRc+yWPtzjjh9MG7idUViXSfhc33+rwLCSMhxmZi4KDwPkR4?= =?us-ascii?Q?BoCsBpZWZCi1h2PeGsHKvgThOeQkHVYaSj6MKhzShDOrUcngf8+cmLTscQRy?= =?us-ascii?Q?5jMlrH72trtGQdNbG7yNf2Ua9Y3/VV1riz2B9o6+fJiG9CGrE2FMfbjsKCbg?= =?us-ascii?Q?iWvXB1HWVSMx9ZRgfeK7uHT7CASqdqlhVcGRra2R+zB/9IK3HxF32nP5PnEA?= =?us-ascii?Q?bIr0VJohQMwQ2IBWAgxhY0H+8XOIvtq0MS9QupeZLgkX3ySX5FDtF/rpBMkB?= =?us-ascii?Q?NYabXfOt2LlTDzdXH5d78hhD5DVXZw5nVazZ8xcrIfdZG/RAZ5B1m4iA8Ve9?= =?us-ascii?Q?K5V1MsZQoNsAVVulxY+XoD9/LdehDz0SEypYiHJt/sDCh9C2Oyrs/hzRVqWa?= =?us-ascii?Q?x6ydTeWqC2K7aRPzW5jx86m6E/gJfOwFR1bUNIyf5fJH0U0SDIxZjtElQ0jv?= =?us-ascii?Q?5m23Uv3IS5Ohbjo71EvmpVsZTeOiTpkeLGgyP20IMlgIkhiVEZdd50OYRzpG?= =?us-ascii?Q?ys+gFXnNpUj9lU7zG9uCw7QvtHSWus0y+lAjtclYZYVhPb6UVXBPZm5KrDBZ?= =?us-ascii?Q?YDIGnPQMFiLpke74wE5xdb5ZpMCYsa8v5FAMn5IGk0KuDoO+ZF6k+/ooMhNY?= =?us-ascii?Q?IK3nN/wfwr0w3TlGdj7U9VOdjj+s5zTGmAHxLRpMgVXWWRdUUm4CSBMVhct0?= =?us-ascii?Q?owVtgFG09956/Ajs+wqgSmqbDbEyheeo+KjlDbQaxrBnzQoy9Z1yKHN+w1lh?= =?us-ascii?Q?erKFlPiw8RXpjdjvzqNWGzz9Oqr1lnG4qt6evvaXK5uS83E//yqyMp3HslAy?= =?us-ascii?Q?CsNc8qLpLvwNvK4TzQfe1ZeN2WAOLj5U60+J1J6xJyCeFmuac1KeCsIUqoxR?= =?us-ascii?Q?xZCka3iGrk0ARaaTOgC3bnqXnXtbqKsCtRVHoRttdwp0OV9PEORHo8dqe8Ft?= =?us-ascii?Q?QgjEJ48H89+mzJhBHtFVcG3Tph2cbERnbdhSFci6nKtJy7Jolpcrr/0N8QmC?= =?us-ascii?Q?iP2GoCdEx29N717n6DFTdcQrMXikku/wJTtUqGCUcySrDedsA2QbzXtUF845?= =?us-ascii?Q?sp5QL7MNHNKoWFVgW++y7vmYz9dUTNgwMc3vSnJS/2V0iP6JWlcAfLOdrsEw?= =?us-ascii?Q?ow=3D=3D?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: b4F3oq5PM2gTmgN+mwdvGMTw0GxST/KTqBmqRaT7DDA8tH6KVvfTD/91J4h1VE4SytXognYwa4wqlt3ZoBxg/gfS7HySjdbl6/To00Af7viO48OXvwA07E47aoWL3E4DipqW8vh6BstlCaVwi8zI/Y6BJu3YqCx+7py/2l8SZO4G2JcQ5HiXUsxzBeO3RYwgSaq4EGTj5PsA+/OmY9pB9TJXtoJsqK84EHLmmxyxB75DxGaxYx3EClmpdtwELER/D0OrEcDwjmx5cCl3AOlKY4pbmF3PaORbqcaxX2GewOHJQj8SLWOK06BG78IHdA0WVGPYZq3sknHeYQPj8yUzvlVVwDzOEElNNdhmmnb6Ow/6abOQun6woU1cfjSSGXyHVLt4FcfqXC/VrDl96L7tFx6Y43GOVZ4U9sfKylwhLKoZy/W1gO1M+mBNsNWvZH30FX0C+ArFb7deeD7bEjiZJ2cb0v9FIUyZLb9QY6tPmk5UnWsEkIGDsOZK/3i//D9FGcW3+8BqX3BpLXHVly1Uy8TC5uLIridOJ/+QMPVr9Oy+qo6aQlQ5HF/6H9irHLQIyNhEJ4b7wUrXQyxbdefdZkjVaMCTBnFJXef2jipom3w= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 980da9ef-25dc-4dff-35ed-08dd07cfe9d6 X-MS-Exchange-CrossTenant-AuthSource: BYAPR10MB3366.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Nov 2024 12:52:52.1618 (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: uqdHPv8YkuCZdxqMf1dCpA8rldj43ZSlINeEd0kKypmSBuQd/drAl5riDC3SvFLAlZ2GMUfe/LzTRhKJq4eQXI8EciZ7nL91Y0iBeYnMpuI= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ2PR10MB7559 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1057,Hydra:6.0.680,FMLib:17.12.62.30 definitions=2024-11-18_08,2024-11-18_01,2024-09-30_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=999 phishscore=0 suspectscore=0 adultscore=0 bulkscore=0 mlxscore=0 malwarescore=0 spamscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2409260000 definitions=main-2411180107 X-Proofpoint-ORIG-GUID: tBu7v8pcTJLFWKOaeUtOOwqIPp-YWe6P X-Proofpoint-GUID: tBu7v8pcTJLFWKOaeUtOOwqIPp-YWe6P X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 7393D16001A X-Stat-Signature: x1nwsz8t6j815brehdax1srodubhjwgo X-Rspam-User: X-HE-Tag: 1731934346-103828 X-HE-Meta: U2FsdGVkX19Z0DJckZW+V3hwD8/gMU5F9N8pG1IU5DX5Xz+8EYQphSoftSeH3KROleaKEOkwY7kK+COxKNC2IPFL7KOko7EDECpe341KkhDozAuQqMJK0sx6V4CoLEryjtm+R4M0/xHnCnUwM3H+M4egREWIRW+0+thtXzsMmy1PcL/QJMVg6S8xkNUfmUU7ta2+7HdB07bTKZaTRoBWg1NXF327dG2ZrXJnVz9/4Y4QVlfv22qKpwWJnFusswutrPvmJ2HhiGUz2TTr8MWRAFmQLQ/yvdH+qpF5bn46ev7duf8S5gTTku8uB+Kx3jHfa6vxURbmivYDOpeqvysN4QD6g52Gf6PVfkGwpKOL/5GqfjO9xzpHIOUADGkTrUaX5BMeBQVnOSSwMScrAO9UiAKpr2SvB9IQEvZaivKk+df4jmzCWbyHKDhExJ85KIlnf9NOHW1Vn4dF7F2WivBLGkpeQj+6VgiKH7LJ9B1d1ndcKyPwT3u+60Kd+L5NA6S6J01sQ7ZfkQOnALaDjj55kUDaBoVK5ox2DFJ4pUXbwaiOYbDpxG45IxiCSNOwfDoZS7TE2unBuk6Z4ZWj9TUfFeg89N5l1uwetY2pFhjtvVKGKNkOICdQXA9ZwhX4UjvluKmhvU/YfSysrh9TKYExaqO30EF0U+ydIUI1U603Q9QWt0+aOXiTGAp6jO2vdd1YcRIucNPh3zfwfGUrMHVW2FMee69huEsxj6pKy2KhprGzQ4TdQm623cjiTiEPXTsXzgSLi8ed8EjqcCa5i0xC62OScLLlOOFEmdHAjyQqVI4j6cLD4RXSJ4jqJo9Skr4wEvKupr3tT/ISjQ/6Doam53S7XY10+mUokRWmD542wIQ1vEg8tPboeNiyhD8qDEa/ZJrfgCVnphdn2hCmtSvbaURD6V3Hs1AB/SMtZpsAZ99Ns4a//qVCg/KFXtlrma0j1NDmggcvRdyjvEsd9RK r4l6tUi2 tZrGKGtyTLJrVuuGJ0BBRGFK5t3MMtF1NKdwMYPSCRMdKYIDhnRpArNBronzFeNbvuY+VRNu71Z3hGHP7RaHjlPTmghGPm6jVyojzYJIXyq5/YFWmreqhpFyJSOcJpm1RpyUfkd7FszDb35eDkG1rKY29Jz4DZ6f++s9fAlauOQzHYc0DR0eYgQYpLm33QgxOE8FRcN+XqSTOKE2Gw9YZgzDgG3JnTthvDo3mz5EJ0EaJzFm7jmGvNYf21Qj7sr8+qQRZJ9zm6ztbf6gPrvGFh44nYxwJF91aSsGEFuf+nJUZXekmfHyqyJJ0BfKf7278D/inRid48WvO3ggyYmlhwe1wtChKTK/mQPVCUfdTOzidQXo89bgi1dFQWOy773k5Q5fTHRkkKRFEi2e7zUc9kQgA2Z6go450jLQabuX+4Cs1iJrQs2NPK1C5+XUXhmPCGp2kl3vVqhH/nuqvt0JDxRQA9+4MnWkUABcd3lpSbSL+XixuGkaUukV3xIidRqeP7AN1YvXyplIVeJ5u+tC5If6tbUHWKCASNdnXUdxCpKKIcL/wLJgWWMt5f2/E6cLyOQWXUToxHpyCzPWYdnQex/7FiW5l2i7cliupcLwf6j8hQPXSth7iWS/7GS0u5Kio4+YDiMDioKm1WG5tPRzWArpSCrR4RsxQXmS5nuQWWIVutos= 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 Mon, Nov 18, 2024 at 12:34:39PM +0000, Wei Yang wrote: > On Mon, Nov 18, 2024 at 10:04:44AM +0000, Lorenzo Stoakes wrote: > >On Mon, Nov 18, 2024 at 02:18:23AM +0000, Wei Yang wrote: > >> Current code logic looks like this: > >> > >> __mmap_region() > >> vma = vma_merge_new_range(&vmg) > >> if (!vma) > >> __mmap_new_vma(&map, &vma) > >> __mmap_new_file_vma(map, vma) > >> map->retry_merge = xxx --- (1) > >> if (map.retry_merge) > >> vma_merge_existing_range(vmg, &map, vma) > >> > >> Location (1) is the only place where map.retry_merge is set, this means > >> it is not necessary to check it if already merged with adjacent vma. > >> > >> Let's move the check and following operation into new vma case. > >> > >> Signed-off-by: Wei Yang > > > >Nack. > > > >As I said to you in another patch, the aim isn't to find the mathematically > >smallest size of patch. The aim is to keep things readable and nesting > >things like this works against that aim. > > > > IMHO, by nesting it, readers will notice this case only happens for new vma > case. So we don't need to retry if vma_merge_new_range() succeed. By this logic we should not split up the functions any more, because the reader won't know that it's only if you create a new VMA AND it's file-backed AND that succeeded AND etc. etc. etc. So by your logic, we should undo every refactoring here since we need the reader to understand precisely the conditions under which this can occur right? Or where do we stop? Because now it's 'misleading' to imply it might always happen in this situation without stipulating that there are other conditions? Do you see how this breaks down? We encapsulate the circumstances under which a retry_merge occurs, using the map.retry_merge variable. That's it. It's simple, it documents itself, and a reader wishing to know when this happens can very easily find out. The purpose of this code is - again - not to be as small as possible or to avoid all possible branches (unless perf numbers guide us to do so) - it's to be readable. In the original refactoring I can see: - prepare for mmap - try to merge - if merge failed, create new - maybe do a deferred merge - mmap complete, do 'after mmap' tasks. In your proposed code I see: - prepare for mmap - try to merge - if merge failed, create new - if merge failed, created new, and we required a deferred merge, do a deferred merge - mmap complete, do 'after mmap' tasks. It's about humans reading this and being able to understand what is going on. > > It seems to be more readable, but maybe you have other thoughts I missed. If > you would like to share some thoughts, I'd appreciate. > > >Unless you can demonstrate a _significant_ perf improvement for not > >checking a boolean here, then you're making the code more complicated than > >it needs to be for no gain. > > > >Right now it's (intentionally!) simple - try to merge, ok we can't, try to > >map a new VMA. Finally, if the state says retry a merge, do it. > > > > So you want to make retry_merge a universal check state? I don't know what a 'universal check state' is? You mean we check it regardless of whether we previously merged or not, well obviously I do want to do that, but for readability reasons as outlined above, and previously. > > >It's easy to read and easy to understand. > > > >Nesting as you say takes away from that. > > > >I also could have gone through this mmap() code and tried to reach some > >mathematically perfect level of avoiding unnecessary things, but as this > >wasn't the aim, I didn't. > > > >Please try to focus on finding bugs, or _demonstrable_ perf issues rather > >than stuff like this. > > > >Thanks! > > > > -- > Wei Yang > Help you, Help me > I would strongly suggest you look at mmap_region() prior to my refactorings (say in v6.11) and after, and honestly assess whether you think the new version is more readable than the previous one. And consider that we have run into _countless_ bugs because of this complexity, much of which very much tried to do exactly what you are trying to do here. Again, we appreciate your help, but I strongly suggest you focus on bugs and performance optimisations (backed by numbers), rather than nitpicking things like this when the code is incredibly new.