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 C7BFBE7717F for ; Tue, 17 Dec 2024 13:00:21 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 24C9B8D0002; Tue, 17 Dec 2024 08:00:21 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 1FC338D0001; Tue, 17 Dec 2024 08:00:21 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 0031F8D0002; Tue, 17 Dec 2024 08:00:20 -0500 (EST) 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 CC7D78D0001 for ; Tue, 17 Dec 2024 08:00:20 -0500 (EST) Received: from smtpin08.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 73B421A06EF for ; Tue, 17 Dec 2024 13:00:20 +0000 (UTC) X-FDA: 82904457972.08.956C969 Received: from NAM04-DM6-obe.outbound.protection.outlook.com (mail-dm6nam04on2053.outbound.protection.outlook.com [40.107.102.53]) by imf08.hostedemail.com (Postfix) with ESMTP id 5933016000F for ; Tue, 17 Dec 2024 12:59:57 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=U4p3IuiQ; dmarc=pass (policy=reject) header.from=nvidia.com; spf=pass (imf08.hostedemail.com: domain of ziy@nvidia.com designates 40.107.102.53 as permitted sender) smtp.mailfrom=ziy@nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1734440389; a=rsa-sha256; cv=pass; b=mN459SQ91A9mb0SJwX9V8ag/b3UoJb8/2IuWNfpUefR4wnTCXWtFwIC8YYfM38UkZAg/iI iz27vD7UTfW1U57Lu3jHJKr3jQYbujhWOa+bAIwCJLybY7FsBqiUKYNwfLl5nF8Rag8XAF AIh8KIK4zqmdmakK6FWk+gZttjXk624= ARC-Authentication-Results: i=2; imf08.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=U4p3IuiQ; dmarc=pass (policy=reject) header.from=nvidia.com; spf=pass (imf08.hostedemail.com: domain of ziy@nvidia.com designates 40.107.102.53 as permitted sender) smtp.mailfrom=ziy@nvidia.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1734440389; 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=53kY3OVylBGM6aQ/K2P1h4Q0V0LKln3+T8cidjLhYsM=; b=iGIlUePeroOcIrORdE5fMfSBBbbWWvLD+TqVWIEcSNDqU1oYtd0AzL/dBNa3SsHSGcjEL0 xL8MKg8QhizSWlpy4A5Bg7htUGc/zHoWB7owEIDH6P+obpMCGYqHTOkF/s2BvGlm2WnzO4 9X7R9Lthk+o8SQlimPMtqs8yh/jCxqM= ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=QFqTobZ6RzOMn7fuIIZ18Mv/Ipm1EWvl3V2Zvy9KkosCerIhtWoeI6EVRbza6WmdRzjeEwXo8AHZbsNcKfyA/aEnphGAG4u1e2hwc30CdwjWtwXoxjq9ROtMbSQcFWSfT69rJMhR6K9EvF0dNlgulmbjmp6/OHjptl5DyK8aOF0M2DnPz3r9akb/sIZ4L2TyQ0XstGFHGfMoSOILkhpUgKpTZsR/5mTUlKWXOyeOnrz6KyTi1F2i3S3FzyOR5nEtKvAyWq4fW/HinH684ZZXqMOlEKpuU5zaN/AeTHgz9xyJt1m5e4Cy0IGiE/3bDXHZuLkvuT/28HD6jmKtTXIZzQ== 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=53kY3OVylBGM6aQ/K2P1h4Q0V0LKln3+T8cidjLhYsM=; b=Fnl0+VP8/dDMWlXqx0rB3VFcRde2uFK13nEhpHW1jGcWjoQNIUr4mst2IgZzfS3zbVOa7rvT/UbUXMamrB+84lqgUkkFEN8blQ95cW0vyRfj1tZ3+MH7pjhaIDrFsH83gepKN2D29gWXw446A4GUaqQv1VJgpMlFax5C24xI6mLbkry+NLWt4VFRTq8l8fO4PmOSXLHTp4anfSSDA7KDooEQxeW3Ik7j6tStBuzAEgtnht8r+d37Tk4KtxkrefD1f0g+3D0kb+IXm18QFgC2hP/lVm7Qr5JGvrDCTMAuPD/qmcga9NzphrdL8KNXIMIydZ5LT7MspAZ0OJQjt21eGA== 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=53kY3OVylBGM6aQ/K2P1h4Q0V0LKln3+T8cidjLhYsM=; b=U4p3IuiQwpXPf5taUbj1cqnF3G3LK36mlIHJmnEYx8cj5fP5woNVw3TrQT+WCby/Ufkv+cE55WOMB2EW/7TIyieqpycpYfz1uourzGPpR0k7Dew6XYuvhgi+MnvCwxTLlTNZpNkg2beND9L2xI93u7dikeDQfxwWG8qcUlTKLlpH8uvPMRapIVPgl1KZfLG12nWvhpyd7xDegw+PxLYBFOpFg16SYw++6wAQJfOxmBlbFYQ/BIOlrySpzbGEuby+uJhHkwwrkgNIhxnW8b5lt+80bKhSbgPhLYjHaNpgjFl8/r6zFlDTpHxlDtWrY6T3G2Vhvv2zvnx59QpgxpRo1A== Received: from DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) by DM6PR12MB4355.namprd12.prod.outlook.com (2603:10b6:5:2a3::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8207.19; Tue, 17 Dec 2024 13:00: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.8251.015; Tue, 17 Dec 2024 13:00:13 +0000 From: Zi Yan To: Ryan Roberts Cc: Matthew Wilcox , Dev Jain , akpm@linux-foundation.org, david@redhat.com, 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, jhubbard@nvidia.com, 21cnbao@gmail.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH 02/12] khugepaged: Generalize alloc_charge_folio() Date: Tue, 17 Dec 2024 08:00:07 -0500 X-Mailer: MailMate (1.14r6065) Message-ID: In-Reply-To: References: <20241216165105.56185-1-dev.jain@arm.com> <20241216165105.56185-3-dev.jain@arm.com> Content-Type: text/plain X-ClientProxiedBy: BL1PR13CA0157.namprd13.prod.outlook.com (2603:10b6:208:2bd::12) To DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS7PR12MB9473:EE_|DM6PR12MB4355:EE_ X-MS-Office365-Filtering-Correlation-Id: 6dea9d65-260c-4c1c-34f0-08dd1e9abee8 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|366016|1800799024; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?QWRjlcTB8d1G8Fd5cbAzGbC3Bc7UStudMHG8nItSSMR/39Dvp+c5xOzLT61z?= =?us-ascii?Q?+EzmhKoSrT22v+dNaeRjlGXWvcFB54+gNcUUWNVJD3E+VkSK5Rdn+3xor4LF?= =?us-ascii?Q?++S1LMmLIdTx3UNETPQyG+RlmaS3yEj7u1O18undjcC0EkqHfn5G/W2BQRvU?= =?us-ascii?Q?qRbHkScT0EgvOQZE83LRWSAcdtCzvo+RGgddj45kmm44HsqjwvGeH+68fcwM?= =?us-ascii?Q?3TGMDoE4Qp3Z69C3GPNoH3f3+xxuHrek0iFBFBp70DTfamSSeLM/Iup+iiO0?= =?us-ascii?Q?AQeebaP394Xqf7MFZHsHHHQmwduyUpmkjBf/JTklpytQTyWFherzGVtpwBRl?= =?us-ascii?Q?AWAp8xL3uS0XMPn+QqXp11QZqh5J46fpbgwl3g3UjCepGD1l+wDByuxna4xX?= =?us-ascii?Q?1WJzmStxu5J9Pm2IlGTHOJSqSm4C2nsKXoHr9smjk7wpzFoA9lO5yYQtFaa+?= =?us-ascii?Q?7LIlXyjl/hjaER00Ja3mU7ovmlaf0qEr81fiAtelPGVgMOwZo/osCqCCiImq?= =?us-ascii?Q?dvz5iCe9Xxka+dZjdaxyxn4B3ou6TcspSkxslGpY07pyYUeOphHs1DQXZDq5?= =?us-ascii?Q?4Wh2Jlbwca9i8tEfnQRS+NwmVQ1F8YfgJZ729NRiyTFeX8rKN0tox1rYxRVW?= =?us-ascii?Q?Rik2qGPdvjtV7ASmitZMemCzLe3ByJBJt5/ANbd+HSOyevJrN2A9xVjhvePM?= =?us-ascii?Q?/SPJCOqlWPjIjm0erc7S/LTYCKd2YJ1yUThx4v7ujJ4QgK/WoIkjhrbEx5X1?= =?us-ascii?Q?d4DA26b7IWX7SAlM5BGBFX3iltfWWoILxpbsp6WUMn4lcAkV2XtGaKYOQV6L?= =?us-ascii?Q?Tu7+D5KFK4HrkVhckfwYMkJZ3UukGVQ7HKcf9Mu3o9yDpM5xcovaZC/q6HZD?= =?us-ascii?Q?NlBGO7V+15Sm86s1+lVOyFMZ4RizvjoVmjGww//K82zvCvh/y0PDt5W3Y3r3?= =?us-ascii?Q?vn9MHBmwbOQ9lY5pv5Tt/B/2/iqxuE8uANoxxNWgCdWxQRUTIVQAJ/WXrq1W?= =?us-ascii?Q?F+ZcMw+fyImym/7V2G/vDukzs8iklT09ranUOQccuSErqTCuSBGfRbFk/OWD?= =?us-ascii?Q?XuEdvkJyntyF1+8CBFrlXOinQEMbktW/RPEblEB8mB2n+AK/ygRXfL5CAz6H?= =?us-ascii?Q?JwpoHdF0iEr6utc9sSlpW7Ifu5A5R9GwzLDGcjRZ5cQFlTYsdCb7GEIqpqHS?= =?us-ascii?Q?UrYn+6tquLicRPN3Y9VX7sFJGKc1+3IBE2DdKZJ/FC+08TR5+vp5LqLhwovq?= =?us-ascii?Q?5Yuw7+8EeX2IQDnLELRaqLUv2NS1qWnIk+wQIapV5tUzvRdAN+g+KuoTp/Zt?= =?us-ascii?Q?5eIioqatCuqlfvuXeePVmQZOxH9nZ82OhIs4dr1Hv+CFvpbKANfRrOtAr2xR?= =?us-ascii?Q?SfvL1/XnH9Y7Snr6/c3v7YDw3LS5?= 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)(376014)(7416014)(366016)(1800799024);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?vSpuhvM2JSW/6vu9MU0dfuM2fNh61OZtULSZwvU8dGIy0EL9lPMb/s5PKb3N?= =?us-ascii?Q?ku79CyoRCCApGHqT0Er0DY1wzi1/cqqlwBuQ3hh2UCOT1qDfAhUPG8pEeUyu?= =?us-ascii?Q?Cl+0o299MwMkzzGUQ1swYHE9UBc6/hALWaRuLA2WOW900Rp9mhDfDegAz0MN?= =?us-ascii?Q?ZpKeuRh3M/1ZZfWdhTWfq7Emgja6ynVmPpd4C/RrJF3GPFAQGT9AWBR0mE5Y?= =?us-ascii?Q?1pbTyBTK4ho1lyOjNUp4OpgQnh4IHQYM7doob0P6ZGLXcY9294aMtoSeW6ac?= =?us-ascii?Q?1mjcjYL6k8wQMlmxgjCGhZgYQjA1IVwmnXnV6Qbifpxnq2zXdX1E1sdUxV7y?= =?us-ascii?Q?Vs7AvUm7qmV6OsuTIvF7t0GuLgF0EiBzGvFJqXWUka/06I/6DpUlVvVMKGsy?= =?us-ascii?Q?KcD7T4PWUYjDBpQj3Fr+YSE/jxLS5bxJseiQTF9arioVmFh3X9ovI4chh3W0?= =?us-ascii?Q?rRP+Oe8i6uEHLTs7mXJh/FWWZDP6IZniN8FWXNAGY2/LIUJHWDxuW4WrIJij?= =?us-ascii?Q?BJiS80MJGNmKJ61UwuuW4h9pYx/MVDFFYZl0MslQs19VMZ/0A7D40lEdKItl?= =?us-ascii?Q?mjkGh/+XtO2J574TuhOJsDpDh2O6FZgLlv5B276Tu3G/jfeP/ODOLSxYbgyD?= =?us-ascii?Q?6OBab3wkqkQZw2HQH1H7PXp+g01O4vM2NT5cwxlLL/Lw3XWZgxdWISF9xu0H?= =?us-ascii?Q?TQskfXMGx7JoqHlALsud7wwGzG/JczeFc74m1B5UFKrk0GwRaDQE/XHgg8Vr?= =?us-ascii?Q?LkUe/S5UT7Ih1UaaGxM/pU0MbOusB5QnsayvXlAF8fvkLw1pOvBwoz68C8bz?= =?us-ascii?Q?uLJWaHUioPqxLoPhM9oU9nxFqA62hDGGsFPAJQF1w5Ca3KbMETX5gexwZihc?= =?us-ascii?Q?5pPUdcgp7hQQZ/fKoqqVp98bnyGIxJdSbskyxuZbaziCbZCTseccByWyVLBG?= =?us-ascii?Q?FbHC/P40KW0NnH3rPv1IEizHGUYvFewBFiwbStzg56/HOXVCT1ilae+79a6D?= =?us-ascii?Q?cAaS52WeXC2SDVHDltp+yS+4ZaE73Upc6Aq7aFd9aB+EmzUkBf8e1uKBAWEd?= =?us-ascii?Q?MdnxXYPw3XfrQQUt1Qi3hFuKNss5R2d0nLOq5KdQjNdcuFJclhSIsl2GwwaQ?= =?us-ascii?Q?h3wO1Y/TsL3WK3N1xvtC4/cJcQudl7mnxNn9AmeD7UvwNdHqk17Lb/wJEC8/?= =?us-ascii?Q?TCktie884aQsIU0nqdwn1fthAK6xxecVk0Im6JuJcn+w4772WRFDc/bhdUDI?= =?us-ascii?Q?ktIIyF/P0+OVQfavc6lzTM6l0NP7SwfyFkTrLXUFk/5PUKKabQyQIHGO4auJ?= =?us-ascii?Q?a3JVF9cFp5EXfzrec5MbbCHr9xWayTRlfBhNbFpJ2kY1Z3vvTC3iemqg3IOB?= =?us-ascii?Q?LVChalEY4EZ3hYuBLu0sl/7YFmK3NPgf/mNxMgDp14nAjm9Q1a9TV7Ksfon0?= =?us-ascii?Q?DLQVvu5vMXc5JDdDjIj/sprooT8cMu8mQlhY6lxSLpPf0BxdCkv5+aT5QRuS?= =?us-ascii?Q?pZ0oBelqeJPDR2k7v2RstzBa8pA8csbgpbOX8+yIeHWz4rYA5oRLXLkkso+8?= =?us-ascii?Q?b3R2oVu2DhC0le2ufn4=3D?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6dea9d65-260c-4c1c-34f0-08dd1e9abee8 X-MS-Exchange-CrossTenant-AuthSource: DS7PR12MB9473.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Dec 2024 13:00:13.4393 (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: nWJEW05ExCLiUxTg0ZFSfAX+karRR5zGiXDtZgjAy6u4UiLg1I6hBp9qdNLfQSbk X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB4355 X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 5933016000F X-Stat-Signature: 1i8o1sckee18gi5ygb4q7k19cfcydfza X-Rspam-User: X-HE-Tag: 1734440397-845516 X-HE-Meta: U2FsdGVkX19nw9o3/pb6bfj1yD0hmNxUZLVP/JFNVMp4eFl3NFnV2mtQM1Ys7RPcZ3iSDWoVCj0bYFo5z8/0b0Jb/FGgE8G+QwbKvTQU2LI7Y/SYJQAHfj7cMnJd01euEwouf19KmilAdY81Lvev178VwToi/L1vNKNjGezTcoSCsYFYMNJ8JSJCy0u5d6jQNSOED1jUdGCrzC85MS414yjGKPSgiOorViV56dnSYd7jz6MqO+h/bSoTSqobob11eS5VNBlIvTT4bQCbORL3VdEb3GxBDwtmp/wVSQgD1k/iI4YZmmjRR9A4yAqCMzV0RW0GVmObyA9qHlhaXAVWQAdduHsCaSEvTh/LpLsUGU/XrlxGbztgGrdVzRYBSpaCvToQvjB3iXv5HVTYVQkcYVqZQ26WSOvBZiMcKkgBwXx8r686TyYS4t0I39K1dX5FJ4Fz7wzOJ/XskutcqE3RWbnVAHHCpC2qZEZ0eNUZPVi+Ee2354vCKnFfvuox7+DHbFsn0wvkvbHNTU9wWUmezK6+8yF5KBH+9URwjZH/WJ9z0FrofY+EWMdeIhF1FNj0LtDIJtT5CR98yOd4FrF/wu/ygK3bdeVZ2Uo+m2ftk1pcJbdpeQhQJ6qVzMUEGdS8giZDfiSDRSGWX9trC+LVuOkzJMtIp0TY6hyOjw+DZkq54j4FT8qxpydQuOh+rle4Hdfkd0QlVgsO5f+KA7GCXBwVAPUfJ+wiBl8RQvb+FDDZ2xjgfmGF17OGFLWMp03vYvKgiJTZk7VY2pgoW1Scr/l00vbjwmlBmwKQIzqnZHh+ABNUu1spjV+rerB+Q+2A3+jY1bILjPP/fAB2n6JkYjjpk0BMnvEWTEmP0Zhwr3nxVESjonfMpE93SkC3R6qS2i0fQkdA6Qu0mXNGiUHpMSF8uszwFBb4J8EEr+rMtK26ZwXPQByx54nxanQTR+aKrTc2nek+T5+sujRbFjh 9O+7Io2s 0uo+07iUo/yhWZWoHiCJG8wgYvAeaQMGeGPYHbV2TTdbZnvc/No5XLCkcIQymZoV1hYOpYdvcoveZivMfeCnUZ2c/rZMMsWfurtxz+EjXpQdRIEg88nF+/SCbEo9nso3pipGhSUED7yhtP9h39COeqsKssNk3JLG8PIhG6dZO0dXRccE5v13iwXm64cGV4AiAKooHpAMwJShMkfQrN6rmYUGUfJDPvCFqZf3UDzQSkU8vwafC6f0MfJVCz1rfVELbw+tuSgKuilb0ZlFtua2cXgAmEspt3pmnPsEt39GkmmqGKWLpWniRhMTK5fR17yB9C1NO6zePsnYka3YbO/RfrydgDX+2m7IZpUlp97X/IjnOelvI1X9cv9u79ClB5Ka1VLqb9fk022CZ1+s= 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 Dec 2024, at 2:09, Ryan Roberts wrote: > On 17/12/2024 04:17, Matthew Wilcox wrote: >> On Mon, Dec 16, 2024 at 10:20:55PM +0530, Dev Jain wrote: >>> static int alloc_charge_folio(struct folio **foliop, struct mm_struct *mm, >>> - struct collapse_control *cc) >>> + int order, struct collapse_control *cc) >> >> unsigned, surely? > > I'm obviously feeling argumentative this morning... > > There are plenty of examples of order being signed and unsigned in the code > base... it's a mess. Certainly the mTHP changes up to now have opted for > (signed) int. And get_order(), which I would assume to the authority, returns > (signed) int. > > Personally I prefer int for small positive integers; it's more compact. But if > we're trying to establish a pattern to use unsigned int for all new uses of > order, that fine too, let's just document it somewhere? If unsigned is used, I wonder how to handle for (unsigned order = 9; order >= 0; order--) case. We will need a signed order to make this work, right? -- Best Regards, Yan, Zi