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 D17D4D20682 for ; Wed, 16 Oct 2024 00:00:21 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 5762D6B0089; Tue, 15 Oct 2024 20:00:21 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 525FA6B008A; Tue, 15 Oct 2024 20:00:21 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3A0C06B008C; Tue, 15 Oct 2024 20:00:21 -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 1E2AD6B0089 for ; Tue, 15 Oct 2024 20:00:21 -0400 (EDT) Received: from smtpin17.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 8C5E8A19EE for ; Wed, 16 Oct 2024 00:00:03 +0000 (UTC) X-FDA: 82677507738.17.DF2B323 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) by imf16.hostedemail.com (Postfix) with ESMTP id 4FF6B180008 for ; Wed, 16 Oct 2024 00:00:09 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="espuS4/I"; dmarc=pass (policy=none) header.from=intel.com; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}"); spf=pass (imf16.hostedemail.com: domain of dan.j.williams@intel.com designates 198.175.65.9 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1729036785; a=rsa-sha256; cv=fail; b=SMFoynFRs+9o9MKePXXmk1m56Nx5gt79Cv6yrdtN6hp1HBDhbUTpPYqbT1fU8MAelPYQ40 8ooCnMQ69SSuuhq45zkttcatRE5lPozXcDKJ7B1m6xe/GtFnmBHbnxkz3R0iCHtzantcAT /AVqiugOPBEGMi5wKhsKsr2vHdK7ENU= ARC-Authentication-Results: i=2; imf16.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="espuS4/I"; dmarc=pass (policy=none) header.from=intel.com; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}"); spf=pass (imf16.hostedemail.com: domain of dan.j.williams@intel.com designates 198.175.65.9 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1729036785; 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=wPZy4Y3QDP7U8QKWKZs/N98LGwTbdPqWry2iwa/Haeg=; b=xAPGQGvOn3KB/6SABkS0sQpO8Jw2UZgzPW+mf5Cg+rc9GcUqvZur6CJbw7DRZLc+r5pP2u zDw4mcjHjpO6YXYF0urW9gB+bq9Kf5CZUST02BcBPa1c1VDh0/7grD/UZIuCXcbT1ff+7L SawX9uIcEQ/Y/l4Hgo1gfiDk+Iyie8w= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1729036817; x=1760572817; h=date:from:to:cc:subject:message-id:references: in-reply-to:mime-version; bh=kfp2wLisexsEB9MBnRXUB5p/KoC5ijejx6TAb8ZLlv0=; b=espuS4/I55HTGQ9161M6mC+lcAPHYketrlqLatj5y12B5pS2E7RACiuV ycGjJaHrUtApB+p6lERz5un1znFmZb6YgPgcABRV6nAqhkHMlFbu4UrBd r5NBkwttTz9xgYSRcWKUxGZEGjEFA6rYWEDm6CxoGU0ZXgBiq8x7/tFGn cMrFTH0N/hdKLC4zwTm8obHufYKWk3sWjfNXvj4pS0PMh4UtPWP25Fvw5 Q4dh1KpZ0Qi1m9pbBGxT4J0v/B/CERBjIKzVwkYvyYxN42JM2+fLQSwek UQ3mswsIMMsiwCi1seK//ZK1h7ncYzTClRP31NV1xX9uKqTt9B8UOuuHK g==; X-CSE-ConnectionGUID: XYuJtpezTsuPzuOOVodkRA== X-CSE-MsgGUID: cpBw2fx0Sfm6BcsQrljdeg== X-IronPort-AV: E=McAfee;i="6700,10204,11222"; a="51000744" X-IronPort-AV: E=Sophos;i="6.11,199,1725346800"; d="scan'208";a="51000744" Received: from orviesa001.jf.intel.com ([10.64.159.141]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Oct 2024 17:00:16 -0700 X-CSE-ConnectionGUID: QtepHMjqQImO5O+vXIZGQw== X-CSE-MsgGUID: G3E6MyncQ7CSch2159epog== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.11,206,1725346800"; d="scan'208";a="115500091" Received: from fmsmsx602.amr.corp.intel.com ([10.18.126.82]) by orviesa001.jf.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 15 Oct 2024 17:00:16 -0700 Received: from fmsmsx602.amr.corp.intel.com (10.18.126.82) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Tue, 15 Oct 2024 17:00:14 -0700 Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39 via Frontend Transport; Tue, 15 Oct 2024 17:00:14 -0700 Received: from NAM12-DM6-obe.outbound.protection.outlook.com (104.47.59.174) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Tue, 15 Oct 2024 17:00:14 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=OUpldyKezIBJk3t168i+fJE5kgtAlGyHSaQgW3aDd3j6PIRWBPBAhRlX+4Gir3Mhla1cjMZCWxQux5h3gbd1TaXM6yIk/LekUV8e1HIAHjgXA0474Sl4wbW6uI/xlIA0SsVxA+h9UkzJVCGcMTOvfePojETAbol5p5+5x7jWkkNsTBzYmxcg5W7t26+TKPBkLiKOfs9CMCWbKFsJMggRG28GpgspFn+gOMuF967cLli0e4DPzwbth/c33eKTv7np4E4CZ68aWIutA25nezZWdII/4mI0FvwW7Y92dhO9Rxcoc2HdgG5+nuRAW5xaojLvFXcozmn9xs0BzHLUWZGbYA== 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=wPZy4Y3QDP7U8QKWKZs/N98LGwTbdPqWry2iwa/Haeg=; b=mT+dnDpiyJ2vaLeM46FtJOUOEnmOmrCid857AtixeW8oVnvBTOopRWCQzUgMxsJcAnmsszOHGDwtDlolzPqnrx+v/QprXdCERAH/A2cl9ahMzUVpr1mqyAJtSDu39X87qMVvJA7qfFmPPOoeuC/M04n17r2SJuocW0T1J9lONOpVepnxeKKd6ty5AKGTsWtjyDFl4vLlHnHTxZEbqz1SYXx4g+Pl585RlmI6wlh3x61giATQWkYQjFvzUR+hvuNNHqlwuePXrlVwbGQks/fpXYWfHysShyWMeZVgRUSAMW1j2xzY5ewy9wkmHCH0g925lQ39vKmAsMmVtulKaNMHiQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Received: from PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) by SJ0PR11MB5021.namprd11.prod.outlook.com (2603:10b6:a03:2dc::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8048.27; Wed, 16 Oct 2024 00:00:12 +0000 Received: from PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::6b05:74cf:a304:ecd8]) by PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::6b05:74cf:a304:ecd8%5]) with mapi id 15.20.8048.020; Wed, 16 Oct 2024 00:00:12 +0000 Date: Tue, 15 Oct 2024 17:00:09 -0700 From: Dan Williams To: David Hildenbrand , "Huang, Ying" CC: Andrew Morton , , , Guenter Roeck , "Nathan Chancellor" , Arnd Bergmann , Dan Williams , Jonathan Cameron Subject: Re: [PATCH] resource: Remove dependency on SPARSEMEM from GET_FREE_REGION Message-ID: <670f0209ab155_3ee2294c2@dwillia2-xfh.jf.intel.com.notmuch> References: <20241015051554.294734-1-ying.huang@intel.com> <942d18c3-f9a8-482e-a166-c7c9d6fb28d7@redhat.com> <878qup94jb.fsf@yhuang6-desk2.ccr.corp.intel.com> <3450df1e-dcb2-495a-8fe4-0a6e096429fd@redhat.com> Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <3450df1e-dcb2-495a-8fe4-0a6e096429fd@redhat.com> X-ClientProxiedBy: SJ0PR03CA0281.namprd03.prod.outlook.com (2603:10b6:a03:39e::16) To PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PH8PR11MB8107:EE_|SJ0PR11MB5021:EE_ X-MS-Office365-Filtering-Correlation-Id: 39b7cf02-e5b4-48c4-6462-08dced758187 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?TXvuZDmI1UCZSM27A/hjBiUKbQCFZvZ/LXblPNvtE2n2j8HohIzhY9mftHI8?= =?us-ascii?Q?n8QWkClMQlmWyB3yRjivS7rRzA71caAJ3oDfPI6SI5+9Q5z1jAPXViygqjAt?= =?us-ascii?Q?P0AW67s8K5PPOWbKkfPSlyC1vMt0pNsial6mIWeEdOlN+xpEdy5Nb1kaR1s+?= =?us-ascii?Q?ZE9/yJLxZJ5ihWVoM2m7Vqm5ER6gS70i/iHd15AsbmiSnrmc8SebyuI7o4jF?= =?us-ascii?Q?AfVprf0zvUO/8LjP4+nZOP0mUTGdJLT9VL2AEW3JeHKjclrRVzacZKcGJgB4?= =?us-ascii?Q?Uj+9N0kKxGmhr/W1BJm7QC2JI+n9mr7lCH3bijTw19KPuNP3s9u44F7+u6Be?= =?us-ascii?Q?QoCijhd5RC5Za1StbGv1MFDFK6L8ad3uk0XEH0ZaAz4zQV9QBtNrVp1joLNy?= =?us-ascii?Q?3oHdzVrU5Jci/jlIVXKV/jkFlCsp8y9Inq7UtGrfYXgX3wYOb7xjZ8i8xrFr?= =?us-ascii?Q?zjIGoafE3DSpyBcDtG5g4GRMW9nrFjF05DwmtwgrWLWQ9rNLBl5RXhWUU5VD?= =?us-ascii?Q?Len+3spvHubMyUgup/AB98jsqxY49zf8NlRfwsobnOac/d47TZz3db1dk7Re?= =?us-ascii?Q?USwk4t04vAXEyIMoQzqREgslCx8GgjWF1kpiTcuH+0iHcm1et6JVIhn4jP4e?= =?us-ascii?Q?igy16apoacPDETMHCBtpihCunRbzDf4DfZ5dYfb8VwjHriUzbxlAETjfrASy?= =?us-ascii?Q?0Pnv58HerScMuikW8qMtLRowBtTs3IgnjFCe42cjLXUSVqmBtUZCRYbb6C7Z?= =?us-ascii?Q?ESNRhsXdnfcbM+XB/xJD7HdZsYaT69yeVZtGDHkEQu30auHlgU1cFZtpFVWL?= =?us-ascii?Q?xO1vlZF7TJ29IlXaqHHCdCVWO5GiJ3gm/d2qj9Lo0Iv3AntsEJ0jTI6pE/nC?= =?us-ascii?Q?GSY6pjdr/wKM7H+F2SZBtG88x3PHhsxHifreaDqvPnqIvkqJP2c4ZXAuDwhQ?= =?us-ascii?Q?cGQRuuzdBjsGsd3harz4FtDtQyAEpsEj/gqHSAt4p1pWrL+y7U2UgxBXh/Mc?= =?us-ascii?Q?aucF6wzsaXbD/AiQ5jTT4gsTkSHisJggleMyyVLd9zauyOtSFEGPsVm6pR+9?= =?us-ascii?Q?WtRHp/T4KR94T+AS8zcKprs9Jf0n4Ol3R6g2kBrH9Xi2I6mDc7VRVHX7Ip1O?= =?us-ascii?Q?vM7vLgsYCET6zhaD9uP2kkVrgoTC7b1iLGwpkMultEr4Q9n3S7hgFRmZ/SLV?= =?us-ascii?Q?+7WoFqX6iVx0Cz0J4/sU5K8+YcoWhDsHonGJNWtJUuO8cqmJJYloh+YLThAH?= =?us-ascii?Q?ndfYP/qwkWtD1vdvi5dtpPYvj3Tug5rrPq5usx21bXfafvuS4w1FlwbA9fot?= =?us-ascii?Q?uVA=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH8PR11MB8107.namprd11.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?QLssXNbHDpHvlHawihzNMXGjIrB/tfBMGXlXEof4OnxXEvqGo0Dosqd15RZc?= =?us-ascii?Q?aTVbFp0+OMIWa4Y46nSpoVumkpW8izZcdYB/6CH7Kc6/TOyIbJxMcty/HMfn?= =?us-ascii?Q?aaiK7PrTzyOnCo+T65JU0jKEZvBdtJgySvZORZSJblyMMRETG9FvLtBD+IPz?= =?us-ascii?Q?reiJqB31iH517kr6ULbbvORmvrDdRlHzAvL9ULXTSyLRX4GwfWT//5MEuk8Z?= =?us-ascii?Q?bhPSiLOjDsZzlAOs/FfeKZ4cAlu70GPOGZ7IChvGwUJuBwV9cWeGakmjMriz?= =?us-ascii?Q?gtw4cga2/D+wj5iri/zkC8JKY180fdZz1m6OF9hZTtfybsoxOD7+51WxRo2E?= =?us-ascii?Q?8caeW7bPzk3D1Pat7wkXX5QzNIitlk9A/1o68aQH/fl5DzSIQascMkye6Qw0?= =?us-ascii?Q?IKHkBllzg6ZVmYw8WpInV9HsQnL9+9mwX7Sr0R1f3eYixLQAbYDiapQAXUv/?= =?us-ascii?Q?CUadvQxFABBBNqPm+bbB6IsLZQpUt3ebl+NNB5MVS3zeEEnNqxpLq1hbKUO5?= =?us-ascii?Q?tWHK9u+b05n+aDOy8lHlTBdwgo5JS1OjotRnn013+DL5/eIswRDXfXYLsqht?= =?us-ascii?Q?U+hR1Qw0jyN1m3BGRIGmyxLIvboPdZtV2U6pu+Ewpyhh9ebZIKAVV8E0UmYU?= =?us-ascii?Q?RUEnyyeAD+pnEQf3Ynn6PM8KcK0IgEaxqA7uL5xTHLF+vLKSpOBw2sS0QwOc?= =?us-ascii?Q?T2jxa4QzUuak4M/RwN6bHydHga7qPSM5QQC78QrAN9RRBpekPGA6poTrMNB/?= =?us-ascii?Q?MWTBqVjoRpa65J59MN+i2z5tgCvFxwHa4PP8afebtcn3p12IXuUw7Otnu+42?= =?us-ascii?Q?ZjFGe7u9IRaPNp1d7oHjr4YhXvyCTno7GE2WxiJWrfACqgikpu9CZklkaqkK?= =?us-ascii?Q?RbCD6FZUiOdzWuH9DWWhVt0c2kT6LOoA9WEjfCZ8Zrd37dl80mfv8drzed25?= =?us-ascii?Q?vb3unZNb3kqls4ptptsZcI0hkq/HzF7iSWwuFgxOIFJ9B9ytVvrjz4DZvxyk?= =?us-ascii?Q?X2eIk4OXM91ozyDf2YMX9m0lR9JrZNlVlbvi8+VMvDQjyyelbnY/HnoyfbTG?= =?us-ascii?Q?2MDc2G0BOyqwQ5x66pBcirGrYk258Ib5PjMPjgVUemCij9B5VFFn6oedtANw?= =?us-ascii?Q?JRwhGYRtjg7Ycail/WbgYKv1cXFGmSdIRWRvmDu836AbffGJZrb56XXBG1yo?= =?us-ascii?Q?eKE9/+m08G8xhPMRESHr7hYLf8MeYyCb6FQc0WSGaau1WMIdtOHwPu74A5QI?= =?us-ascii?Q?aO0PrMHxGKmE7UA+7R9QPeXJEZPlQdILKyVC8fV0jXXaHT4Pq4HCYNsmqELx?= =?us-ascii?Q?BqClDP/lMAUPnsfq5qbEibNZdgpgKu6ODvQCgynnrBCD0Qx5Sac5u2fTWdOb?= =?us-ascii?Q?fkY+mf7PfjTJdwP1WGnxYjMVJqSnHMLEkSVlOnvxFeK81aZpB9m51vGBovUS?= =?us-ascii?Q?vAbJZiBnYcozKF0WPJboB+rCWKet5hpQbazzyN7a6S24KjsGTcrrW0fbui1r?= =?us-ascii?Q?nyEk0B9i7FD2ww2g5f8gE/CgSv3jIBFk3nOT4ftDcOCMJIaAnKrgZGmil2aU?= =?us-ascii?Q?OWaVlI19pP3okePAYrie+AHQoBsGbBDwN4Vh6KpqNxtvdosuDMFkMX6qEstI?= =?us-ascii?Q?BA=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: 39b7cf02-e5b4-48c4-6462-08dced758187 X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Oct 2024 00:00:12.1602 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: dhCnj1vnkMdJLBsBOfP/7Pwqi6IrKxPMz/HAw4rDnjW85DRikvpmUj5BQAqldepoypVVYXMAnUpBmkhryUrJ5PzixIjPyA1T0StcmmUjwUU= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB5021 X-OriginatorOrg: intel.com X-Rspam-User: X-Stat-Signature: 19wmw4j467xd5i5mxocgq7cg3ds9js67 X-Rspamd-Queue-Id: 4FF6B180008 X-Rspamd-Server: rspam02 X-HE-Tag: 1729036809-268020 X-HE-Meta: U2FsdGVkX1+BTxM2VH3PIlpa0GFk9dnLpus7Wc2OMTq9eoMfc++5NiDsyA6oX2bzRQteDCUB/AtITrlOdj4ag2VovmMyq51qAJPwLvcuE2n1yziEyv6QnXs+eSwqhoEJgl2nH0H0jFGzhMdIY3q+bMwyr9vu5bhnhm7NjVFIruNEQSuUYZBzHVuzcQeVIm4Vdvo1hi7UoNxkFeB9qZnVjAWvl6+KqtxKHPjK7ECCpZ0V1aYNhe2Qmv+454gmK7NWMacXn4o4rLRkCXyvGZ4tK9tQt+YIVuWdUS1cderv47Ntbac0shCfqZLn1uebfJrlTVAdUOj9mNMj5mNM8VMduMhjGqKQZ73AlQd2668eCKxgWRXFjCGij1MBKtEkcl+Os3DpElMX4blpy5aFU5HJO59cPJYQMFsTz7Nq1hLzUvB6/gv49JCds0z1puEr+Qhgut0zAjV1CQ1SawR8pubsMZbyLPrnZ6WVNR8XgZZWHvOdt5xzRXf+P8BKSaGxVbH8TwCLhhApAMnDEL/xjY5Rhrdmd27ubVGs0xiVge3tDNT0XUOnLfnvPaNp84cpHUj5yWjcizkcFRr+wB3T+4CFXm2cNC1FQBfrTpkcFDlfOk9ucrc+r7WLcYLONvcXw3jFaYvM2vIfNz4869Vs03403P7DJ9xDJbstje8bLO4OsAGyzmTndyW8QBNAWkYcCbA5CRcKf/jjMQACfVR07eK3BQFAtp39G58lOEJOUkTgX9XVkLtK8NZyPveEpVaQvwsSlSbxaZOrcyEWtJdzLq3U9RsOyMRhjm5nbdpntH7tNcI9CE7tS/S4njrKGvKvaUv50ZpRr3WdBFE9Ki1nvUiB2PMaBN+0CeOU7wNOyeXwVzU7XG3q7eTEsgkYgkZsMX7w5Lypig49LOp6KtUDJTrFmjY+Nc8nhvaGkwm8LfZ8wrycYUKMREV8k616Emm71DaLHj+Iv/NLkrXGoeT6e0v tzC5qwl8 1MD8fYrMHtKCFvEAY5MXEh/Ib8pS4iPGugSGzyRjYUVlYFwWinolk1mmJWnohjAg6Z6yRW/q/Uapnf2dg2APdXUOno7EZmkPEGaZW3DkG2oG0eZz87i0g8soXjhMbtu6Eksa/+KNXKX9/6gC12r4BGA2f9SBXB6VrYiaRwYSbxVL3p6tMVwNIsKT+DUQheUqKBUtZfdI3YR+vExiq9544/dvngj1tnMVlwpqwxkrBw/4VV2BB5Y5P/lcNVm456Ec8fJLPkcskcE74XDimZlp9d5gki+31gd7HwLP6dK1Fg+knqlu34xJU+x27gIy5T78+VdS2StY1RUK+lJCcnsib3fw45eZ8ikmOFKJsWPp6EGbgQA1fFkyYfHwMJQuYLgXPm8ahc444YrDWLLWcbiC3zzsoR5b7M1bhXjEo6duBQT4NwsVnDJq16AwIZRZ0Ut5jJu7Ovk2NwuFp0Tck2VA40FhbpLC3hLT5MdHV75CLbKPKAeSHjE5wNESlMKSK8TOyvbeFVmW3ss5M/P8EpnAPsIfBLEmY7CQ6MPVF 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: David Hildenbrand wrote: > On 15.10.24 10:03, Huang, Ying wrote: > > Hi, David, > > > > David Hildenbrand writes: > > > >> On 15.10.24 07:15, Huang Ying wrote: > >>> We want to use the functions configured via GET_FREE_REGION in > >>> resource kunit tests. However, GET_FREE_REGION depends on SPARSEMEM. > >>> This makes resource kunit tests cannot be built on some architectures > >>> lacking SPARSEMEM. In fact, these functions doesn't depend on > >>> SPARSEMEM now. So, remove dependency on SPARSEMEM from > >>> GET_FREE_REGION. > >>> Link: > >>> https://lore.kernel.org/lkml/20240922225041.603186-1-linux@roeck-us.net/ > >>> Signed-off-by: "Huang, Ying" > >>> Tested-by: Guenter Roeck > >>> Cc: Nathan Chancellor > >>> Cc: Arnd Bergmann > >>> Cc: Dan Williams > >>> Cc: David Hildenbrand > >>> Cc: Jonathan Cameron > >>> --- > >>> mm/Kconfig | 1 - > >>> 1 file changed, 1 deletion(-) > >>> diff --git a/mm/Kconfig b/mm/Kconfig > >>> index 4c9f5ea13271..33fa51d608dc 100644 > >>> --- a/mm/Kconfig > >>> +++ b/mm/Kconfig > >>> @@ -1085,7 +1085,6 @@ config HMM_MIRROR > >>> depends on MMU > >>> config GET_FREE_REGION > >>> - depends on SPARSEMEM > >>> bool > >>> config DEVICE_PRIVATE > >> > >> Added by > >> > >> commit 14b80582c43e4f550acfd93c2b2cadbe36ea0874 > >> Author: Dan Williams > >> Date: Fri May 20 13:41:24 2022 -0700 > >> > >> resource: Introduce alloc_free_mem_region() > >> > >> @Dan, any insight why that dependency was added? > > > > Dan has explain it some what in the following email, > > > > https://lore.kernel.org/lkml/66f5abd431dce_964f2294b9@dwillia2-xfh.jf.intel.com.notmuch/ > > > > This is reachable from the "Link:" tag in the patch. > > That should be part of the patch description then :) That Link: does not really describe the history though... The description I would add is: --- When get_free_mem_region() was introduced the only consumers were those looking to pass the address range to memremap_pages(). That address range needed to be mindful of the maximum addressable platform physical address which at the time only SPARSMEM defined via MAX_PHYSMEM_BITS. Given that memremap_pages() also depended on SPARSEMEM via ZONE_DEVICE, it was easier to just depend on that definition than invent a general MAX_PHYSMEM_BITS concept outside of SPARSEMEM. Turns out that decision was buggy and did not account for KASAN consumption of physical address space. That problem was resolved recently with commit ea72ce5da228 ("x86/kaslr: Expose and use the end of the physical memory address space"), and GET_FREE_REGION dropped its MAX_PHYSMEM_BITS dependency. Then commit 99185c10d5d9 ("resource, kunit: add test case for region_intersects()"), went ahead and fixed up the only remaining dependency on SPARSEMEM which was usage of the PA_SECTION_SHIFT macro for setting the default alignment. A PAGE_SIZE fallback is fine in the SPARSEMEM=n case. With those build dependencies gone GET_FREE_REGION no longer depends on SPARSEMEM.