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 4B3E6C52D7C for ; Fri, 9 Aug 2024 14:16:53 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id C93806B0089; Fri, 9 Aug 2024 10:16:52 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id C43406B008A; Fri, 9 Aug 2024 10:16:52 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id A95D86B008C; Fri, 9 Aug 2024 10:16:52 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 89F476B0089 for ; Fri, 9 Aug 2024 10:16:52 -0400 (EDT) Received: from smtpin24.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id DB48616114E for ; Fri, 9 Aug 2024 14:16:51 +0000 (UTC) X-FDA: 82432908222.24.3B25CC3 Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) by imf16.hostedemail.com (Postfix) with ESMTP id 680CA180026 for ; Fri, 9 Aug 2024 14:16:47 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=SuCEY193; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=ixcysoT4; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf16.hostedemail.com: domain of liam.howlett@oracle.com designates 205.220.165.32 as permitted sender) smtp.mailfrom=liam.howlett@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=1723212941; 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=k9bFMrwLQUGnNHgd2Ns4lQcnlRYPWDUhKpZAqaU1a9Q=; b=J7fJ22ldBCIyv3yWxe7x0ba0s95m9ahkDz7x3BI5rw+q3jQEBb3y1RrOGS9PZ8CNVM/2dH zW+CfdiWgDm3JnVqCvqgRs0caojjyL+TCKC2KuLuLEH1KbdS1NpVLSjxzqjX4gxTr0NDz0 V0qLD2/kcc+KopLwFr3x6XpTqX/Ae2Y= ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1723212941; a=rsa-sha256; cv=pass; b=A2NR7sMzzbsHnjVOVG++H/WZ0DIhO36n1p/WqNfONyk+jvCL6+AHDEKcHTT1TqeHnfPrbe RV1xCbSifSiuFosmOlZwZgsB5W8x7vr9R+nddJc0qnCqAErT88pwBd+I2BnrHwKaQSFOOs mTfT0RH3WLc/fXSUhTpg3AVpZ2gEs3s= ARC-Authentication-Results: i=2; imf16.hostedemail.com; dkim=pass header.d=oracle.com header.s=corp-2023-11-20 header.b=SuCEY193; dkim=pass header.d=oracle.onmicrosoft.com header.s=selector2-oracle-onmicrosoft-com header.b=ixcysoT4; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf16.hostedemail.com: domain of liam.howlett@oracle.com designates 205.220.165.32 as permitted sender) smtp.mailfrom=liam.howlett@oracle.com; dmarc=pass (policy=reject) header.from=oracle.com 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 4798fVQO004799; Fri, 9 Aug 2024 14:16:45 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h= date:from:to:cc:subject:message-id:references:content-type :in-reply-to:mime-version; s=corp-2023-11-20; bh=k9bFMrwLQUGnNHg d2Ns4lQcnlRYPWDUhKpZAqaU1a9Q=; b=SuCEY193KQQ9sgp5eQ7mhpxCrLknAYk DlI0w2PfYDRg+xnmvrCekyaXt6wbV4qQJjF/sXj03BtVgCBXPulbqUC3CWGNNFIB gegrv+a8KyDcRMa6hrfMPkYJWznDpZWIwNce82s+ilJosnpun5hbNyYAjbjD3Mp4 nFjLBhcJbbjQLHfbbNz9UyviU+wcRkK9SLsKM6Ka1XdZHyggHRu+6HS+ddntTlYN B0qRNPazmc/kuM3M13Zx6NgdRWNUrvj8Iu/VrjvkBdVnYh6KV1ogwK9sgG8H38wl j0rsAUgOLeXA8XzzS2ayFVFRL2XGSJK3BuFHZBsJYBPPBFiZMoe7zAA== Received: from phxpaimrmta03.imrmtpd1.prodappphxaev1.oraclevcn.com (phxpaimrmta03.appoci.oracle.com [138.1.37.129]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 40sb51m42h-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Fri, 09 Aug 2024 14:16:45 +0000 (GMT) Received: from pps.filterd (phxpaimrmta03.imrmtpd1.prodappphxaev1.oraclevcn.com [127.0.0.1]) by phxpaimrmta03.imrmtpd1.prodappphxaev1.oraclevcn.com (8.17.1.19/8.17.1.19) with ESMTP id 479Cb7mU040753; Fri, 9 Aug 2024 14:16:44 GMT Received: from nam10-mw2-obe.outbound.protection.outlook.com (mail-mw2nam10lp2047.outbound.protection.outlook.com [104.47.55.47]) by phxpaimrmta03.imrmtpd1.prodappphxaev1.oraclevcn.com (PPS) with ESMTPS id 40sb0d2xpw-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Fri, 09 Aug 2024 14:16:44 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ftIRN7Yy/uYyjvqB/VhyuZLO7mtxDVq23Cmk2xhBudv8iQiARala66C3zZauvNxVq21TTubpMrVm9WrzxtNHpuEKT4021D5NhRno+CA10JEIu7KvnxvdKHrGuCYIW5LYQenxh20/82q/fBUQo7Vd2XA3SO9okIf8jAOCwU93K8Xvkd4T/UfyrYHdHI/s9K2jFJqA6wJn7lmLgkkbDsWnNajUJ3m8QqNqv2JCcDZAK4vrJHa02iOhZNuv8i3r5aFKUtgtf4LByi5bB+18K7KWERmMsTFjMaUpHDw1YiGOeUMWQwglMaKmP7MkAT1sw7EWRFEgbia0vYIzkgttYEB4xQ== 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=k9bFMrwLQUGnNHgd2Ns4lQcnlRYPWDUhKpZAqaU1a9Q=; b=kUWsnHbGidO+Ok02aSWaIXUOMohaE5aHjceubkqvP0vZGgTnxdh9ZtcZa9X3GVAddgamCNZcTYZ8ezG7v8wAMDKWSvvi/ZZfL/2fBr5lgZ5vRItVE01HUCIzE2EBy6SmcuNnjPGRnCxeMVlW42yWcULCcfWThERcaXbQwy744mgm2m/2xBfe1b8C2mPlUzXKc8vhE4oNLn00gVQJQLzSVjdkRUx21bzgj9Uvv2ufYv2MEV129Y4wxoJYK5Gui03hT7CoZpmzKujQeQPhJwpaAZEctuQa8JFrPrnWK/ylcxlCbLeS2Y6AWwt0/Aqi8NfaVmpXeHErh+zSu7qT9mbrVw== 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=k9bFMrwLQUGnNHgd2Ns4lQcnlRYPWDUhKpZAqaU1a9Q=; b=ixcysoT4Jjjmn/FXPcUxEVeZbXl5y2FMaNxC9CbD/MT7LQpVZm3YGm1VGcVEYL3ibJ3ujfPxlzYidC/O2xmoqfOnwAxpgeNdGfVXc8Jx1A6O/D684KelCG0zN7lvY5EJftsT2/4cBq/n10Qo4g/SXwM9hQCf0244fudc7G1cwzY= Received: from DS0PR10MB7933.namprd10.prod.outlook.com (2603:10b6:8:1b8::15) by BN0PR10MB5191.namprd10.prod.outlook.com (2603:10b6:408:116::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7849.13; Fri, 9 Aug 2024 14:16:41 +0000 Received: from DS0PR10MB7933.namprd10.prod.outlook.com ([fe80::2561:85b0:ae8f:9490]) by DS0PR10MB7933.namprd10.prod.outlook.com ([fe80::2561:85b0:ae8f:9490%3]) with mapi id 15.20.7849.008; Fri, 9 Aug 2024 14:16:41 +0000 Date: Fri, 9 Aug 2024 10:16:39 -0400 From: "Liam R. Howlett" To: Wei Yang Cc: akpm@linux-foundation.org, maple-tree@lists.infradead.org, linux-mm@kvack.org Subject: Re: [PATCH] maple_tree: use xa_is_internal() for better reading Message-ID: Mail-Followup-To: "Liam R. Howlett" , Wei Yang , akpm@linux-foundation.org, maple-tree@lists.infradead.org, linux-mm@kvack.org References: <20240808043717.10930-1-richard.weiyang@gmail.com> <2owp7odt4ryc42fzbtbxw6fg7zrkfi5g7tptyy4hgjpzqsthy2@46gbf75ibmz6> <20240809015852.r3ksdkgivhht3ey3@master> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240809015852.r3ksdkgivhht3ey3@master> User-Agent: NeoMutt/20240425 X-ClientProxiedBy: YT4PR01CA0030.CANPRD01.PROD.OUTLOOK.COM (2603:10b6:b01:fe::7) To DS0PR10MB7933.namprd10.prod.outlook.com (2603:10b6:8:1b8::15) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR10MB7933:EE_|BN0PR10MB5191:EE_ X-MS-Office365-Filtering-Correlation-Id: c427f84b-34f2-47f5-63f4-08dcb87de3de X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|376014|366016; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?KlMDTLclGHDXe3v7R190aK2QuKi67oSCFlRObUAsKXA48NUX+UkGXTEgHMuy?= =?us-ascii?Q?4yI93K2k97n11x3/rBCv6B2TXVqIEjSH7OOdgu2PQD2mBnRv/N6IhtDFXAPc?= =?us-ascii?Q?SKXQYuHYY1efjlwTC37moqTrpurvyxCPoCR9ozf1zuRAvDGKQCqsR8dXxuA+?= =?us-ascii?Q?Wpz/dyoo1w1NeH8orFKWiDCCe2HjVTYgi/lWNaL0MEMbD8s14xnQWxIb5szo?= =?us-ascii?Q?ydr5/0If5Yp72RrJE9FeOFfmeJ2zOlmY2x7PIUj4MrDUnGYqZn5VWt6Uwwl8?= =?us-ascii?Q?NSCSR9dF5s8OS1K0Y0f4wd7c4xINxDyMnC+JWV3BxYNiyVj4KOrR1zO8hLwL?= =?us-ascii?Q?BqRIVKnzVvnz71Q2V5S99lOfFE0Ga4lL+4Ag+EVKmLBIIb06HQ05MIR9wiwS?= =?us-ascii?Q?Bq6FttDu0usyODeCGfosCGjknXICM6Qd4gQgHYDah+aF5Hu8Xe9KYIer/aEI?= =?us-ascii?Q?3qx/U8YFApPatMOloLU10ItZvDXPUOpvYoBU/CWjttdrvBgBosIPpG84731S?= =?us-ascii?Q?vDfXDbQrwoe1DHiQHLHygHfwKVqjrLiu2quElM4iod7FLLvQMAzMJp5IgiWz?= =?us-ascii?Q?th5RtuZqj+oP1TM4EDYFHVyvBmmXXbb1ML5qSairel+zxATUZJKTih3liTu0?= =?us-ascii?Q?udEgHonEozEowf6zYNNhIT1g1tAkiSL5f3cCZb4rwuzO6hRWLks1EPxeSvZ+?= =?us-ascii?Q?LS/ttqfOnwOQj0xxN3b3dGUj5vJ/7E5W+8uB5pMUXYe/sxSknqEV7F3zW1OM?= =?us-ascii?Q?FZmLABvZE+hcEzYCosDRYBaDlT7jA6ELgqmUJrzmNdX+38fgkOdjOxU/uaaa?= =?us-ascii?Q?W6M2y5TwG295h0b8uuz87FO3yAO1HEwJlqOHr+qJRx11UVjH/2RzxBdlRxW9?= =?us-ascii?Q?MYDXLFrWQDpL42nmKrreiRy+yj9K9NOisulmlwIVpDGvJiHRTuslQMf6ZsLd?= =?us-ascii?Q?ZK2+NC7klCUhv1PoNumkKnyMaoLapZe4X9z2LvYJBSbcL+J3w/g7j09aVmdv?= =?us-ascii?Q?/mHeD1y4nVBEIIW/KbPMCn79K8X+aKhEWCFh+xol3ELyJaXkcSrbtdQ9JqBO?= =?us-ascii?Q?HT3F6ipxS/Ib9r7vAYzPc619MuU223fj0MR1uNmf1K6x0A9QtKiIayApgx4w?= =?us-ascii?Q?llcJeyxPZ6zlbNGxrrNBBbExHOYbRe82zdChhvV6Ch46LLfLwctvLAt+W7KQ?= =?us-ascii?Q?A/s6pNmQU1ZsYGsCJfBsOn7DJU1smyFfRs4HdshrVFT64Ge8j4O62GJDz+57?= =?us-ascii?Q?nUcCBv5N45r5za7Xx4xMT/cnQWxpQFpM69ZHmFpvGNL8d7FuJkW6XqncJjmG?= =?us-ascii?Q?PqKGZ/40Z5JUR6OEkjPwiVmpT6GyoTFXQNteQBugS/td5w=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DS0PR10MB7933.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(1800799024)(376014)(366016);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ATd/WaFkwDUbn3Xh1VWBLnh//D4741N2cfX5Jc7atWdgEK0j8mc4lybiipld?= =?us-ascii?Q?q/5/TxbvRfW+fhJWEGMl1whrmPVqmCbXnWnKE0WiVwHR00JHiBF9Ph/fn2S0?= =?us-ascii?Q?/9KsJkBoCzcGILD0CggOMtCNNbWg/xbZ6qW8YdhLL4wdPhPtugQBO28WiY0Y?= =?us-ascii?Q?o3Zs1DBGDGq/VtlZ1dToXLSflsl8ns/hYdC9Cj7xJJMmoBpoAuYoV4lj3ZlL?= =?us-ascii?Q?mUqeH+C7N18dsEl+M015Bq429OEIMu0JN9UIp7afiBru6yUmjRDoRWq3kMbU?= =?us-ascii?Q?/A3OkLZGXcs6Tudmtvnwy2TOH9s3mFcZv8TZw/B77NiCnc8rA3aHThOCGKk9?= =?us-ascii?Q?TRv9mwHAT8KJvjaOBT5xHryb+xE3qLnih1JyWrS5OeE6arSafrQQS4oHROL9?= =?us-ascii?Q?pzCAQQ+SZDeUB4qRZEhuhwDVkNbe+uOE5iYISAEwzRNIn+KuFfZV1QQ4a/0H?= =?us-ascii?Q?VkBFghUAaIhyxGlvro+5XYGK0o4f1XXXtGS6+PyOggzrH33l1Lb9mxhwrxBH?= =?us-ascii?Q?aJ2JfBIFEHQ0426acwE+JSR7XIRtTTHJnqUaU6pVhkQ39JKMYmLTwP1PEG9T?= =?us-ascii?Q?dmQE2svgGZnrRRXLsd0atdeWCAGhLnb/HaJ80/hUmM5PFyrIkbMQRbuKPnrx?= =?us-ascii?Q?eiaTx10YY7DEh7k53oMWrGb4rV6jpXEVY5s+gp53iIqtjK2xiZ7HDZku2LD9?= =?us-ascii?Q?NCGwNgjkhZUfPcwKKP4BS2RstdfcPMMEeQ+LoPn3Mbu4dXMbl1FjN+6tKY8z?= =?us-ascii?Q?8GvCcL8yJSfxsYrLPI2jMo37j/6tU03OLUyLG3fz/cklLQEtrWC1/CzbxYWG?= =?us-ascii?Q?iEGwTLDu1FI9Zmm7jmwXzSSWGgBjLnFajDgF0uJhsnP2+b8PGQ+JcmGK4Irb?= =?us-ascii?Q?kSFRo9Umaa0ZX4d1H06oMAUKj2HyesOWMUVtf8YzWlegqtEB/L3YRPL+ExTO?= =?us-ascii?Q?QCP30pnrca5ioeZlY3UNieW8ZKd10s3hDJcSYiFq37g3mrVNn9l6JLavJEKL?= =?us-ascii?Q?WrdMEsuoWiFdSWWR6OVsuvcnEK9sTJvp1Gi6K43cAFhFYnQa9CDoPu2rHSuR?= =?us-ascii?Q?rWaxtuHP0iNeKlpVHs3nIbi7Vc+LkotDUvvk7WiauQwqzYhdsYgRTg2VfIef?= =?us-ascii?Q?ojfVGWmsw/1DG39NO7QXfVOfHL+wDYgpIp3YWv2PZ2NVodfbRq1tPRifZnOA?= =?us-ascii?Q?vIYANB8Vc0dR4gR3L0VUNbYMgXNGhG6kOrfojnmVim5mR6myOGzb6ufP2P3u?= =?us-ascii?Q?bqjYD48A6ZbSuG5i5tl9dRB3rpunf2cGhrf5PmOmR9oNb0VK2Pirs7nfwBOz?= =?us-ascii?Q?8WNEX8S6jFshKF7cEsfZJyJiqcMXNxivajuSZjHKgz06TvenvIsbdlL7wMSZ?= =?us-ascii?Q?pV2yKZ/heG2xFfY3qHZDa0H5ihO9Vs8y8h4gyusqoaN+ybh9kifJV68CIoMY?= =?us-ascii?Q?M+Ls9py/HuXsi0M5EIRX82dDVDE5Ywi54jp1t+ATXrN/dlmeEu6WvtT8pteB?= =?us-ascii?Q?Rc+KmNKAN6f4UYgVQM+QxieFMr8efgBcLEjfaxZUiPBviSeBxoRobhbt3Ygr?= =?us-ascii?Q?4KzIanHqkHEi2M/SLUQLCdnlP38jO8EL5F7EuJKlkS7uFbHYPtxMfsl6n+fs?= =?us-ascii?Q?TA=3D=3D?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: Nhf0M4DpmonB0lhC7eQ3WPf3T0U9Yb8nVo/Buz/En3P8MxbJGN5Vfn1wHpOXtjfFZ03nd3W54qtmMmxUOu6X5KQ60pAdDe/8Q6GN555C/MjRQc6svt25P39jJ4u/CDmVxB+SGGAgvXj2t60aT6IIY6POubdw8jwVOpM2DJAbCiH9MN5Q6QTehdcDoBjhLBJLdr7YntHZzX3uH3xlh8mCASYYfoO99uMeJRVzg5jI4ArwOBj3nj3F7Tz4E1xZ8wRjmQyZvW5giapJ8ZjQYzKmOTZUwz/rcJuKOn2jdDPk+8PWao09tWXaEihAalilVdwcF59FFjl2kIqCD/Zs25ef+ja129BQs5uZqFyikA4lOnNnvwUqD4pawzCVWd6jARzkSNhyUpcVpgOicDKDH7GRV14GHfUn9rWhfU+rGO1Ltd0vc9GuEWz5BqJ0q8eXnfIp8EqIueAStHDFfQnYaC1S4/Ql2PsjsnvsRlk5h149bCUYmBl2On8Xc1btB81fITLmzg69Q9impIOkUsXT+yJvB5FdgTp7SUVJKlpGbGyuhdDOLBoJqHAua6keQhSEpQhfKqfG2aY7sORqqr5NHnOQ2VuFynWwLKej5EjoTVMT1lo= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: c427f84b-34f2-47f5-63f4-08dcb87de3de X-MS-Exchange-CrossTenant-AuthSource: DS0PR10MB7933.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Aug 2024 14:16:41.4485 (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: qqUtjZrADAa+2T/AxwGU+nGFKz19Q3A0S+Z1R6CXu16gCcmSyxgrUCRL1j9a7y0cWhGfFJB/HG6fm73mbg+Wxw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN0PR10MB5191 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.680,FMLib:17.12.28.16 definitions=2024-08-09_10,2024-08-07_01,2024-05-17_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 spamscore=0 phishscore=0 mlxlogscore=825 mlxscore=0 adultscore=0 malwarescore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2407110000 definitions=main-2408090103 X-Proofpoint-ORIG-GUID: Umi-mXN2M2isonQggLsOdnsX1bHdTt7B X-Proofpoint-GUID: Umi-mXN2M2isonQggLsOdnsX1bHdTt7B X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 680CA180026 X-Stat-Signature: f5bb39t81s149my8wpdakbutpxshsjyi X-HE-Tag: 1723213007-895384 X-HE-Meta: U2FsdGVkX1+y/yWm05xfeT9QNqVnFsbj4bE+07xZFvGeAF944JiPZovdrj8XR0gs5kohG7wZ/9G2Tf9yHV/NawGtj5eTPQ0/LgX1uZ8H08txtmwKzHajKoKHeBDS9YI0m9H25Yxx1TccpZ6Kt5/EEnA/V9v+gfhHRnJGiQ9J8mTHImJqUkFJgx6CR28JP0rPT+dGujRyBTylL50J8LpHsX0tJNOmoCoHXYjg4LPhC4iXWoMzIkky5vJzGCPcSvUdd5V/bZzauWnR7L7CnrxaL00VLEOBL5SYFnakr8rE5cGqCvdVlyEaP76XPgPiDMOJt3KZwC68a2i/CrFUWAwfxr92mfDKIgH14d0W/xVfIgdlo2JvolcWkal4vSriyVnodIc2+FyRtnGtyhM/eqvDbYQZ62YjfGyFXGw/DsiI8xUt4O8bdtzwtB6Hu3uLL6WXsYPqPIxf8IVZs401a5v5qA1OX6zEZFKLyLpDKoo0KXW40T35pq9ZBJe8eUTsGtUTg4dDKx0pn2Tgq7YV6f/gYrf6QNuqEahB4Ae/ZzeEdx/QblBbcsWmiKYN7qcP889hR9rcNPTIJNZsysGmG0vLiKXd/+N+//Gfyw4h378Fg6e36Gjjh5iSm/DXrTeYlZ1JPpA0Z5VrdJXyqV4chtCLHY356RR2ErAc2Go82HoKHBPKCdsmSI+k+0Dpf3U+Zsvqj7CSu05/mgOpDzRsUp5ML98n7YqvyY28ZKLXY0qURAo4ATqWms24vPI9Y408v6ppJUHcCtNAdyt42k6kXfoN5306PkUczxU+k0qDWW2yLkdFnrRksB7ZdUzJ89hy3WdBGlBktqck1A6QdgWJFWS6CF73Dp48ZwJAOhRc9gMmsJHDbebZuieQvtMrnQEIJlP2nvz87eTS1jG5gKm1zQPzlm3Aj/lvsgijZV8RYA90pRoSNdj4s1IAkk8PgmBoyN8+f4qhYJnVIJt+eMnwkdJ v0uIJ/9Q lGxP9QjCAIb3Tf2ArMtJHYUK/HAhnLNSw1E/AleqG+pfrdMb1YaP9eZS/ZY1mwicGRCYFREcBhRG1MXSli9kfvUP7Ijx+ItT8LWLJiI1Ete1tgiH6ZqfFIQrdFxnWwPN4oWgxGNtpyrs0DienltEGhEpkdnYiITKtW4X0jH3sSB7cCnihROpuzPj/maOsbR1QNeWhOwvrFY86AVzvPe+id+9EBz6P82y0+Sd5k3Bkot2d/cFZNGT6A2diX68eVYbpkO2wFrmz2PD+HPcAoRbSDMuGnkJGX7OZAZun9wDkAu8IrAxQsV/d0c5rBD0NBc9WT7SQZv6ZH32xl+JuGbBo6u4rKxO/v2fk7/rKbT6ia/7Mozb/waflH6f48dyhQiyboKmjL4+On57hvt9Ziez3SrI0p74fXUkgTAJl65YJcrKGybjsrkChQFfdCaGdQRctL+3xigTyzZNI85W8U/5tppbgZKGnLNhLXQXzqdSvGLVETRGuCicL06MnyU6ZUzJWc0FSzL2bB7Loq+VdF4+IldXT3mZ4vvx/mJOcVs6ri8jhN7c= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000017, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: * Wei Yang [240808 21:59]: > On Thu, Aug 08, 2024 at 12:53:55PM -0400, Liam R. Howlett wrote: > >* Wei Yang [240808 00:38]: > >> If entry is a special case, we need to expand root to store it. This > >> case is exactly the case of xa_is_internal(). > >> > >> Let's use xa_is_internal() for the check, which is friendly for > >> audience. > > > >Nack > > > >This check is to see if it will be detected as an internal node - this > >much is correct. > > > >But changing the check to use this function reads far worse than what > >exists today. If you look at the code below, it seems to indicate that > >the internal entry is being inserted into the tree - but it's not an > >internal entry, it's an entry that would be detected as an internal > >entry. > > > > From my understanding, you want to say xa_is_internal() only applies on "an > entry" in the tree instead of a "value" we want to insert into the tree. > > But it looks current code already use xa_is_internal() to check the value we > want to insert in the tree. > > For example: > > * xa_is_advanced() This is used much lower in the stack, where you see the entry is coming in and it seems obvious what is going on. > * mt_is_reserved() reading that something is reserved makes sense. Saying an entry is an internal tree node makes no sense, it's also buried in a function that doesn't immediately make sense to be branching on a value. > > xa_is_advanced() is used in mtree_insert_range() to prevent inserting an > advanced value for a normal API. > mt_is_reserved() is used in mtree_alloc_range() to prevent inserting reserved > value. > > Both check apply to an entry which will be inserted, instead of an entry in > current tree. And they are clearly to inform me the value we want to insert > has special meaning for maple tree and need to be handled carefully. > > >This is not friendlier for the audience, it's confusing. > > > > The confusing point comes from audience would think the "entry" is already an > internal entry in the tree? The confusing point is the resulting code doesn't make this any more friendly for the audience, why are we expanding the root node on mt_is_internal(entry)? Why is entry an internal node? How do we get to this function with a node as an entry - That seems wrong, let's waste time digging into how that happened.. You didn't make it more friendly, you made it more puzzling. > > If you really think audience would be misled, I would suggest introduce a new > helper, e.g. mt_is_internal(), just like xa_is_advanced()/mt_is_reserved(). I'm all for readable code - in fact, I'd be more inclined to accept a static inline function that just does the check with a comment about what it's doing.. > > To be honest, an open coded check doesn't looks a good practice. Thank you for your valuable insight. The entire point of your patch is to make it more readable and it does not do that. If you want to make it more readable, then I will look at that patch, but I don't particularly feel motivated to spend time doing that after this exchange. Liam