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 C87BFE77184 for ; Thu, 19 Dec 2024 03:51:20 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 21A5C6B0082; Wed, 18 Dec 2024 22:51:20 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 1C9E96B0083; Wed, 18 Dec 2024 22:51:20 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 0432C6B0085; Wed, 18 Dec 2024 22:51:19 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id DB71C6B0082 for ; Wed, 18 Dec 2024 22:51:19 -0500 (EST) Received: from smtpin21.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 4D880C12F5 for ; Thu, 19 Dec 2024 03:51:19 +0000 (UTC) X-FDA: 82910332386.21.3AF5A22 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2064.outbound.protection.outlook.com [40.107.237.64]) by imf16.hostedemail.com (Postfix) with ESMTP id 5DEB8180006 for ; Thu, 19 Dec 2024 03:50:43 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=VxCCr7b7; spf=pass (imf16.hostedemail.com: domain of ziy@nvidia.com designates 40.107.237.64 as permitted sender) smtp.mailfrom=ziy@nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1"); dmarc=pass (policy=reject) header.from=nvidia.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1734580251; 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=86RPvin39O4mywxFbwStx0eOX7qewHxeLz/DWiFsnZk=; b=VGW3K0FK9fK1468RHkV/yNbnac+kzAV5nadW4ILPruhUBgGvAvgkoQB0yBiyTddJFNpyAR Jh+rl5DPbwxYKoBpNTSO/yQ+vEhO3leIigm/kU+0vmmzS683UdllO4whZTloIuEIN/cbrc EfuKnZ/HTq/BLFr1jpQX3fooieebevQ= ARC-Authentication-Results: i=2; imf16.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=VxCCr7b7; spf=pass (imf16.hostedemail.com: domain of ziy@nvidia.com designates 40.107.237.64 as permitted sender) smtp.mailfrom=ziy@nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1"); dmarc=pass (policy=reject) header.from=nvidia.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1734580251; a=rsa-sha256; cv=pass; b=m1heekcr7k04+0E89zJvxE58LeG2t70Sqjmm8BY2jQFPS36nBkRoGsGOD2Y+7r4CFNntiB 0JFfsu/0E8wdy7gi8YXpx+oazAPCSzoH66eUgzn8vLtUK+itkHcHqbVZG2ob6p7srALFII B9OB/fodZ2ztcrVNg+cxgqwEx+IhWgE= ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=cjEPjG9IFb+BodJSth99oZo/EegNpV6Cl3ZR8D8Qie6dAnVyYxn0DOrW4yOXBH8CVtnB/TwPYrIVD6w2kaI0DZhcJ7MsA5O6Ac4mrqNHjathkxTVytoAXt6XQFOW9qV5xNK5K4xXivFJNLgFGsCsVcMikoKV2/KzqZucre4QiJthVy8yc5Lx7NrmOFU2BzQXLCHFw4ZgzIjpzIZEuijf2HDkbVg3yYIW6DjJ9Y2qw36K6TdIWTCIrwhJAZ2FQoZOrH4PQoSLJRYQ8jCsj2z3DSnAz1CUWOScKLTgYTwhlARt6m4tJCBAo2oGYHODcRLr5Y+YdhaTAS7Ami1oC6cZ/A== 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=86RPvin39O4mywxFbwStx0eOX7qewHxeLz/DWiFsnZk=; b=sD0VP1G2BrliLFFspCzAtMScOzj0Jg78GwHMMiqJeS0T3R4QYtTQhDwOsex8V5ZPqpJ6dXsRY6TE82BA0vabEltm+jH9HegoY7P7rNYN7o8QYgUbDOQLJAoly7xzzVChWIe/pyD+3UUFS+xZQO37C0NZWGNC0+d6hCHHncLClG+glECCeCcHKjl70TqdrnRiA/hjmRiRwfA8ynR2PxKJETlebs1vdPZLdguw3Kh3SrQ+9hAnRPtprSCOLmHkBKtetqlraSKqdqEbWT7jPI1BEVNgpgx2xnJikqhjU42Zj8aUv0/dWGhgX7puUXWNarlIZBRdpyYJt4mSmxyhK6vgpQ== 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=86RPvin39O4mywxFbwStx0eOX7qewHxeLz/DWiFsnZk=; b=VxCCr7b7/1kRd/m/OQair5QCKXTHjV9wrAyyYp89c7PLJj2/8QzbdmdCP76dLEoZZLUjljXovUeNYvPDfSopHQLRRCeriT8W8zRaA6hLWvnpCSFJeISbZxnjdx4MT2KG6jqhQ6trAMMy4zQsy4abiVeDmhlxZ5GvQVQEhCncVPr+youpCH8wo6pJ4Vvytyh4pwyg3Ed6Lz4i2bC8/AFx3VSuqt8lunynKxaetujHQ1kOqVLsYYAYNTC3Fl/xHq0mxU4GwBi99FBwHQh5eF+CZxpdzsWsXwxAzPW0p31u6MdGY6fYWLqoblGEDk+2yT6KkfTqfYJCbAzaen3GcLgnWg== Received: from DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) by IA1PR12MB6140.namprd12.prod.outlook.com (2603:10b6:208:3e8::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8251.21; Thu, 19 Dec 2024 03:51:13 +0000 Received: from DS7PR12MB9473.namprd12.prod.outlook.com ([fe80::5189:ecec:d84a:133a]) by DS7PR12MB9473.namprd12.prod.outlook.com ([fe80::5189:ecec:d84a:133a%3]) with mapi id 15.20.8272.005; Thu, 19 Dec 2024 03:51:13 +0000 From: Zi Yan To: John Hubbard , Dev Jain , Ryan Roberts Cc: akpm@linux-foundation.org, david@redhat.com, willy@infradead.org, kirill.shutemov@linux.intel.com, anshuman.khandual@arm.com, catalin.marinas@arm.com, cl@gentwo.org, vbabka@suse.cz, mhocko@suse.com, apopple@nvidia.com, dave.hansen@linux.intel.com, will@kernel.org, baohua@kernel.org, jack@suse.cz, srivatsa@csail.mit.edu, haowenchao22@gmail.com, hughd@google.com, aneesh.kumar@kernel.org, yang@os.amperecomputing.com, peterx@redhat.com, ioworker0@gmail.com, wangkefeng.wang@huawei.com, jglisse@google.com, surenb@google.com, vishal.moola@gmail.com, zokeefe@google.com, zhengqi.arch@bytedance.com, 21cnbao@gmail.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH 10/12] khugepaged: Skip PTE range if a larger mTHP is already mapped Date: Wed, 18 Dec 2024 22:51:07 -0500 X-Mailer: MailMate (1.14r6065) Message-ID: In-Reply-To: <00d429c9-6ade-42c9-a1f3-a7519375324f@nvidia.com> References: <20241216165105.56185-1-dev.jain@arm.com> <20241216165105.56185-11-dev.jain@arm.com> <7cc1840b-6f6c-4f82-86b8-41bb6fbc1b81@arm.com> <00d429c9-6ade-42c9-a1f3-a7519375324f@nvidia.com> Content-Type: text/plain X-ClientProxiedBy: BN7PR06CA0052.namprd06.prod.outlook.com (2603:10b6:408:34::29) To DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS7PR12MB9473:EE_|IA1PR12MB6140:EE_ X-MS-Office365-Filtering-Correlation-Id: 8757db3b-b6f1-4f47-283e-08dd1fe061e6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|376014|7416014|366016; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?lQjn+P07pPYK8eN3MgTgO1AbG8GT+vByimUxEl5JgmL3nCDf54e25SWW8Q2j?= =?us-ascii?Q?/D+JIaD/BaQAhtCXCFz5uPz2CHwsyevaPhI/PlAMyyI1KvpzD3lcyS7+AAiq?= =?us-ascii?Q?5khNvsfDY7BL86mIv8vjfG15c8dqUvStY9Mz5zS7/+JvrejcPVpWCfgFFq+F?= =?us-ascii?Q?4XU4AytJUEsOAKb0kvdxKt3HNTj2ewMRQWtC0e+TeMEMLmdS3+BsmUXBetDG?= =?us-ascii?Q?XpM1gALgBEC1W+8xuxppclnqNnuR8CbAjlC46Mpz39kDOkq+AgRehaC3aA5B?= =?us-ascii?Q?i5xb5JYCiKAMZzt9mAV6Tcea1uX70rjA28QaGahPf7UpqS5Z8QxCMQFvRFvS?= =?us-ascii?Q?8XL3f90iU182Ou9AChwQJesD/SD0LioZcGV0iJhVvPNPu6Q6kUYnO0NtlrAk?= =?us-ascii?Q?0+VU2L1ymmklw9JE5SjniEdQbErtldZy92+JfWgLaynaOJ/tJSGESrv58j3t?= =?us-ascii?Q?GSSjKZHmy0e10YMT99eclmuiUBNWEWuEDCNot2YNyE3TyF7QfW6EbW1Z4RYX?= =?us-ascii?Q?WVGgkDYspR0AJGbPab4JYDkPgYuF6w0Dkat+65YxLgBzQG7Trius4OXsKGbM?= =?us-ascii?Q?KCTL3Pq9SBNwD53yvDIeQUrsZ/wAMImBOg474ZqlYIKu6P2Dy70zICHKaK6V?= =?us-ascii?Q?Pa3tcRS8nPGe4Op1W1sPmxE4BYm6Q4gRS6TxWYGdwfSM1bfI4XRgWoHs0sZa?= =?us-ascii?Q?Gcs0pbP78Ix5lQuDxMpU+gU/AwsH9s4S0ynp8I2CHz0Z4PItbWrbx9y8pp2a?= =?us-ascii?Q?gT5ApcOmPN2YiUsypJcsTlur6mvOOzqpTXh9Owb6eW8fAKyToBz5sewVWUqy?= =?us-ascii?Q?g9K8q/bQ/1YOtAHYqufXMD4PTZTrAYci7HD2f9KY0Tdm18HhhDLTa1ts1ek0?= =?us-ascii?Q?voXiG4qHzN59dT9uGafHN6vZAQPg/PE5X343mvijNuNLS2d7cclXQsMwGX55?= =?us-ascii?Q?hZQeI6QgYlqVHSDTx9gZ2ckxuVAof3xOIuAMLNFM+W96CLVjV9zEXc8z+KHy?= =?us-ascii?Q?DZXjS7BQ21dNwFpq5+77dKmUgT2TMfx/nNhdhF/giKhrjTkQPfDjvVmAMhge?= =?us-ascii?Q?cgO7bJWAmWN6eA2+K1lUr8Vc/7+dUm/s+A7YTgJxkWyjwMJrMbDJGxwyl+tv?= =?us-ascii?Q?w8mLzMjQUgaN4Y9zd871uKmHey3vAix5O5uY/Ef2k1KrL4+N9YWneLcLx6Xl?= =?us-ascii?Q?j/IEyjhUZtDjEj7F41/wXzSRk+dYTKNtTUZhmHuIupury8BIj06aBzhEhTpk?= =?us-ascii?Q?2UHn+E9z4GHjMTaZCT2QDtCBtsx7OD5IvhY+izhCpEb34hKv4caZHcH6epkL?= =?us-ascii?Q?COyx1ITBRTW5ntkgBoi86AxsvYvDToWhQgF9NiqUdw1kW0PDVnlxHcLqeP8g?= =?us-ascii?Q?38ixgbUkuDWn2X7AfJgeJ9sVt8Gt?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DS7PR12MB9473.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(1800799024)(376014)(7416014)(366016);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?9/C3p2ElG7wUEs6GHMUSGtNSPCMdiSU5avGK3J5Y3702FozWC5hxZtpK2WyO?= =?us-ascii?Q?QKySBYMA/ziFowcOuAe3otIIoaJ2MCqf9N7ZxaFZ99lkS7n2VSwVhpuVBpLp?= =?us-ascii?Q?Pemxy+5zLQcUE+4VMafitJdK8GCGjv1pnN0By7v4bcUz8YPdRSOmWyljU4y1?= =?us-ascii?Q?MhqLcY7vAA5LcnudcxoLmF4ocEyyGdWQOZh9xlWrBMRvjpi8MylYTyreAB/Q?= =?us-ascii?Q?+we3Tf5B8vdaSZIDQAzEtBxqpRSGDfQSdLLam5y3HVMmadxUABFshGOWfzcE?= =?us-ascii?Q?dzEPyzuZQkrFWxS9kn4FgyfdFLpdxU2dGDttq2nhX/TQGpWs+7luDBQNC0WQ?= =?us-ascii?Q?Qhufe30zmn+TfEPevK/0F6ezZGVJICjxeWH4wNzAZ409h1Jyghxi/MmoFKFI?= =?us-ascii?Q?IdQh6vClvVlOe+fowiFpvVlg11Zb3haNCRmU3kVpJJz/NtvX9Svq/Acq8yNf?= =?us-ascii?Q?7MIrQCy9IA0/PHumvfM+t7jQfwdxKNCKDS/sTPBb5bOcf5/uD3pmFad6YFTL?= =?us-ascii?Q?3KpnfNSCq7Bdj5brm1EmcvZRppEm1YJb7qoeVgLYC2AhcXd/HmvT37Qmolla?= =?us-ascii?Q?O4/I9VpxGArC3B+uV7ww7Gy+WVF0EEHZSfx4kSAVLsIqoZUEbpT110LibtGK?= =?us-ascii?Q?wLonBu4/FPHN6AmQwXbVIAodX+FJcAlxLCD/p51TfRcIYkMDmNPxALsnIp9h?= =?us-ascii?Q?y3pujT7oXzmS/pjzCoFDA92Xmfxpcd4OfkxA+sZwLLe2HQRMeGZSj4rqAspK?= =?us-ascii?Q?3L/InmCq6LAVot8s9R4NfsrTy95jK5Hg5k5mySFDR8YqEwx2o8+OGoM9Eqs5?= =?us-ascii?Q?gwjwVQoL7v2pB/UHUvFgp09vOOqCJs7why81sKsUZKxGZmVU+ujCEp4ltcS/?= =?us-ascii?Q?0zBDGHR2LqY8AwWLuqFfLwcopIzt6dTy5UuNdgoYSbM1Uxa8KBnRGJ2p2DKr?= =?us-ascii?Q?Ytva+dwnx1SPLhjgvFzmpUcg4Xr4Es9NttE7I04hGtBVYxVDMIYp7NYM9/XI?= =?us-ascii?Q?Ia6aWZ4dSA19Fn0u2Y4k758VEQliCrTiCAjlDdLsPwELuupfXN+1nYOg2qBh?= =?us-ascii?Q?iIvNqtKEnR2FUdhFwDrGCjkuxa43wHndbzOeqH0cfplIHBVC0O5BNcPa6Ck3?= =?us-ascii?Q?nppjoPe44smhuejtIteGKPbPsXSKauY7wF/jkNpYH61nzRTvUncqk6noxLSh?= =?us-ascii?Q?f0ewL0mfQ/7nIWwgfy/orX6FXFdFGsecBbgXpNvDcx65epAmr5QFr66LXiuR?= =?us-ascii?Q?MR0AHvAoU0tesXPqZuWyCrkcMKZrV/hIF139yYZFc7oSRLyRVFWKRejgpae1?= =?us-ascii?Q?DdUXNgJURBOiFELoEDk3GHhOVQcNMAQfk95m3YRUzi9ydcg4KZxdua25AMoE?= =?us-ascii?Q?U/4B4XOLw7bYURHr7t6Ee7D1bdbAkR1Tb91NZwNHvKc2xOG7OFatalM4vWGW?= =?us-ascii?Q?fG3LD75SCoZIO6e+6v7ae5NueIZqI1t0nikg0CafDGXEaSzXANLPIuAUfUY2?= =?us-ascii?Q?5SxN55UC62IMlo2wwTpCdrmMZA5lzfZ/1sU1Wn8jqdoW16MDUv+i31AKqCYW?= =?us-ascii?Q?ymI1QQoFOJZ81fhcVT0=3D?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8757db3b-b6f1-4f47-283e-08dd1fe061e6 X-MS-Exchange-CrossTenant-AuthSource: DS7PR12MB9473.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Dec 2024 03:51:13.4269 (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: 2CkXQWtraz4jsJ+2zrbwfmrpny5fz7dn56Fiy49z1D/3Rg7m7XeBuswb59dlEQoW X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB6140 X-Rspamd-Queue-Id: 5DEB8180006 X-Rspamd-Server: rspam12 X-Stat-Signature: k3s5aebtbf876181uwpsf9e866hmhch3 X-Rspam-User: X-HE-Tag: 1734580243-492271 X-HE-Meta: U2FsdGVkX187lUt1cTQu/bz+0cToBU54ry3R49R64ps/v51sWpNqJUFKMmA1KP50k+vmkJBrh5Y7f0jCCh8cEtTQQYvBA7BVHaHND5v2sKWQ5WGyRm8AzDFkQ7QauuBI0Ts/ilPKtFdqilCkCXYC9Q6K02Q1+jQKqBmpDj3pfMAj45uOL39/9l9wajps0C2vZ9gZ/qQL2G3a5bPP5dFFE/Cc4uAd3AqoC7oXzE9AiGDUZoBWvEizmxDRrTBIG6+FNPVwkV1d8waZzJJ3+mUnLYaM5Vp9+zkLReqbwAivxM7JyCu+BbGGfNwNguFohI0p+mx1tmRB/kCStOCaWteFdXMgmDyNCS+4kv8n6DFDTvsdSGGRT/rKCRJPJ8xGhFo597qORL4yL8704eW7HzOgdDO5cYt7ICUyESo/J9ol7sb/GjJtoQXl/P0U2yftr9UHc2xHcM5G9lKTKqBWMyV43G8FQxuC8F02PfnFGAjr8eSFn7snR9AX0laDHe0UrKDwROJTciBYM7sMq7meVGoTsb2TXX3faptUbIw1J2OedOHBtHvctKPWHE4gBluSoRDbFfmswA/BCZJZGZthneaPPMMjdy/eg/mwvP8KBDzr/lA1Eqk+r6Wif8XULAecC8sAZ5C/bykCyO3vj2RMyKK2AcdUzIYETGPwKcnbOF3BmqXMB7OglI0zuhkJ5QeKeKjdZ9qCGgVIJO2il/dG5hlz+6k5emPadq8W+djmOw9K+ziCujQdU3v/oUwKGPgld2pUpPf5vZz5iuoS1XtbsOHWVmoY2G4/p+h4EbZISPCr70vf9veUvOsIhv2BvF5sHuwNZnpBPDnzaIPNnVaMSEDOC08JB2caVNqxLA0TA+Wn9ahl3rsyh0B/KWHvkga1zO0kgJAMhu92YJCmtV3Bvlm4m0ROUSNi/86ALtQCwbjD9wDX1IrHGWKrgKVuTYY2ZNRDB8qMw579LpK4y6iwRBL T8gkovyM BZ/0xnlMUppYPlq0fN04WtLie/QceqaUwTcKMGuoiVgKJRbj+mptp24GZz3/sb/jeXKYYGvcP7+Z5VUtMyAByip1lvM0EQGZTmf1hRvx1XgkcK5VFQ7m8aU3hoH9TUQyeSgPrbiCtePZk6vajwcdhSexe4vLSYtRlNDt2AhHIl7yn4Dnova44Eb+faWOHzB6xhAY757Bq3LWQ3QGhbIHAr5rfzHnGS+SShYlM1DEimhqqDKU6XYlEwrJCyplKlC3/Pa3JxcGie7cZp7IUgPhl7edHskPLreslobkqrcak8fFBNGgriK5ZKEtvWmRTuAsBfpb2Oek6HKG/M6sviorTwxkAre9Xm2xd/wxr 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 18 Dec 2024, at 22:40, John Hubbard wrote: > On 12/18/24 1:34 AM, Dev Jain wrote: >> On 18/12/24 1:06 pm, Ryan Roberts wrote: >>> On 16/12/2024 16:51, Dev Jain wrote: >>>> We may hit a situation wherein we have a larger folio mapped. It is incorrect >>>> to go ahead with the collapse since some pages will be unmapped, leading to >>>> the entire folio getting unmapped. Therefore, skip the corresponding range. > ... >>> It would be good if you can spell out the desired policy when khugepaged hits >>> partially unmapped large folios and unaligned large folios. I think the simple >>> approach is to always collapse them to fully mapped, aligned folios even if the >>> resulting order is smaller than the original. But I'm not sure that's definitely >>> going to always be the best thing. >>> >>> Regardless, I'm struggling to understand the logic in this patch. Taking the >>> order of a folio based on having hit one of it's pages says anything about >>> whether the whole of that folio is mapped or not or it's alignment. And it's not >>> clear to me how we would get to a situation where we are scanning for a lower >>> order and find a (fully mapped, aligned) folio of higher order in the first place. >>> >>> Let's assume the desired policy is that khugepaged should always collapse to >>> naturally aligned large folios. If there happens to be an existing aligned >>> order-4 folio that is fully mapped, we will identify that for collapse as part >>> of the scan for order-4. At that point, we should just notice that it is already >>> an aligned order-4 folio and bypass collapse. Of course we may have already >>> chosen to collapse it into a higher order, but we should definitely not get to a >>> lower order before we notice it. >>> >>> Hmm... I guess if the sysfs thp settings have been changed then things could get >>> spicy... if order-8 was previously enabled and we have an order-8 folio, then it >>> get's disabled and khugepaged is scanning for order-4 (which is still enabled) >>> then hits the order-8; what's the expected policy? rework into 2 order-4 folios >>> or leave it as as single order-8? >> >> Exactly, sorry, I should have made it clear in the patch description that I am >> handling the following scenario: there is a long running system on which we are >> using order-8 folios, and now we decide to downgrade to order-4. Will it be a >> good idea to take the pain of splitting order-8 to 16 order-4 folios? This should >> be a rare situation in the first place, so I have currently decided to ignore the >> folios set up by the previous sysfs setting and only focus on collapsing fresh memory. >> >> Thinking again, a sys-admin deciding to downgrade order of folios, should do that in >> the hopes of reducing internal fragmentation or increasing swap speed etc, so it makes >> sense to shatter large folios....maybe we can have a sysfs tunable for this? > > Maybe we should not support it (at runtime) at all. We are trying to build > systems that don't require incredibly detailed sysadmin involvement, and > this level of tweaking qualifies, thoroughly, as "incredibly detailed > sysadmin micromanagement", imho. I agree. Regarding sysctl thp settings, we probably want to change its meaning from kernel only allows folios with enabled orders to appear in the system to kernel actively creates folios with enabled orders. Otherwise, like Ryan said above, if a sysadmin changes sysctl thp settings from order-8 only to order-4 only, kernel needs to scan all memory to split all order-8 folios? That sounds like madness. -- Best Regards, Yan, Zi