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 BA7E5C5AD49 for ; Mon, 2 Jun 2025 15:04:10 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 5C75C6B02CA; Mon, 2 Jun 2025 11:04:10 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 5A0406B02CB; Mon, 2 Jun 2025 11:04:10 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 48F286B02CC; Mon, 2 Jun 2025 11:04:10 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 296E06B02CA for ; Mon, 2 Jun 2025 11:04:10 -0400 (EDT) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id C135E1409F7 for ; Mon, 2 Jun 2025 15:04:09 +0000 (UTC) X-FDA: 83510781018.19.E6324CC Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on2077.outbound.protection.outlook.com [40.107.93.77]) by imf04.hostedemail.com (Postfix) with ESMTP id D6EC140015 for ; Mon, 2 Jun 2025 15:04:05 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b="kyKtcMR/"; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf04.hostedemail.com: domain of ziy@nvidia.com designates 40.107.93.77 as permitted sender) smtp.mailfrom=ziy@nvidia.com; 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=1748876647; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=f+Tjl4U2HmJ1UPWNMJGyYTlofAAh25xpvEFPc+MrgV0=; b=OjUthBpfNoRFTVmpL23Nf044mQa3MdxrAN28bG6wG8gI/9FUMUOWzkRSXTGYjEFtThJa63 dqB3nBrYvmjuzw8pTA250gKUE89GzXmUJB3Z4vAd4DnSZWVJ9ZuY91FSOrHQTMXMTL70RF TI7pBUmNV/T1fNZJP2NyCTroakE+Pcc= ARC-Authentication-Results: i=2; imf04.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b="kyKtcMR/"; arc=pass ("microsoft.com:s=arcselector10001:i=1"); spf=pass (imf04.hostedemail.com: domain of ziy@nvidia.com designates 40.107.93.77 as permitted sender) smtp.mailfrom=ziy@nvidia.com; dmarc=pass (policy=reject) header.from=nvidia.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1748876647; a=rsa-sha256; cv=pass; b=MUL9mgHKVcj4ddPjupqDfKxfHkY0b33S0TI1pJ+uHgFPGeAgv87wMD+it/6DIw4c7Q9FD8 rwRRU0tMNlUGBj2B9JevXqzbwMx1XgNSj9CHHvNBhIpY5VHKZH+d6NGbDwDVK1UbUkihNI 84W8GLKDl5gE8Q+OmVUfL3NLy+6LNsQ= ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=jNKDIzrdcAo4iOtJ6dXEXezU5sUFLsqS9ywFk5sAB1x4hpBPkpTvmCL5VggHP/GVQArhgQdN2mpMeLW7j9GsIp4ZNPchnI6laguJJoY3ZWEkWZaq8k8+P086DLDzaMFjhpFy3jzd9w+pSAvMM2w2b4oBr5zYnK2bEEV4FzbnYaC03Fnd0yPiA9C0lrnml+a1OJB+RrpbNrZwixbqGpqfbKyDHcKNaQsxmiOKwbvrRfPc6XmLkioI4vtw4bLfzmQi+OM4Ydw7vVd92kx3vAcxwFIO9TTDX+5qKewOQblg0mPqRYP3LHHxzCBQKPs50hcHJB7dJKqq4x78nFZ6mdr4yA== 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=f+Tjl4U2HmJ1UPWNMJGyYTlofAAh25xpvEFPc+MrgV0=; b=frnY3ayVj2K6GiSxZgSPD61KR0ijfi6dmSREV8QRKWA5OQZbQrh/dJgpskiWikNYrR6UO8qC+kQQjb2N7QtKuFC1cqupi4lOPPyW4NziBSBdP7r4LwAkAwJS2d4q7i3DI5g+WXd7LsCtRp6paLfHymoOMrT9ot3EzxDhB8yWyibm9WEEESHP5Ep9vS5ViHkMZpC98dUrkm7JaM2aQ5bnaMGrQpUSDD9o1kuEJ4c+FG0fZ/TynzVxOQzQPuNeSfMW+29ekipi+GmnOz5C+39cPTEQBqgZYokzGbUSS6exSBbkL4hy3/w/PytqSPmodf5cHTeNO2e837zHFhr1CQOReg== 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=f+Tjl4U2HmJ1UPWNMJGyYTlofAAh25xpvEFPc+MrgV0=; b=kyKtcMR/qJzb2W6mOdYS77Vp4nw0h71w+m/Ra6PJnYukbiZwBSH4SIAfQ+zrNGcl0dxLpq5tcs4XDL+lzGVc5aGFrLD294nJSp+tVVHrtw2PV655VX8ggUJsdsxec+nEyK2SNPhFmONIwBEl3YVnztXvyMlkDMVlZFHGV87NFQqSstbioyEcWrZBOe6/fzhYJGI6ZClAJhIIUBiipge6U2aGvUFccfwrfjqA1J/bz5689sGZb02mt1Zr0C28+lyHNtg9Cvmr/5bm9yOd1AZSAeP4qiwHtUKaywmizfbFCU7gVTXbauhIPwhvQ8rJ1avzahbh+p/14ay2txwrG3yh5A== Received: from DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) by SJ0PR12MB6806.namprd12.prod.outlook.com (2603:10b6:a03:478::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8722.33; Mon, 2 Jun 2025 15:04:01 +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.8792.034; Mon, 2 Jun 2025 15:04:01 +0000 From: Zi Yan To: Dev Jain Cc: akpm@linux-foundation.org, willy@infradead.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, david@redhat.com, anshuman.khandual@arm.com, ryan.roberts@arm.com Subject: Re: [PATCH] xarray: Add a BUG_ON() to ensure caller is not sibling Date: Mon, 02 Jun 2025 11:03:59 -0400 X-Mailer: MailMate (2.0r6255) Message-ID: In-Reply-To: <8fb366e2-cec2-42ba-97c4-2d927423a26e@arm.com> References: <20250528113124.87084-1-dev.jain@arm.com> <30EECA35-4622-46B5-857D-484282E92AAF@nvidia.com> <4fb15ee4-1049-4459-a10e-9f4544545a20@arm.com> <8fb366e2-cec2-42ba-97c4-2d927423a26e@arm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: BLAP220CA0003.NAMP220.PROD.OUTLOOK.COM (2603:10b6:208:32c::8) To DS7PR12MB9473.namprd12.prod.outlook.com (2603:10b6:8:252::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS7PR12MB9473:EE_|SJ0PR12MB6806:EE_ X-MS-Office365-Filtering-Correlation-Id: 979257f2-4d84-4329-44d3-08dda1e6b589 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|1800799024|366016; X-Microsoft-Antispam-Message-Info: =?utf-8?B?TkxCZFIzanFRVGJpRUtOQlppZzNMRTZYT1FpSjhsbHFJby9FR0pTMlpOYW5r?= =?utf-8?B?VUJod1dYbUxnQThSenRucVZDemRMRUdLK0hidVdaVEZRUDM0YldRYzJHVWxk?= =?utf-8?B?THRRQmtXRlpUQVF4ZXJ3eG01dHR3MEFjUVo0UGxaaGhUQXA5K0tKdy9hZU9t?= =?utf-8?B?V3FvNnQ4NHVnb2JmdVcrZmtGaWlQaTBTMU13OHpONkpNcFFJR2h4TUY1M0tS?= =?utf-8?B?MHc4Z2laUWxhTEsxV2hOcXNKa25VdkxTbktMVTh6WWNTeXdnWnpNRVdjdGhJ?= =?utf-8?B?bG4xR2xoZkJmdUV1TmpCeEVFNmNXQUpvZXJ5S0xwS2JUYkxGUlc0bWZPV3JH?= =?utf-8?B?dHVMdEdIUzFKaWgyd29aSWpMSkk3V2NKZDNJa3lwcW90eEpaSHNvMG5rN2tQ?= =?utf-8?B?M0l0VTBablNTNUVucUVPakFvNi9zRlNHdjNaSmtvNVJHNE9ZeCtlZ25HeDFF?= =?utf-8?B?a2plbURtam9WNnRpUENIQmtBRVZkVnM1VEtjcWYyRjJoK1pjbWNaMUN6ck1F?= =?utf-8?B?MXZSWmVmMDQ0OE1sMW4xN1ZNeXFnSEJIVlJWMmU0aEJDdFd6WWJIZzZpeHow?= =?utf-8?B?YU92eXhPS2lQajZ3cFRxTnJQVDZ0ai9OOGxHWlY2QkU1S2d1MnVBYXRTNlFO?= =?utf-8?B?RHM4MW9tbHFOUW1ydkh3SE5RYlNtM2hTd0x5WVlMQjIyZkMwMHVWRDZZWDlS?= =?utf-8?B?aFRQWHYwL1IrR0d1aFQ0emlSMWsyL2xEQ0E1enNGMDVpZUFsQzJZeVh6RnhI?= =?utf-8?B?djM3WlJjeGFmMm51TThPRkQ4ejFmNzhLdDlWR0hKNVdLbXN6N2tSclVwZFRQ?= =?utf-8?B?eDVJdzBpL1dkQjZOUUgzVzZvdk1uU1lxWndJY0hCTUhMY3FRTG9ML0FYcDRp?= =?utf-8?B?a1Z5S1pabnZvMGtvckFVU3JtQnlzRUh3bG9kUVM2STZVTmloOTBGSms4Ky9i?= =?utf-8?B?YnBvSVM1bEpJTWZPL3lDNDNYRDZyejBVc09JaXpsZldFdno5WkNoWmFkQTRG?= =?utf-8?B?N1U4aWtIWnl4NVZmaTJsMmNTV1Zwd1pjdFVKd1ZIWWpOc2lVdXpFb1pnMGFt?= =?utf-8?B?Q01oYkd3aElwbm00bEZtMmZXVDVFd2VITm4ya3VBMzJYOTVGRE5WQkdyRFZj?= =?utf-8?B?ZitSZm9mQi8rbzA2NUdnZzRIZDE4UzlRSHZZWTNPMEVZc3BsUjZyQm1aNVFM?= =?utf-8?B?eG5lZmRELzFpVWN0Qkl2RnRCYWRIZG9WU1U1NnR0ck1Sb3lGdnh1enVUOXBY?= =?utf-8?B?SXpiSVVmajBHY291QllxUzMrSGxXY04wVTQwK0JuOXd1dU9IZnlWZXVkY3RE?= =?utf-8?B?MDQ2SC9VRUFrQlUvRlNFZFZROGFhcDVTL1M2TGFYR0pWVE9zamJ4SUxqMFVk?= =?utf-8?B?UU1tc1ZkN0hZVEZ6UWxWVTRibFNCMHZVQjJTVTEydk4zN3hkRUkrT1R1NUEy?= =?utf-8?B?VGdma1pTajg2bmJHSGZZTVZKUG1aeFBQbUd1N1hPNENJalRMaXY4cGlpTE5j?= =?utf-8?B?WjVSQlBuQlN1QUc4dktkMGJHdWFLTzlyVnpKc0lNOEZEcUMwdHlMVzlHWWRH?= =?utf-8?B?THhNM1lDM0RiUFBPZU5FSFFnNFNqNGN1QlE1OFQ2VXB4YjNwOUQ4enlwZDQ3?= =?utf-8?B?N2pkbUVHY2o5ZTFSVlJBTXU5SFhFcXpBdmRyVEdRUk9hL1E5TitOUCtkYVFx?= =?utf-8?B?VmtFZXNqbzllNWZtRDRteWNsSDlUbXh0dHM3ZzdaendoMm9GZVNqM2RWMjZl?= =?utf-8?B?c0JYQWhoTitzUHY2bE4xaDh1MXJ3WnhjTzVXZFdYRFpUTWVhKzNkMENPUGha?= =?utf-8?B?cEEyOWttbFJJdUJscVVBUUYvRUNnQm9jV2hucld4eml0cTl2bm9RM2JERlh5?= =?utf-8?B?QkJxSVF5aGZXWjdrdXNCWEZWMndmdC9ZOFRQZFpzbVNWK0E9PQ==?= 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)(1800799024)(366016);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QWVJL2dDbmI3d080VTkwcm5zQm1LZU16SUxVOGZ5YUpsQ0VxSHU2Q1NmOUIy?= =?utf-8?B?Wk43aEUyWGFuQU5sYUVZOXovY2hYa1B3S3BPdlhZaVIvY0x6eVhyLzMza0tC?= =?utf-8?B?dEhXUmdYOTJaMDlFMU8zQ3RBNGJieWpqOFIvRkd4SFNHeUk5Z1NjTGdzdlhn?= =?utf-8?B?VG44cmVabnJpYkhZd0F3aE5HZk1iTG85eVJvdkV2a2NJVE00R25yYWZvQVpS?= =?utf-8?B?YVRnTUZJd2ZVQlRGV29VcmxFd3VzbHhEK051OWtXUlhCNG5uYjBDdS9XTTlX?= =?utf-8?B?QnRvMk5SQTRobHl3Z0R6Wis3RGhsN1BYM2FVU09YSWtyRzd4VnRCdU9MM2ZW?= =?utf-8?B?Mm80Y0h4MEJ3MVZUMkNEQ1RVcWxjazFtZ2JCUzRFQzdKYjZZK09JSG16TUtZ?= =?utf-8?B?VldVNS9SOEhvQnFVOWpxZk5zZTJlQ2FqQmU5QkYrY0NYWXgzU21mcDROaUxV?= =?utf-8?B?c3dXa2xVSEpDY3VhUzhNR0ZJNWFHa242bzNDd0M0c0I1UHp2TjBicjkzdllG?= =?utf-8?B?TFNkaGphaEZTK3pseEZmVSttK0dZZWxoa05aYm9QamNBcUtZUEpCY1pLaWxm?= =?utf-8?B?VkZKWExhSDVwR0I1dVJKNDhGamxUN1NZR1U4MzhBT2pUbDJtai9lcWZCSjQ4?= =?utf-8?B?WTBPYUFNYmphRGlaZnZTUjhRUWRZVDBxZTdlRThFVkVvUWNycFRTdWxIZkJR?= =?utf-8?B?Wmx1WlFPWFN2L0R6SmEvRVExdFBFL05sSHI2amtXN0Y0V0hRTkxMaHdDM0hQ?= =?utf-8?B?Q005Y2JhaGdoL09XYVp6VCsvRTJSSk9UTnBYaHpYL0JJRzMrL05UaUMxYitL?= =?utf-8?B?Nk95L2V1ck9mcDIzVm9GR0ZrSnBiZ2JXNWZVcUZHZWV6cUludk1PdzB4TitY?= =?utf-8?B?NXVkOWNXZU00aFAvNVZqbVFFbXVnNU9mV3hjeHRabkJiODFhbENjNU50YklY?= =?utf-8?B?ZlJadGRNdjhRRXYvSzM3SXBwK0FKZ0R5TWp5a1Z0bng5SUUyUFozZkt1b3VJ?= =?utf-8?B?dW81dy9aSzNyU0NaYXpyMlhXZkgvcWtQdEEvVWhXaHJYODRkeUhqSDFydGZ2?= =?utf-8?B?bmg2WnBlODhraGg1Kzc1THpTRVFHSCtlUDBLeHQwdWtmeE5wSVNxRE9XR3Ru?= =?utf-8?B?L1RJbTFLNktPazdnYk85SVNXYThVaGFuQXpFSVNnSHRXbUdDMGZqU3pOSnBC?= =?utf-8?B?TjNhbmFDRXNuMWtkY2xlZTEwcmttQyt0VzEzdTlYaWgvZnMrR1U3VGVjSUJq?= =?utf-8?B?cFp1NWNTYm11cEVxMDV3N2lZaUpCM0kzQkJtc0d5NExacWFMODk1TG9uakg1?= =?utf-8?B?VjMvTzd2bzI5Y3lRL0ZsOTAxeGdQSGhlRDE0N2NBMm9pTkErSlRTMmdDUGV5?= =?utf-8?B?b1FieGhjTVZRcXdyWFVURW9abHNxSTNYOHUvQkorYkVrZzQxc0xSS1JQNmVB?= =?utf-8?B?ZFhFRHpXUkFvVTE4Z0hBU20rV3JZUTZlaFRNRjdSemZlNHRrT01ISDU0R3JP?= =?utf-8?B?Z251b1BSZ0hubFJKb2xIM2FTSmEyTlYzbldRNVRPUUkwT2Joc2hXZ1BwYnRN?= =?utf-8?B?U0QyeENCcG1heXZwOTlnb0Z5cXR2MjQ2UnJSWkszcEUveFd1QTRVNDQ1QzVu?= =?utf-8?B?dTBhUXRTcjJLUE9TMS9KVmh4MjVoU21LZzhrYm1HRzMyUUtNRUZveTU1a0hO?= =?utf-8?B?UEVKZXJKU1UzTVBDYS9JdjZZZ0Z0WHpRbjdIWEZXZFVQaEhyMnpBZzVWYXpx?= =?utf-8?B?b2xXNDQ0SVI0R2VSdncrekJ2UDN0VVhQbGdmWC85OGlVcVk1QlUwZzVQYXRQ?= =?utf-8?B?WVVuenBmQWdCcjZodGNlVmFGQzlTd2ZnRWhZTnM4QjlpNTVoN3prejExK1dC?= =?utf-8?B?MW5WcUF3Z1BrRVVpY004eU9GSSs2dzFvbkRPQUxXSTY5Ym0rdFVmM2M4eVY5?= =?utf-8?B?UHJZUERSNFZ4bHpKZ2JaMkZqUVpvRWRMNjAyUksrbThBemFBSFZVcEgxR3lW?= =?utf-8?B?NlE1aVp4bStXaHU4eTJ1dzBZdkpBMGxDa2Jxa1Bjb3hZZWFEdVh0VnRoazMv?= =?utf-8?B?dWlnTm5SZ2ZtSVg0MDhWMk1QRG5KRmRPSk5XdFYzcUtCNFlWUmdyeWpWYXMy?= =?utf-8?Q?/qlEX+7DBFGRYVKIaHgIKUilb?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 979257f2-4d84-4329-44d3-08dda1e6b589 X-MS-Exchange-CrossTenant-AuthSource: DS7PR12MB9473.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jun 2025 15:04:01.7702 (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: PPwbvFmKUg9vFVqKhPHuIOayvq5VGfQC8VAqIxnHYIlfhFUYid0AquNLV9mNdBRC X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR12MB6806 X-Rspam-User: X-Stat-Signature: mw6fix6i3xtpjdzwkqt85soyopp55m6i X-Rspamd-Queue-Id: D6EC140015 X-Rspamd-Server: rspam11 X-HE-Tag: 1748876645-795183 X-HE-Meta: U2FsdGVkX19QeEyy6Bg8nkV7F4Zw9HJRJD+u/lRNl+d1F6YcVT12tCMHI9fWEo0024d8bbW5ZukWdfBix0h7bekwKOJ7c+1zU96xkRd4LplVHgjVn9A6kmhDEtxGeOWGThMWib3QzlQgxaIVy18fIGFtSLVZ5xaaH1feXka+NNADML7n/l57zAyGA5inaQyvJKKaZnXXfNtr0RabDjqjcQutpxT85r5uRAv5NeQYPbNf2ii2jkp3FsI6KSxSVLQ7nXKxNZXGi24jBQ//SscYbrh3f8e3mtum5RZx0twcM3NjKKIT/hbG2kXYytMkYWWAMHy35PePDc+bDZA9xBvB/tq63oBPbRTvBqRp2hNzQUdIj/2rF9ULhRX2LzsRQGbqmck00KpPQXmSJss8t4a5CPxGEkhyOG1qWbp+HspMP6U987PT2xceOTHIB68imzka+rX3zB7qHX6F7ObwHXIJtX5KesjzrkpR+35Z3Eln3hPiZ8chyeqaByKU+L9fgshrRF5jqlyPPJlSygrrDv3d9tX797y2w4SEydCVO392cp+Hu9bvPWWotcwFyOrlHtORKTkXlIISPb4WrmLlNm4Ot5rghl5cSGlGW0pllNzwNXbyhvKtVTZfnxc4q+HiKdhFaCvRs6EBTPiinVocFdLXBNH9+JNvERXiOOH6WhOITqrN51vWN6QAllR9HhnwV0q4cSRSVp1taurwZzS4Cj3HTgWvfco74OCQmbAJ1phaP1wVpd9tf6yNWmP44TaWBzfC4qXRfHm8QW5caxSjijcg70jdyggKUncML2zduDtmxOuz8pf0dBmS/huiPqRIV0eCkMw0pfqqE0wzfuQlnOcycDF5oIc6us0+r6udSxr478YZYUnMFOAmwVMhKwArYMaz5jXSUuWef9GWK//hGLhSQh8JJy765L5ZXJkaiJ0Ouuy5OV9hZvmbBeLmAJeGYdYkS6pc/4C0H7q7mmLGHZK BUmy+Pb9 rWR7elXi8/Sv59vk9y0wQ3lqHKcxmmuAVFIm+/nj91r87qccy/IzlLFPyvkOatBXYLgSe3tttRb90USL41Iixr29dz3u+KOvNByrTqNQXRI18xMGH3b/HmeWUwpShtamy5V0FrYOV6CM6qB5+wurcStmJKwqnI5BTLJZm343yI1vBrsrxnJlHx4KejkLWtkX7FoWViLKHhc0RIUz5GtaOOylIWwoUm4OtikyZ/RVKS2Z3gsvWFv8k8AG1IAxxl2H1ZwO6PDTipOnxjXk/GB9mjfAvr/4LVoDz9bPUgqb1SaHTHyePIwt7Jw4YI19Z3R8mkOojfV8rdxvUI7O9OWqhhfvNGps9Sd6q10xEGYt2HIB1H9zufn+jV2Ft3wgWts3Sn09Q5JXlQ7PJYFNIGi3Tbmf7Wolz6KwN5pOH1Lp/mKGPp2TKihyIfAoOdCx74Bvc+zdCLmg7B7KXulm23RZWdq9Gj28J86Xqxg9dVtcPYxRocri4WdX32u/wvDRNDj8OXUbwtts9jrRc8s8= 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 29 May 2025, at 23:44, Dev Jain wrote: > On 30/05/25 4:17 am, Zi Yan wrote: >> On 28 May 2025, at 23:17, Dev Jain wrote: >> >>> On 28/05/25 10:42 pm, Zi Yan wrote: >>>> On 28 May 2025, at 7:31, Dev Jain wrote: >>>> >>>>> Suppose xas is pointing somewhere near the end of the multi-entry batch. >>>>> Then it may happen that the computed slot already falls beyond the batch, >>>>> thus breaking the loop due to !xa_is_sibling(), and computing the wrong >>>>> order. Thus ensure that the caller is aware of this by triggering a BUG >>>>> when the entry is a sibling entry. >>>> Is it possible to add a test case in lib/test_xarray.c for this? >>>> You can compile the tests with “make -C tools/testing/radix-tree” >>>> and run “./tools/testing/radix-tree/xarray”. >>> >>> Sorry forgot to Cc you. >>> I can surely do that later, but does this patch look fine? >> I am not sure the exact situation you are describing, so I asked you >> to write a test case to demonstrate the issue. :) > > > Suppose we have a shift-6 node having an order-9 entry => 8 - 1 = 7 siblings, > so assume the slots are at offset 0 till 7 in this node. If xas->xa_offset is 6, > then the code will compute order as 1 + xas->xa_node->shift = 7. So I mean to > say that the order computation must start from the beginning of the multi-slot > entries, that is, the non-sibling entry. Got it. Thanks for the explanation. It will be great to add this explanation to the commit log. I also notice that in the comment of xas_get_order() it says “Called after xas_load()” and xas_load() returns NULL or an internal entry for a sibling. So caller is responsible to make sure xas is not pointing to a sibling entry. It is good to have a check here. In terms of the patch, we are moving away from BUG()/BUG_ON(), so I wonder if there is a less disruptive way of handling this. Something like return -EINVAL instead with modified function comments and adding a comment at the return -EIVAL saying something like caller needs to pass a non-sibling entry. Best Regards, Yan, Zi