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 1C558C3271E for ; Mon, 8 Jul 2024 09:31:08 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 706EA6B0089; Mon, 8 Jul 2024 05:31:07 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 6B6C56B008A; Mon, 8 Jul 2024 05:31:07 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5570F6B008C; Mon, 8 Jul 2024 05:31:07 -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 388006B0089 for ; Mon, 8 Jul 2024 05:31:07 -0400 (EDT) Received: from smtpin30.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id AD8C41C28ED for ; Mon, 8 Jul 2024 09:31:06 +0000 (UTC) X-FDA: 82316066532.30.18C679A Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2056.outbound.protection.outlook.com [40.107.236.56]) by imf08.hostedemail.com (Postfix) with ESMTP id C85A7160026 for ; Mon, 8 Jul 2024 09:31:03 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=amd.com header.s=selector1 header.b=G3OW9PTC; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf08.hostedemail.com: domain of shivankg@amd.com designates 40.107.236.56 as permitted sender) smtp.mailfrom=shivankg@amd.com; dmarc=pass (policy=quarantine) header.from=amd.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1720431035; 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=fvNjQ1voaTq+R8yyKsHLvOUSfWKliSRfW5ISoRF+KFg=; b=nnySkNk+QzTSTmQvXNw12/pjuDxUfZcrx2am+eDiiWl3I5dI0HbJr1+eqdi1DHEIIkIPaC VUpZggID8+khAZFHfpDVLCU2P8Gi3orjXiZI99jWXphQoCrU25gPs7V/RhmwJFW5s2L3J6 9wjtvueENn2YxXcjMBR3vj+3F/H+e+M= ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1720431035; a=rsa-sha256; cv=pass; b=6vfWkXlWKSawo5AciQT5snoyffOdbasBHpFT7SgNmhwT1oHRxRFvlRu8oKQi+8XpsiutvY Ku5o2ayGMiE80SzaNkccB83qu29I90anZTGRnGMDdswHtMANVHTWuF+zNfqu1AgTzzKTY4 qWpz6+D7jDras4lVkIQjS0PVIgMauOk= ARC-Authentication-Results: i=2; imf08.hostedemail.com; dkim=pass header.d=amd.com header.s=selector1 header.b=G3OW9PTC; arc=pass ("microsoft.com:s=arcselector9901:i=1"); spf=pass (imf08.hostedemail.com: domain of shivankg@amd.com designates 40.107.236.56 as permitted sender) smtp.mailfrom=shivankg@amd.com; dmarc=pass (policy=quarantine) header.from=amd.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=N1TRUwXaDnO6Y69tA7OsmGoxtD0gP/iNh8qs1cobKIU0RSasI3Fjkp6VamHSBRd+9y1oTtnWjaXRN5l2hptPg8ZIRtCWYOkhXyvhz2Lcdj3VVUWUQWnVVXmsuI8fngFLhmV3H+tBN5SWzRsYD/GNeoK0EkLfcbqI7Sjeenk3xsg1FXDniXcJtaN74ghidqUq+3RocfDksw10W0yV/jFNXBH/Ud8J9TI4B8R176POUOLPqH0mBhn4uENmrGOIj0FzZIqJ5fY0HvhugnSyxr12CbbX5m8/VhBHrFpbZxS9aK8zds7C3ik6u/npybOUzt6utm1BQfF9tVQTd6hrGkszqw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=fvNjQ1voaTq+R8yyKsHLvOUSfWKliSRfW5ISoRF+KFg=; b=akHfYEcdJ6rROTCdS5fcqQ20aOWwZMDczvzC4TsJLA+7vncAS78RFRJbTykfr5TNPPN2r2XLsJdzDomf6x63Gk4lrHRBSOkkhO7E3RNIoujEq/e/AVMtX/R5q7/hrQM96M4U7ueiNpfCnaigToD6TVVtS93p/S415CMYhPRsoH1DpFWMF5EB7E2rR3WTF0DPqSf2YpP+qMIcW9FIMQnCUdWul/f0SHjrGUNqgyMQhoWhKshy85JSOogbezYJ7G2CyMTi4D64vExdv2ot1pWASo0x6XBvI+7BFPE3530shz6uAV1szlJowVRZKqFk9alsNl+L4mEVVRsBsjHjYt0nhw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=fvNjQ1voaTq+R8yyKsHLvOUSfWKliSRfW5ISoRF+KFg=; b=G3OW9PTCdc043lbGQDTrImfenhYv9nLODMBDTfiF5ZL6pYYoWoHZB3YsOKXhnuS1r4yTkvaOmrmqNTbBG2w84zlA0ud7iI7xbQ53QfTy3TXtrQaZhAABzuaOUhwyNSaa2A7IRb4OdU2Cf4KLsIpxR4eXkxTskiAhAF3HDaEWrYI= Received: from CH2PR12MB4262.namprd12.prod.outlook.com (2603:10b6:610:af::8) by PH7PR12MB5928.namprd12.prod.outlook.com (2603:10b6:510:1db::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7741.35; Mon, 8 Jul 2024 09:30:59 +0000 Received: from CH2PR12MB4262.namprd12.prod.outlook.com ([fe80::3bdb:bf3d:8bde:7870]) by CH2PR12MB4262.namprd12.prod.outlook.com ([fe80::3bdb:bf3d:8bde:7870%6]) with mapi id 15.20.7741.033; Mon, 8 Jul 2024 09:30:58 +0000 Message-ID: Date: Mon, 8 Jul 2024 15:00:50 +0530 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] mm: use zonelist_zone() to get zone To: Wei Yang , akpm@linux-foundation.org Cc: linux-mm@kvack.org, Mike Rapoport , David Hildenbrand References: <20240706015044.27789-1-richard.weiyang@gmail.com> Content-Language: en-US From: "Garg, Shivank" In-Reply-To: <20240706015044.27789-1-richard.weiyang@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: PN3PR01CA0120.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c01:96::15) To CH2PR12MB4262.namprd12.prod.outlook.com (2603:10b6:610:af::8) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4262:EE_|PH7PR12MB5928:EE_ X-MS-Office365-Filtering-Correlation-Id: e0935374-82de-4495-994e-08dc9f30ac98 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|366016|376014; X-Microsoft-Antispam-Message-Info: =?utf-8?B?OE80VTk1ZEg1NnhpbHF6LzBkRGcrYzk2TnQ0d0JuV0RleStiRDk3OS84ek96?= =?utf-8?B?NGh2ek5DRkE2TStsNkV4bVR0SENFK2VoS3JBNkU1MEtyekFib0R6MmhvSlJK?= =?utf-8?B?SUdpc2RDWFcvMlpGNnRLaktOU3FGRkFwN3JZVGNVKy9DRm1jTmpKeSs1OGFL?= =?utf-8?B?OWp4UW4wQjRJaERXYVR2V0VjMUFHb3hqcTh6OVRNU2t2YlQ4ZkVGQ0VRck1I?= =?utf-8?B?ZCtsdjBYemN1Q2ZpaTVROUNFbTJIZ01xamdVMEp1dWxtdlF3dndIL0JFUkx2?= =?utf-8?B?clFKUkpzcEtlWWE2UmhrKzE1bGZobWYraURTb2lDbGk0RXBvQ2JvRFBYTUJ6?= =?utf-8?B?ZmtabEdBTjFIbWtQNERUZ3RNbGRrai9uMzVKQ2M2bVJBR3VBemt5VTJSN2hQ?= =?utf-8?B?MVI5VWxWYmdBckozSHZBSzM0NzVUaXE4MjZ1SWxBNmp5NC9iT01ZejZEdTEx?= =?utf-8?B?ancxZVNSVjNhR3NNanN0bmtpL1NSOEMreHAxMWFDTlo5OEZPNnY1R2VtbVJ1?= =?utf-8?B?TUExZThFVFZhTW4xSFdVS2RveVhKYnZld09iUUcwaU5wWUpxeHR0cGhqSzFX?= =?utf-8?B?eThQMEFUb2xCS2g2QVNFbC9FdUdoL0F5NkZlU2tZT0JTcTlQSHlTSFdrampK?= =?utf-8?B?YnNyVkh4SDFoRDlHNTVlZkNoUTJlRHZVNEVLWXBPeEx1OXJLTU5UcVBXNXZ5?= =?utf-8?B?dDV6cW4zSjdObHJBdGl6OXJGa1c3YmMwNVRiN3VUcVdWQ0pEd2JFVUFST3Y3?= =?utf-8?B?aE90SHoxUlVuYVBNTXRJT1RKU2NqZnRUaUsrNFo1SnYwMitQeXpIQys2OGN2?= =?utf-8?B?d3lUTDRRSUNDZDZZN3FCZHZGSUVoSkF2ZUJrSkJkVkIxNkdSMy9pbmZKc01h?= =?utf-8?B?NXVTN3ZNb3VVak81WUhyZTlPMFVtZ0VoN1dlVmRhVzBsN2U4bHFCSEpGUVVl?= =?utf-8?B?VSthQk1DdW1QTi9yb29jUGxuNlQyY1JuKzRZdUFZMVhPMUx4amVyZ1hlS2tS?= =?utf-8?B?RGxwWUwxL1F3elk1UFN5aTdtWFdvZ09qaGFMbVZucmJWaE1JbndKYjZsQ1pQ?= =?utf-8?B?S0FhVVpiRVJxUDRkOFlxNFNkSEU5MlU4UWEzdy9qMm5QSXIyd2M1S21BQ1NU?= =?utf-8?B?TmFNQmwveElHOFNicTJyanREOGppRGRZck1lMzN3ZWVlTmRFZ2lxWGpoeHNQ?= =?utf-8?B?NW94RldoeDJ5WnRMdGpDYWhkdEI3UXZ3aGp6UDBXdTR0cWVwZ3VwcDVaR0pk?= =?utf-8?B?aFlQSTBlVVFXTnI0alNCUll1bW02bGVMeUdNeXl4RVI2MVR4enFmMm92eTBU?= =?utf-8?B?UWlIVFNHaVpDUzlMTFJTVXozOTdMbDU3S0pPcWxOeDRoeEpSSmgvNVRURkxU?= =?utf-8?B?ME1oRlJoc3duc1JOVTdoQjcvek5rYnpsdHhMcUFUQ0gxdjFPUE9qUEsyMitJ?= =?utf-8?B?Nm1kMzJCTlJvcGxndWowY0lOTDJYdjIxWjZCQ1ZBdUhzc1FRanY1Ti9iS0lt?= =?utf-8?B?WkdUYkJucXlyaVA0cnlZTHhmdlk4VUp0ZGRtSzFHMFg5cGhnY1NrOXpSdURk?= =?utf-8?B?amZNay9GSHpndEFNbDNUZzZ4bExNN0ljUGtndTlNOEF2Zlg0MzRUdjFhSnJ3?= =?utf-8?B?VDRsMVRIWjYwS3k0WExKVzhTTHlzWWRUOGh0enJyd2ZqTmtVY3JBMWRtTlh6?= =?utf-8?B?Y08yYitrQlU3SVp1TWI1aWs1dGp1OTRnUXJOQ1F3ZVJJL2RiWVlsSkZCb0dC?= =?utf-8?B?TmxoNHdYVGJsSmtDTC9yb3ZrWWhORDR2cGsxVmNRNXNnMG5FVkd6Y0RCU0ZU?= =?utf-8?B?RzFCeDNLNHlXQWFRd0xlUT09?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CH2PR12MB4262.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(1800799024)(366016)(376014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cmRXZEVyMUhKL0JKRk56aDlFMVBuYjdITXArNUZUMEJhTDgyVDFRbWFFL1Ax?= =?utf-8?B?b1NKYjB6N2xjekJqbjJLMVFuM0dJWkJ2a3ZLMzFRbkl0aFRPZTMxWEd4czJN?= =?utf-8?B?K3VuZ1Rvclo3Yk4xYno2Q0FQRjFRdGdMeEF0Y0VOZmxoS1lnbUVmSFpMbFFo?= =?utf-8?B?VStBL2J1eVpvQ2M0dDJLOGZDd0Yva29ENUl4Sjh0MWl0bGxzVUhKWkxlaXNN?= =?utf-8?B?WWxlcENuZmhjS1BQOUJGWlh4WWhEWHUwWTR2cE1uakIyb1ZlRTFjVExpenl1?= =?utf-8?B?aXlNRzhKK1FIeVFVSFdwT25UcmRtN3l4SDF5clJxNmYrS3hVb2p6TzdtSTZJ?= =?utf-8?B?N1FtV2I2L0VLS0tKUlJsQmFEbG1LMzFmWkJjK01LaEpJMUt2QTVUcldldXVh?= =?utf-8?B?WHQ0OFRhcEZ6dnNDNjlVNWVEU2N4eEpsdHRwOWFaMWlCYXRWQmRVZ25sNmRw?= =?utf-8?B?UTJYVmt2ajdGN3RncUJPSEhKSHRFNW9Kb0NsTG1Lb3hHWkFyWmRwK2xJTFJv?= =?utf-8?B?NXpZMEVtM3NTL3hmcWR5bmtySlAvMG1QeTdBUXY2V3NNd0xvbWdsV1g4Vks4?= =?utf-8?B?cHBMcXFsSlFCeEx6Zmxxems4b3FpOUdQMHJxZ2ZLZ3NtRElCOVZkUGY0cW5q?= =?utf-8?B?K0NoRUY2NGFxS2Q5NnJieDAxQTB1ZU8xaEZXZVFEYlNTMVhSTmxQcUZ6TGFG?= =?utf-8?B?RlZKam0xN0JGd09HMENOMHBodDhCcDlUOERBaVZZKzZ4UXFXQ3NCN00rdy9v?= =?utf-8?B?SmlRQjhyRTFJdHpCSDZaZy9WdElUbGtTZjZCSDd0UXZBK0RpaEhBS2RvdVJC?= =?utf-8?B?ZkJTbHY3TnE2RGdJczVzZWhiOFROQUxJcnRZbmVLd0Z6SFF6cEhOL0VMaEJ1?= =?utf-8?B?UDRTU2ZIOFpFcUVCN0NFTzR2MTlqOHlFV1pZeVNYZHBuQnBoOXRhQmZCdnQw?= =?utf-8?B?TXl1MXVjdEdRMmcvM1lBM3BZZXVFNFdYeEhvbVhQVG9meWY4d3ZYWjNkeUp3?= =?utf-8?B?cVNSb25YSzEzcFlGbDhUWU9QaG9RQUY3R29uSlBBbU93R3lEYUNqaTRNOTZP?= =?utf-8?B?YnJ3eTF1WHBZV3Z3N2s3NkZDN1dsbmg5NlU0dGpOVkUwZTJnOThINlkrR210?= =?utf-8?B?M09LVHNlK0VJVlhtZ3ZCLzlFa3FVcm1TWU9vbWx0Q01tZUtSTzRFc0JlYjZY?= =?utf-8?B?VU9pRk5JS1JKU2t5RUxXVjE3YTMzeDlpMTIrK2lycmI5Rkd4SlBmQlRCL0Rr?= =?utf-8?B?NXllTmRGa3libXEzakdtOG5Ua2FHR0E4S0NPekEzNnFsM2M4QXhYZFhtV05p?= =?utf-8?B?VEJValcyTWhJN01PZzY3bVhZK0ZabGVNVDVmZEdySkN3QTRKSkpUWVNkejlB?= =?utf-8?B?bHVpOVJEcXM3Rkx0dE1SV0JpeHNqN1ZFZDlsckxDQkxFSVNHRExsUTRoNG0y?= =?utf-8?B?WWFTTmpGcmRXbGk0Q2JENFdQcWZNZWFqOFpHOXJOQ09MWm1Ia1RHS24vcEU0?= =?utf-8?B?cVpBc3pkNjdyVGQyMlovVFZ0Qk9wRy9rYUxRcWlYb3Avb0paUkxXWXJraG5H?= =?utf-8?B?Zi9GRnpWL1NsUndRYnI3YWc1MDN6OGF4UVBTdGtybEdOQ1FpQjFodXhVTnJ5?= =?utf-8?B?TEtpeEVVblpuRVh1NFVja0V5Wko2a0tPM0NuM1dwTEdtU2NxTEl1b0ZDV1hr?= =?utf-8?B?ZHlOb2dHY3I4TXBpQlZwdWM4WkFDWk1weEhwZEZkZUdqSFJnT0F2dU56TVRz?= =?utf-8?B?TTJyNUVldkJDSk5VZ3ZMWlAyclUrM01jdFJqMnI2dCtGWHNBbmJ5SHFFWVpT?= =?utf-8?B?dGFKNE5SVDAwcDR4RG5Zc3JLWThvK1Urak41Q1RSaEtlQXE5enVLclZld2NB?= =?utf-8?B?eFpITmhNKzJJZFRxcXF4dWJPelIzbi9ndklKU0hBVUZ6WEZSdURXbmtGMURj?= =?utf-8?B?L1o1OUVOd0kwcDNHbDJ1bmdBS0l2b2UzL3FXWlcvSEFmMXdtQXRxZWxWZDlo?= =?utf-8?B?K29OSHNtMlFINkd6WWgrQVQ0OWdUL3R2cS91UmRTZ0xWcnVyK0w2SENmOUNr?= =?utf-8?B?WFNCWmxQQm8zT2RBRUZYQ1llOWh2M3ByQ2JFOU1CbGMwaXRXOTlXMGZzMEc1?= =?utf-8?Q?EE8ffgwuVHHf3294GvQ4VgIXE?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: e0935374-82de-4495-994e-08dc9f30ac98 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4262.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jul 2024 09:30:58.4988 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: bO6gAL12Hgxb35bysJaBVy9VMhn+8e8o4SfPtG2T9o86zY6kb23edJKzM98WJ90zeU6+5V5/islKCovLllVaVQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB5928 X-Rspamd-Queue-Id: C85A7160026 X-Stat-Signature: bc8crfpcd3qzuo8fbxfg5mtxqt8x98ek X-Rspamd-Server: rspam09 X-Rspam-User: X-HE-Tag: 1720431063-233772 X-HE-Meta: U2FsdGVkX18aAmqS1JcSK10pYwjgIqxbSGcWEYfZRpDROyFaNR7lqdYFGwkyA7ZEf2thlozXyinH0YPSUJ+BW1rzZ6Q9TcI9Qv5bTT8uNiBkHPaOFKCOSZniRSszfEV7Yywx5Fx1mMlzK4uW3QuVRBIPaDT9HCnuuD5RY77nAEwgGnBPZspzrCnlC+lgRxAvtbPjueL56xtb7ws9WUm0PtaKIniwObUNihtyFn6YXV3CyAgAi56W2u4jjnrzNSeCZhwqNZ+O8KhWb9hXxQQ5BzfnUQ0ysrdDmeeUDbvt1F8+qZcM6U9x7VotP2gYWS9gAh0oQV1bTHT7rsPwl4HMKP2TqqFowyaXHt0/VrEjCqjzvOazTVYOHJFh6ONb5ILR70L1riYDiGOtVooXJUqwhcC4t3RXN/tqVKEfN7i3UqBkNIf5DvKO2RnZGYnww/2sLwdVas6hRIPjIwjIq+Y0WLh+Qniyn5FQMJ/Clvp9wSPktJlQOvT3INFhuATxBf+5+7Gk8bYUsMvmicZn9UGDj+G4vHz8bIAqPQXpO8pparqe9KEj3bMufA3d9Ye+3m1HODzAFOL4R56FcGPDP/hsLXdhbU3SHeDzo671FvxoS/xrA0voIf1mqLJOfHf4JnyLF1XyhPtW1bloAVc0rcUqcoTrHkCM1B+G/d4aXbq1wt2vXHo9qZKXCX59UB2kxMSiEnQYLjrMX8/jso7aI8PXTabHteis58bSXluuZ+80i853nkHCtIl1p2X7M/J7Ecl93WWY/L8VFUWCG1EPujjO0X99PCf6bvVXvc68mLtzHdVKDpp2zUcMV4NXI8ciI5jlhk17m6dprAjQ+IlRCX3BoAdPR1C3cf7zCIA7qmgaMrW7Mb0O+OK2XvtanMOCFP+/z22aDkeEVf4g7JLUjCba0PVW8vY219aWQboAIPMgYCSqtJ9WRQHlGP+yd6EHBV74sBSjJKX944H4zmZHUg2 93/xb7tS OZca4pZrjpXKSCBZ33JK2jhi9gQ6NLOi22arhat+bG3vIKYBgxcZ0YYwvkioZwMXzzlR7yqFClIfTkjmlVQfUGtLkTdUxh+qrwy/qJJ1yUNdBuaQfofDYaDJCPeBGWrAAoKnhZmYWIrMER7P7SMm2DJg4skH80JhNOKC2lvLEp/lpjyhBbhgSGGzkBBm2YE4wa/7SDJ0VkLare41vb3yOMajVk5y0CyRNFN9KulXhKW7elAQ9HbCEG0AYxzsNzfVTDIlphJ5VRk6oILgaTbPFNX7J3oTB2Dicmx9xDXaXfX4hdrAGZ1ITI1A8lxnf9A1JNP+ps6PQuRYrm95lkEQAlp/gl5Db8cM+zk7hcJkdoH3/wdbcMxNV5zzD87J56NK6/XNMkqn0+3cxcb+iBsWaVBmSPZjwI0Q4shNdK/8MiSL74Mp6JcI3IzXi3dglxixh+Gy7IbEiJSUaUS3xj1vjZEUh7tDHoNnqLEDrCYbLwYA91S6STJG1v4/PZCiuo2pcpGty 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 7/6/2024 7:20 AM, Wei Yang wrote: > Instead of accessing zoneref->zone directly, use zonelist_zone() like > other places for consistency. > > No functional change. > > Signed-off-by: Wei Yang > CC: Mike Rapoport (IBM) > CC: David Hildenbrand > --- > include/linux/mmzone.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > index cb7f265c2b96..51bce636373f 100644 > --- a/include/linux/mmzone.h > +++ b/include/linux/mmzone.h > @@ -1690,7 +1690,7 @@ static inline struct zoneref *first_zones_zonelist(struct zonelist *zonelist, > zone = zonelist_zone(z)) > > #define for_next_zone_zonelist_nodemask(zone, z, highidx, nodemask) \ > - for (zone = z->zone; \ > + for (zone = zonelist_zone(z); \ > zone; \ > z = next_zones_zonelist(++z, highidx, nodemask), \ > zone = zonelist_zone(z)) > @@ -1726,7 +1726,7 @@ static inline bool movable_only_nodes(nodemask_t *nodes) > nid = first_node(*nodes); > zonelist = &NODE_DATA(nid)->node_zonelists[ZONELIST_FALLBACK]; > z = first_zones_zonelist(zonelist, ZONE_NORMAL, nodes); > - return (!z->zone) ? true : false; > + return (!zonelist_zone(z)) ? true : false; > } > > Hi Wei, I'd like to propose extending the use of the zonelist_zone helper function in several other locations. I've identified the following places where zonelist_zone could be used instead of the current approach: oom.h:58 __entry->node = zone_to_nid(zoneref->zone); mempolicy.c:1956 return z->zone ? zone_to_nid(z->zone) : node; mempolicy.c:2805 polnid = zone_to_nid(z->zone); page_alloc.c:4205 if (!z->zone) page_alloc.c:5285 return zone_to_nid(z->zone); I hope this will further help with improving readability and maintainability across mm. Thanks, Shivank