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 BA0B6C2D0CD for ; Sun, 18 May 2025 00:07:39 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 1C6556B0082; Sat, 17 May 2025 20:07:38 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 176B26B0083; Sat, 17 May 2025 20:07:38 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F30196B0085; Sat, 17 May 2025 20:07:37 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id D619D6B0082 for ; Sat, 17 May 2025 20:07:37 -0400 (EDT) Received: from smtpin15.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id A08E2141701 for ; Sun, 18 May 2025 00:07:37 +0000 (UTC) X-FDA: 83454089754.15.E04B1DA Received: from NAM12-MW2-obe.outbound.protection.outlook.com (mail-mw2nam12on2073.outbound.protection.outlook.com [40.107.244.73]) by imf14.hostedemail.com (Postfix) with ESMTP id CD8E0100007 for ; Sun, 18 May 2025 00:07:33 +0000 (UTC) Authentication-Results: imf14.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=qo1ypBbU; dmarc=pass (policy=reject) header.from=nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf14.hostedemail.com: domain of ziy@nvidia.com designates 40.107.244.73 as permitted sender) smtp.mailfrom=ziy@nvidia.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1747526854; a=rsa-sha256; cv=pass; b=LpnuByx4ItarWxNh4G7Dz6ySYSwgz1vbBF+zkDsc/VMAA1QexQed7yNj5SmIJLt9L4BCrz fM3UCdfObcbulUZ0+4ERPYRVIr8pwn2A6T8nNKnOaHxdO4oCTGp8HStvr6atiVJ0hdTq/m t0qP3LZhpPMW7s/RgxiC5jCkrKIkPc8= ARC-Authentication-Results: i=2; imf14.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=qo1ypBbU; dmarc=pass (policy=reject) header.from=nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf14.hostedemail.com: domain of ziy@nvidia.com designates 40.107.244.73 as permitted sender) smtp.mailfrom=ziy@nvidia.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1747526854; 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=epgR2MFJIEV+dv31BTFylVdT7KDP5m6rROfhIdvNLZ4=; b=8hQtq2o41QjxzESFOq26xGC9dFcjpkyV1PVe6hQXDtfe7e42iKJvipg1O1xyFLXl2bGr2s DHn/ZzaSddhsNwF8C7GzefaV94f/SP2z6bQc5dCDR/AlR0RYHaSdVwxgmVCnu051ZVBOMd TXpdvcIi2fxNbiCgJ9fgGZswDt5MER0= ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=hP0K8ZiXJnTlwcaVjyXVwAbxjtE9mUjCvXJ+ryltTb/xw6yEZFa08dQy7Ihh+DYpINC27i82WjyQQo64NrKk86CFJXhGQMKVXE/Olqhem8KIbCR6blL2ca66Xdv+fxfX5wqfVHoo/awuPQxvQWO94lDiVrzjEDjA7Hw7hwG0YaPePqVo2mhnMU/KV8TNhORF4ehatsqXcwNE56S/SWw1U0jg4CjztmVNjaLyQBx2+uSbkHdpcadnZRU3WXVFQ+B2H0jUjnC3Gn/TNLjb/DECEWzMsRrpwVIGfM1wgRzQkNks8qkwcdQG3YASB+cw/6/mLz3Ikx5ZoW62waY7lNtZYg== 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=epgR2MFJIEV+dv31BTFylVdT7KDP5m6rROfhIdvNLZ4=; b=kW8h7NHqCASIRXDwry6f++DxrDfn8zJPnWX1XbyTTmNt8VfiC2dmI+J2+D2Bhr+ifBfO6TmjjyFQACfWeONA1Evg+hawkqx/+VSJ5+Wl1bdLVXV7vukwFzLaKQJa6eTLXzLVTTBkZi923ub1QdDR4aDCP2RE2H/3cqj0QdFXKVpyrNcXfkaBZAYAV5XQJ7WFBPePeIqgDhxl7L43qlQ9xgobJbIXdBIWqlUInTVx0XddcoxzHPFTNrUKlIQhWshU/nqKrq1aPApUnCDww/wUsVm0lOkDwvJRepGvdTaq9GyXr/DIIyGYosrNgFcmmNm3us4dhP5glEedbrBD0+WogQ== 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=epgR2MFJIEV+dv31BTFylVdT7KDP5m6rROfhIdvNLZ4=; b=qo1ypBbUbEeXSLbX/EaQEBcIvaCjUbqzUYgHV35g0rU/RjFaEQzCUT9J4jhOIyyr8Ju8uxaqlBfnPgm03EC+FYlOhPV7kk2tagKMZpapvFVypDFX2WfAxK1q54l70wVbhqlx4Wi0bfE3ms5Kkx2oRjXXWVuD7jrnd7CA8strvit9ceYZzCIk+j5CMvQymQxQiFeKBNJn3sqVCuXQ/Xr7q+r8Zp32YZzveAfAm/aJ6HxVTVfXilXiO4vGe46lkzHocdf+uuUDVV2HftBFwo0DpZNU9DKmgtAXrZlYkJk4NIeMWowOZFQYrDxoC4aPyDte90E1TCQTNU8vRA5qSj2FQQ== Received: from DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) by DS7PR12MB8276.namprd12.prod.outlook.com (2603:10b6:8:da::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8722.33; Sun, 18 May 2025 00:07:30 +0000 Received: from DS7PR12MB9473.namprd12.prod.outlook.com ([fe80::5189:ecec:d84a:133a]) by DS7PR12MB9473.namprd12.prod.outlook.com ([fe80::5189:ecec:d84a:133a%5]) with mapi id 15.20.8722.031; Sun, 18 May 2025 00:07:30 +0000 From: Zi Yan To: Vlastimil Babka Cc: David Hildenbrand , Oscar Salvador , Johannes Weiner , linux-mm@kvack.org, Andrew Morton , Baolin Wang , "Kirill A . Shutemov" , Mel Gorman , Suren Baghdasaryan , Michal Hocko , Brendan Jackman , Richard Chang , linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 4/4] mm/page_isolation: remove migratetype parameter from more functions. Date: Sat, 17 May 2025 20:07:28 -0400 X-Mailer: MailMate (2.0r6255) Message-ID: In-Reply-To: <72e15406-ffd2-4344-8cc6-1d54005138ce@suse.cz> References: <20250509200111.3372279-1-ziy@nvidia.com> <20250509200111.3372279-5-ziy@nvidia.com> <72e15406-ffd2-4344-8cc6-1d54005138ce@suse.cz> Content-Type: text/plain X-ClientProxiedBy: MN2PR19CA0040.namprd19.prod.outlook.com (2603:10b6:208:19b::17) To DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS7PR12MB9473:EE_|DS7PR12MB8276:EE_ X-MS-Office365-Filtering-Correlation-Id: 84d47310-4c5e-41d2-b976-08dd959ffb29 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|7416014|376014|1800799024|366016; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?x0wdhmrFAe/8FGLbHgX3RAGvHMzS6nh+kQHHvsX2ogRLsUx0fbYLGRU81RUa?= =?us-ascii?Q?uRqRWKLzmf7i1eG1hSckkG5kfslWgHFe287tp0YyxsHgr1Ru+rMGDiP7lQX6?= =?us-ascii?Q?P/QRjb8eG7zEEmzcQolqi7kLDEf/9SjBINowj0NordFr17Rw8U+CLqD4Od0x?= =?us-ascii?Q?gYLufbMdKALmRtbLe0xSsIzRAs0RiCaTslBbaQKr8HZExpdQySSqki+WGwby?= =?us-ascii?Q?tdc1JlIyGgsfxY3ji8/v1IsiN3W30FzJa9XycdJKqlfe376OVKVJIS5onnGd?= =?us-ascii?Q?mjBUDDo8BG2RKETbcwYatmTRa+NehtXEMEmayvW8fX9Nqi1sWW4DM+zAbW7u?= =?us-ascii?Q?qib4XZtKgmrpK7TlRUo1crDXaq53Fd47N7bN1US03y0Kh0sbLm0oJwZCWJlX?= =?us-ascii?Q?htFHPyA7FosfLVC//U20pv1jHnfm0QGwqtUu5NuQQgta8OauKkfshG7dp5gg?= =?us-ascii?Q?dz6FG+TC/9mWIz63DmMOXjaS7YpmSfOG6QvWsGpRvUuy1z8dkMWWwFjTJyQ8?= =?us-ascii?Q?XUM3GXiLcNL5Tns24K6vpcGyk8djlGFpgXxxrqya8VX9T/RN0OunQ6Fh+lcr?= =?us-ascii?Q?F5+0TiAHgMyLJMv26+CNR46s3a+P4ZpUuFlfqa3HpKFmdKU66km1DN8UtofJ?= =?us-ascii?Q?oCDnKZSlIhbqD5ol2i06Qxfqske+WEpxaC300Km8HdVeC8rwIbZMAxjc9cbc?= =?us-ascii?Q?X64qp5EqSK+TYAediUpDfHWmcJi5cX3fznkjJb1Yztpg0gWHXtYE+LeBHsW+?= =?us-ascii?Q?Z7RM4PweUR2GFBrC/E/EwVh+qc0lWAV1tu/49o7DwzE6xLL5KOCe9iIwC1Qd?= =?us-ascii?Q?ovv6ndu8YMXhAYfV9RoD1Bea3+9jk1pikcd/5LOIeT5ERMP/HQOa5AIQlXzY?= =?us-ascii?Q?EBS1pa1trXjlb5GQfi1LcxRznGoRGUvvR81IaKJ4ztuWtTg5Fu2RMF4iwybL?= =?us-ascii?Q?AlV9ip/Y0/5M2Zrw2P69ZavCVQjmiVcnM7DuAmjtLfFXGOFbHnbnUSxF04iY?= =?us-ascii?Q?hhqDUlWq/h4ZKadTqpQOEfvtwbl7jg8r/i8hkE1Cpm+YY/ij2Qu7V6FoP6Z/?= =?us-ascii?Q?U1MZrZBJPiFurZm22lQNM9ennGvfHb4B/x2p6eVaR2W6mbGS16ANFTKVd3Oi?= =?us-ascii?Q?tqF1KFjR19xx3i3m9DEj12QnfTSMq2QVk5rIj8UDUfBEzg72h42ESYbzFKvi?= =?us-ascii?Q?27fLZnu5YU/rnX6MsunLPi0SOfB0QHsN/m3yJLKEJTsufRPfyTI7G4KQFdnJ?= =?us-ascii?Q?ZEnqwtYUebb5UY8H5N6Hf/L9jMjY4QZidv85Po9JUhn3ccmCfqblGgSoU3ww?= =?us-ascii?Q?xqQ27vM7By9oM9gaPCPEgFErXYBzrtk0OC0wGj1e2lzSayT+VYoxqqRTbWRc?= =?us-ascii?Q?+BLXhMXQ5TkDAaMK5o2gmWlW2ZCNC87vERxQv1ygSKRvSI6Rmw=3D=3D?= 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)(7416014)(376014)(1800799024)(366016);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?7Wh1mpoXo1R1+FunYf059V7Jr1Tw7uwEMEbPDSAC7D6HCuuzTTe2gk2l9z+L?= =?us-ascii?Q?kJF/PHNa30eWYlXfhMu+n6Syq8qJcfnmztgU78WwZXrybKOG1Wppk09JRQ8A?= =?us-ascii?Q?eC7tC2AimgsowqFnKp3BMvlLUu2X/2+X9R1Kh5I3Bne87Q/0jhqjhi3pzqAj?= =?us-ascii?Q?0oaMpM7N+uRuEUoNBqmd1FtgoMsRziZZhWWafpkovrN3jf/yjeSb3I0qKWQz?= =?us-ascii?Q?6dNiUxCiyIpjKtOGJWfTNn+FHLnAy1uMfbNYEO882hhzKegzOAhDK4RN0DOQ?= =?us-ascii?Q?0y9ioJOVXHJe5A475rIf/GJDa+AnXtJK7nmJiBOs5eRBS1NiD0GXI1W5X1yR?= =?us-ascii?Q?fSg/prdy8vzbUUCyQ/xqZ4iP08WbORJxgZclhPH5ShiuyUU99IeMHQFM1ipr?= =?us-ascii?Q?434G09yCq4g+f2I/CJliUMh6fqt+fgONU+Fo5TLaFpfA0/Vv7oyr9bvScg4h?= =?us-ascii?Q?m5JP/+hD13/exVg+O/XS7y44DDPdYe2uFv8Pt5DmCxeMvF18SL/k9VIBfL0L?= =?us-ascii?Q?LM4dd0GJt5a/oLiENg2ZRp4cEqSXhLy1a7E+Ex6/xiNJndor/d8Q5YHu38MF?= =?us-ascii?Q?kdRvXnJqXwucsXiEMziZ6FM23rTn1PXd8bltib38Gt0V63p+VRcNytY8C48N?= =?us-ascii?Q?4GUxthRa27XeFxVhf4PVDr34wscm0kaMt0qUabOI4EhkYRb3Nweac+QgHC73?= =?us-ascii?Q?5VlYmSECWRepFxOKBJGbPA1nRFH/e8ioyM8QT4ZDn61jrD1dfJNjW095FK+1?= =?us-ascii?Q?tD4TobdtrBezFjRofa+zROffRhjVZuJJOXCNXnW+9cBiiE3Irz0t9MQg0Dr+?= =?us-ascii?Q?a+cA5p9PXJ5BjAlXRLSnTqVd7GRV/4+cW+yREhomH7xZGLJvkKcMUxaUZr3O?= =?us-ascii?Q?tC+FPqu3YaQR2HvujWgEA9++tHwMbNBS/z1RJCOGvKnXsqZfnzOop6hkywtv?= =?us-ascii?Q?Az8a8pgRFPoFxvh3r6+ZiQAuySwuwBdFmJNHU0nYrJEPhaVRXV9BhKM9KMg0?= =?us-ascii?Q?/m2uE9RaYL+RTnBCJWAPUt7HjJ3tHQbPFIJPycVsSnD9Ay5ZS1SUVs6SMS2+?= =?us-ascii?Q?f2VSXdEN3Z9YZlsaYV0KWTvKpoFHjf+0Wgc3PQowYCLoz7w4xoI00tLZo1yd?= =?us-ascii?Q?Eq1zY5fPqb7KPx2D3Pftqcj/yHufpKLtS9uHEkm3uwRiKjPEKlyUIsYUEbl5?= =?us-ascii?Q?YPd+17Ppy1EeZahBz5ae3Z/33eDAaIpyrNxU96Q+p+R4MewCZH8ur9jrHn6q?= =?us-ascii?Q?qm2yDTBzeIAlir3nSi8GWtdrHiuAsF6fsJhw4EzVZgIavPWCp5cHeyAEYpBo?= =?us-ascii?Q?aki8QmpYAnG4BsZSuHZAenE8M8Yi3iD/2XAR2nvlZ5/Hr8nm9NF5KocLyMZ0?= =?us-ascii?Q?46iQhqZWAO1HM9NFq44sOrNnRoyFbUdUjPaLULTzzoNl6+RRPWNqVMz1q79S?= =?us-ascii?Q?Y3+sW+KFKvTIWNGM5ATGuoI1mFy/MS8vccCi6sbSwMF/0ZSNCSOArDXEJli3?= =?us-ascii?Q?yJWsHAzDyftQk+3kzsRrv3/2Rkh4VTem9Z8Ds95FvdYCowXubt9esIi8rbvv?= =?us-ascii?Q?rwFjMVQDmTh2kjO8PxZOg90tWu7AMz3osFk97Ejf?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 84d47310-4c5e-41d2-b976-08dd959ffb29 X-MS-Exchange-CrossTenant-AuthSource: DS7PR12MB9473.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 May 2025 00:07:30.3727 (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: kdyx58df42WkL+mXwYOJwRd6rX60HiTT0tuNxF3WxLe+K7M/CHB4L/G/wjb5nM9b X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB8276 X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: CD8E0100007 X-Rspam-User: X-Stat-Signature: kxx9te4qeftnoxa5kzhr5c4n4teszfr8 X-HE-Tag: 1747526853-416563 X-HE-Meta: U2FsdGVkX18olUkjzShUkrEvwFzVzQYdccsO3Lpp3Gg48p3Q1u270h96RfSUfCbiatQhomRNEzq9LFp9uaYwJ9TGu/dYmYM6T59wsgiWnn65A8+BGh48PauOJ5OdLRKEx02aMwzhGVEE06FCTRa80J9BmxpEBja3dvxYE12CDnofYIsfwx5dogiC27ZqjxZ08vupep62Dqa2zs5F/CXlVkU/+qwstkM8UWEsbphN26dlUyWZMFQyxv8g6n+paPBasGLJNuRF5RkSGfof+XttmeLvilJ9A15/nBmwY28/i4JgGsn928ntZ/P7FN6e9LPIlPow7aUwewGoNIsXqjI+c5L7WEEvox3/6dHFuERf+BSRwgoNBhD7JDrrnFzi+KlQg2hTYBgPjeY/4jKH1YpdmY9Vn7X+i7kWssVGIQsLr3vKiPN7fCklKbWUtS2WfEo4yyF+9N9BWbw+YLJBJyGXiZv/OHfh28TsqPbUZEOMZnxXfVFEm/JvnYJRD/mytbSZCKRchJQ2RD5fFe1gUTcdIJGY0gB30oyWlzD09NFV7KoJbnUCGK0sGgUpRzcbSQyLH+BHfGpkb4LGvgDryLOkmA5mioVNYrTE2O+Y7lvcyB6E+OpFQ1dw8wL3um7hmNSpqrK+llvBxcps5PMzkq2MFSR5nIkhbEhCTvBnXtsXK4EEpRssvq8naUFl10DHaSEHPf23Qjqo/RaCYXwy7UeVPZ7Mhv3yyDO1rD3BNdFxI3n8jV9fVsu1xbk2vQxVd98POlXnt3pqx5VOheGiZcO8fYastzwCXEozP0UMSOOWmxM0SrO7nPJ1M2rO0G8LJtX+++jsm+8bCEZfXw9Kr6Hp9nrWyvs4mDtTb23dSMgGdetEnk+lLH7mwz9SJM3MY9ehmtETLWJ7yOX8CVjIkenL8RE73Q1tgIDCd4sdOa+uMntle+1ooA1IJ8eZM4J9yKPojbXQHYdbmsMKWZGwK1B BQbBI4SG gSgS1PB0U7cDKRCnqlVIxy2nvq5lZ8kesMUUgGVcw/RF2yPNYF+ntZA+bjgky0iOOOZEbfyJo2gHUN6Et1OnbuD2V9RyxNSt1UjNF0d4l35Wp4ErCjHh6hK2W2wL7swNiVi52kHwU5lPrC64DSg7TX0Qi/+lugK34wo8W+yuTbdA1Jcj4rtF8BH9JDzJ1Qj4obOBvIuSmgMBaY+v43lpknXdDTJL7xxQBs12Wckm3zpjwWqdg7yaVKWUb7mnq+WcWau9pw9laGb/XK302WABZc5YWs9RPLnWW2B9VfwV/t8VjF0o+qUVzXg06ccDa1uLYnn9iEIOh3yh8Ae1I7443J03UNj5E7ety3RGgA+PdslWtjmw= 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 17 May 2025, at 16:21, Vlastimil Babka wrote: > On 5/9/25 22:01, Zi Yan wrote: >> migratetype is no longer overwritten during pageblock isolation, >> start_isolate_page_range(), has_unmovable_pages(), and >> set_migratetype_isolate() no longer need which migratetype to restore >> during isolation failure. >> >> For has_unmoable_pages(), it needs to know if the isolation is for CMA >> allocation, so adding CMA_ALLOCATION to isolation flags to provide the >> information. >> >> alloc_contig_range() no longer needs migratetype. Replace it with >> a newly defined acr_flags_t to tell if an allocation is for CMA. So does >> __alloc_contig_migrate_range(). >> >> Signed-off-by: Zi Yan > > AFAICS has_unmovable_pages() adds the flags parameter but doesn't use it. Yes, will remove it. > > But also, I think having both mode and flags is just unnecessary complexity > in this case? CMA_ALLOCATION could be just a new flag? Even if some flag > combinations wouldn't logicaly make sense, this has only so few users so we > don't have to care to make them exclusive with the mode thing. I was doing that until v3. > Also I think REPORT_FAILURE is only used with MEMORY_OFFLINE so it could be > squashed? Yes, let me do that. Johannes also pointed this out but I missed it. In the next version, I will remove REPORT_FAILURE as it is implied by MEMORY_OFFLINE, including isolate_flags_t, and keep the existing enum with MEMORY_OFFLINE and CMA_ALLOCATION. Thanks for the review. -- Best Regards, Yan, Zi