From nobody Mon Feb 9 20:58:49 2026 Received: from SA9PR02CU001.outbound.protection.outlook.com (mail-southcentralusazon11013029.outbound.protection.outlook.com [40.93.196.29]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2EE2A32825B for ; Wed, 24 Dec 2025 11:21:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.93.196.29 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766575296; cv=fail; b=SXKaGb+rtGXibGidXWS1E/+MZ8BCs1DxE3PZYu4yBZzY5BVYWy+8UPqnsMkDrxQdX3/Jipsamz611LgtrDLuVny9UossoE0two+xud50/dCpYyJbJgxxBDHuGMZELHcE/XuVh7fZNZsArAZnDZWOXdGu5GnzehPe0Un9/aad1pk= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766575296; c=relaxed/simple; bh=8srrG3TJiXVBT3BYZZ84bRzRT9Yw5h5+cJU2u56oz/U=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=EdzC1/wR4OoHaRUjDPRBIzDIy1D55Bck9tbmpYBAXzFnuN6CeIjE8bMO0FpiViItN9Gn0OYZNZMzXIG2K2MQgT7JmE83j/7rCl+/NoGiAvTxrWM/+7hIwLh0Fi400N4Duna6jyJoeNKpUXUJSeIoc03LbDabIlhjGsTUX53OwBo= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=CkF1hId4; arc=fail smtp.client-ip=40.93.196.29 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="CkF1hId4" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=oDS5IpmPTbfKOs4auYWOBueg5akgjkR13EsnBsb1UXR7E4JAvkS65VHbWc2iHw3rtfOxkyBRVrxIYOcl17dn/dGBjimimEdizR1YoNffJxHD2HLJZeQVLHymkpev4KqaDX7ctH2PXviF2QCuVjTbE9D0djSsOHFayuuU+esbO+1ql29YZl6o95e6IgnOEUuUClXW/NA/+TixdnLCeeFz6XrrqqoiGWDgi0/ap7/kkC6/RHcddHO+K/vDdSiXAHatMJpMiAJHJw7tBs+HZVC17X0d6MO0SO7g/etD0x64wmjdLNSQxf6m5Pa3/wqcryLcEnP/T1N37GfcaCLNvs4FJw== 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=L+XCQmbTtTpVGHq/Y77lT5EOkvPBu1bacPRFVKsUP+s=; b=lh2TpU1Qtvd4zRYAmTDJHcckHsOFgdbRtgBJ11Lwfc6ful/Ka73mv1vBkNiXsw3Whj6NH8b6zt4EV9bM8KsBF+XxBH1Vqdnxyfxwi+yYxBSFfCk9s4SBrxvCffhk74toa+2ANfmnndctFLi3+wJ/1dZoyR3E6rq7YXtB95GVqT0wfrv0oePeIheZATj1eK7tT2oqTp77hYjwrkk4CWDxG4bO8XuHQ8Z/2B+8JCrtaBcOwivNK7/Wyzj6V4NdDtvpJZxaSq6oac3gZUwdW0vJ4JzvcRIJLo0WZwkp0Phob+aL6v2YsJBnEmnkU1S1ug1vgObZAraa/hReVO55j/BRGQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=linux-foundation.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) 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=L+XCQmbTtTpVGHq/Y77lT5EOkvPBu1bacPRFVKsUP+s=; b=CkF1hId4fyMnmOb13rnt38pyFD4lqQrUO5oeKGmIyVrkv71fUU8/W5aOUn99w/mE7OegPq3297ICWZFCei1KkEqkzTepsbExEcNQdat7uZ44x5gWEGaoEqD8RNdkXUJLWPTuHJ2yHEwA0H83f2VZ8bHjiggBZvXGSRiRgb+CfV4= Received: from BL1PR13CA0088.namprd13.prod.outlook.com (2603:10b6:208:2b8::33) by SA1PR12MB9247.namprd12.prod.outlook.com (2603:10b6:806:3af::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9434.10; Wed, 24 Dec 2025 11:21:29 +0000 Received: from BL6PEPF0001AB55.namprd02.prod.outlook.com (2603:10b6:208:2b8:cafe::ff) by BL1PR13CA0088.outlook.office365.com (2603:10b6:208:2b8::33) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9478.2 via Frontend Transport; Wed, 24 Dec 2025 11:21:29 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb07.amd.com; pr=C Received: from satlexmb07.amd.com (165.204.84.17) by BL6PEPF0001AB55.mail.protection.outlook.com (10.167.241.7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9456.9 via Frontend Transport; Wed, 24 Dec 2025 11:21:29 +0000 Received: from kaveri.amd.com (10.180.168.240) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Wed, 24 Dec 2025 05:21:25 -0600 From: Shivank Garg To: Andrew Morton , David Hildenbrand , Lorenzo Stoakes CC: Zi Yan , Baolin Wang , "Liam R . Howlett" , Nico Pache , Ryan Roberts , Dev Jain , Barry Song , Lance Yang , , , Subject: [PATCH V2 4/5] mm/khugepaged: use enum scan_result for result variables and return types Date: Wed, 24 Dec 2025 11:13:57 +0000 Message-ID: <20251224111351.41042-12-shivankg@amd.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20251224111351.41042-4-shivankg@amd.com> References: <20251224111351.41042-4-shivankg@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: satlexmb08.amd.com (10.181.42.217) To satlexmb07.amd.com (10.181.42.216) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL6PEPF0001AB55:EE_|SA1PR12MB9247:EE_ X-MS-Office365-Filtering-Correlation-Id: 44f92967-e8d4-4b0a-1b1c-08de42de95b8 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|36860700013|82310400026|1800799024; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?yZjeuE3GwdAQNW6019A7wGcsJaaJD0jxuIZmPLKuXKOM2U5Ywx2Gszs1RLxY?= =?us-ascii?Q?9TUADDJ/lrRmVgGJA9jfFExGFkje+NxJNDhQjEJZeVbsKUvFUHIh/nKYmxbH?= =?us-ascii?Q?waAVH9AxIBpDV0k76ALng4bWOreCiPaUM8LOgEqm0z0e2FozaXMN/Uzd66/9?= =?us-ascii?Q?pO1U003RcjpA1PgzdlY1wG7Efbgj3kiU5Au5f32bwmW9v5XgeJkrGJPiR1+u?= =?us-ascii?Q?KEFeGwoSv5l8nu1hjmIbF+qWn+LAcnOT0pf9hd8yP3XJKlIl1XGoRpW2c8Iv?= =?us-ascii?Q?SgO1hIpJddt788xMAz43B7jatN803gsjFrieUoKbtUMW5aOClHKdkCbBCbot?= =?us-ascii?Q?aO9k8Z2zvFtURCd06zE+326Axq/iE0Av6U+Kxpr1o4GdOC99mexOwSyGFvAl?= =?us-ascii?Q?WXnxc4NqhHrxLwC+GIHoaeQTq3tNQ1veTnG7eHgEXsaAZbhYZCslKVonjH+9?= =?us-ascii?Q?yo7a/QmbUGRpH9OKj0WFDIfUvkd+opkeOBpTkKJAnX7/NRqhUszByZajRk89?= =?us-ascii?Q?O9e8pp7NvMQx16E3lWg/679yZ1SdbmvtN0KcSKfc29V3gabr2i3HNqKEDXm2?= =?us-ascii?Q?NPo6FugvhNoZZcD+aWpbXB5wLZgQZHUc+88dlAzL/x6J3l+uNvPcve/eEuXG?= =?us-ascii?Q?AUSr8+kXRkejDJ75w72dIRWl5BHgD07+Cob1GNfktK5uT6Vf30doPMTT5S6A?= =?us-ascii?Q?sGycw0J8TXHlRfvL03/psq4t6NEGjMo5ZCAG5BT4SatBDFvDkRH7CA4Eatn+?= =?us-ascii?Q?poSYKb3aYgWBsMCdP9WyUicOJ/RzoWd2s5+7cyHwEKjss3def4Lavm2Yfflx?= =?us-ascii?Q?nLRF8uctxX2Q8EdGyJ1jBNfTHbrPe2tP7doQjSLXr3Uxl36ST/UyTa9S56xW?= =?us-ascii?Q?Rus5EIIroRajjiRSfjuLuXyyq4nMjdxgx1OSXVLYskYwUtadbzwikSi4Lyqo?= =?us-ascii?Q?32hJWe5FlqSYwq9dfHNQ7k9pz0S4vj9l3EttDKAijXwOsrI2D/HKcS1JmzXT?= =?us-ascii?Q?XtV/dH3zWHeRtYNsIvUSZErSkIPoW9rcdCU6brF0WpMCDQ8qgeBETd1ehJFs?= =?us-ascii?Q?KuSXAP8mqeIO6tEJbKBJrJ6LG26SVJyEHQhfutw0MeItn3Zs1IjDCJXsihsN?= =?us-ascii?Q?6Lr3x1qR2dytbBa9Ib47pbw/BdTVVmPV2/86THOIgUEYjpk+LKog4oz3LQXk?= =?us-ascii?Q?prNRA6AsN2iPbygM/WzbQqMZ2lyDi0ghXlLkK56txqA2W4JuwM6s4Mk/Xbob?= =?us-ascii?Q?es4WDOZJ2dVndA7EhAnkZL+GHzVcanylgeyj4ZyjW1VzxAatPbCONm/QJ9+Z?= =?us-ascii?Q?4THdLlj3XnldPsT1GlnmaH0LHJ4EPNNl6v2mV+kX0Sw3KbaGxIDiypq83+yn?= =?us-ascii?Q?IFEpCdIguvgVxviZ5doCy7YcjI8TtDJkzmCPKG/RAGCRS7n5DNADo3ySWXV4?= =?us-ascii?Q?Jf1KdU3uvhg5W+y8E5J8c95CPt+qD0cgiWfKlYrES2eP/03JJooD5iDoLdoS?= =?us-ascii?Q?n/AJyUTVNlWkSdvfdKOwyIYTgHBzXmLqhwqkVAyPzjpfJZH58MzH3SXjOA?= =?us-ascii?Q?=3D=3D?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:satlexmb07.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(376014)(7416014)(36860700013)(82310400026)(1800799024);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Dec 2025 11:21:29.4775 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 44f92967-e8d4-4b0a-1b1c-08de42de95b8 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb07.amd.com] X-MS-Exchange-CrossTenant-AuthSource: BL6PEPF0001AB55.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB9247 Content-Type: text/plain; charset="utf-8" Convert result variables and return types from int to enum scan_result throughout khugepaged code. This improves type safety and code clarity by making the intent explicit. No functional change. Signed-off-by: Shivank Garg Reviewed-by: Zi Yan --- mm/khugepaged.c | 111 +++++++++++++++++++++++++----------------------- 1 file changed, 57 insertions(+), 54 deletions(-) diff --git a/mm/khugepaged.c b/mm/khugepaged.c index 04ff0730c9a1..6892b23d6fc4 100644 --- a/mm/khugepaged.c +++ b/mm/khugepaged.c @@ -537,17 +537,18 @@ static void release_pte_pages(pte_t *pte, pte_t *_pte, } } =20 -static int __collapse_huge_page_isolate(struct vm_area_struct *vma, - unsigned long start_addr, - pte_t *pte, - struct collapse_control *cc, - struct list_head *compound_pagelist) +static enum scan_result __collapse_huge_page_isolate(struct vm_area_struct= *vma, + unsigned long start_addr, + pte_t *pte, + struct collapse_control *cc, + struct list_head *compound_pagelist) { struct page *page =3D NULL; struct folio *folio =3D NULL; unsigned long addr =3D start_addr; pte_t *_pte; - int none_or_zero =3D 0, shared =3D 0, result =3D SCAN_FAIL, referenced = =3D 0; + int none_or_zero =3D 0, shared =3D 0, referenced =3D 0; + enum scan_result result =3D SCAN_FAIL; =20 for (_pte =3D pte; _pte < pte + HPAGE_PMD_NR; _pte++, addr +=3D PAGE_SIZE) { @@ -780,13 +781,13 @@ static void __collapse_huge_page_copy_failed(pte_t *p= te, * @ptl: lock on raw pages' PTEs * @compound_pagelist: list that stores compound pages */ -static int __collapse_huge_page_copy(pte_t *pte, struct folio *folio, +static enum scan_result __collapse_huge_page_copy(pte_t *pte, struct folio= *folio, pmd_t *pmd, pmd_t orig_pmd, struct vm_area_struct *vma, unsigned long address, spinlock_t *ptl, struct list_head *compound_pagelist) { unsigned int i; - int result =3D SCAN_SUCCEED; + enum scan_result result =3D SCAN_SUCCEED; =20 /* * Copying pages' contents is subject to memory poison at any iteration. @@ -898,10 +899,9 @@ static int hpage_collapse_find_target_node(struct coll= apse_control *cc) * Returns enum scan_result value. */ =20 -static int hugepage_vma_revalidate(struct mm_struct *mm, unsigned long add= ress, - bool expect_anon, - struct vm_area_struct **vmap, - struct collapse_control *cc) +static enum scan_result hugepage_vma_revalidate(struct mm_struct *mm, unsi= gned long address, + bool expect_anon, struct vm_area_struct **vmap, + struct collapse_control *cc) { struct vm_area_struct *vma; enum tva_type type =3D cc->is_khugepaged ? TVA_KHUGEPAGED : @@ -930,7 +930,7 @@ static int hugepage_vma_revalidate(struct mm_struct *mm= , unsigned long address, return SCAN_SUCCEED; } =20 -static inline int check_pmd_state(pmd_t *pmd) +static inline enum scan_result check_pmd_state(pmd_t *pmd) { pmd_t pmde =3D pmdp_get_lockless(pmd); =20 @@ -953,9 +953,9 @@ static inline int check_pmd_state(pmd_t *pmd) return SCAN_SUCCEED; } =20 -static int find_pmd_or_thp_or_none(struct mm_struct *mm, - unsigned long address, - pmd_t **pmd) +static enum scan_result find_pmd_or_thp_or_none(struct mm_struct *mm, + unsigned long address, + pmd_t **pmd) { *pmd =3D mm_find_pmd(mm, address); if (!*pmd) @@ -964,12 +964,12 @@ static int find_pmd_or_thp_or_none(struct mm_struct *= mm, return check_pmd_state(*pmd); } =20 -static int check_pmd_still_valid(struct mm_struct *mm, - unsigned long address, - pmd_t *pmd) +static enum scan_result check_pmd_still_valid(struct mm_struct *mm, + unsigned long address, + pmd_t *pmd) { pmd_t *new_pmd; - int result =3D find_pmd_or_thp_or_none(mm, address, &new_pmd); + enum scan_result result =3D find_pmd_or_thp_or_none(mm, address, &new_pmd= ); =20 if (result !=3D SCAN_SUCCEED) return result; @@ -985,15 +985,15 @@ static int check_pmd_still_valid(struct mm_struct *mm, * Called and returns without pte mapped or spinlocks held. * Returns result: if not SCAN_SUCCEED, mmap_lock has been released. */ -static int __collapse_huge_page_swapin(struct mm_struct *mm, - struct vm_area_struct *vma, - unsigned long start_addr, pmd_t *pmd, - int referenced) +static enum scan_result __collapse_huge_page_swapin(struct mm_struct *mm, + struct vm_area_struct *vma, + unsigned long start_addr, pmd_t *pmd, + int referenced) { int swapped_in =3D 0; vm_fault_t ret =3D 0; unsigned long addr, end =3D start_addr + (HPAGE_PMD_NR * PAGE_SIZE); - int result; + enum scan_result result; pte_t *pte =3D NULL; spinlock_t *ptl; =20 @@ -1062,8 +1062,8 @@ static int __collapse_huge_page_swapin(struct mm_stru= ct *mm, return result; } =20 -static int alloc_charge_folio(struct folio **foliop, struct mm_struct *mm, - struct collapse_control *cc) +static enum scan_result alloc_charge_folio(struct folio **foliop, struct m= m_struct *mm, + struct collapse_control *cc) { gfp_t gfp =3D (cc->is_khugepaged ? alloc_hugepage_khugepaged_gfpmask() : GFP_TRANSHUGE); @@ -1090,9 +1090,9 @@ static int alloc_charge_folio(struct folio **foliop, = struct mm_struct *mm, return SCAN_SUCCEED; } =20 -static int collapse_huge_page(struct mm_struct *mm, unsigned long address, - int referenced, int unmapped, - struct collapse_control *cc) +static enum scan_result collapse_huge_page(struct mm_struct *mm, unsigned = long address, + int referenced, int unmapped, + struct collapse_control *cc) { LIST_HEAD(compound_pagelist); pmd_t *pmd, _pmd; @@ -1100,7 +1100,7 @@ static int collapse_huge_page(struct mm_struct *mm, u= nsigned long address, pgtable_t pgtable; struct folio *folio; spinlock_t *pmd_ptl, *pte_ptl; - int result =3D SCAN_FAIL; + enum scan_result result =3D SCAN_FAIL; struct vm_area_struct *vma; struct mmu_notifier_range range; =20 @@ -1246,15 +1246,15 @@ static int collapse_huge_page(struct mm_struct *mm,= unsigned long address, return result; } =20 -static int hpage_collapse_scan_pmd(struct mm_struct *mm, - struct vm_area_struct *vma, - unsigned long start_addr, bool *mmap_locked, - struct collapse_control *cc) +static enum scan_result hpage_collapse_scan_pmd(struct mm_struct *mm, + struct vm_area_struct *vma, + unsigned long start_addr, bool *mmap_locked, + struct collapse_control *cc) { pmd_t *pmd; pte_t *pte, *_pte; - int result =3D SCAN_FAIL, referenced =3D 0; - int none_or_zero =3D 0, shared =3D 0; + int none_or_zero =3D 0, shared =3D 0, referenced =3D 0; + enum scan_result result =3D SCAN_FAIL; struct page *page =3D NULL; struct folio *folio =3D NULL; unsigned long addr; @@ -1441,8 +1441,8 @@ static void collect_mm_slot(struct mm_slot *slot) } =20 /* folio must be locked, and mmap_lock must be held */ -static int set_huge_pmd(struct vm_area_struct *vma, unsigned long addr, - pmd_t *pmdp, struct folio *folio, struct page *page) +static enum scan_result set_huge_pmd(struct vm_area_struct *vma, unsigned = long addr, + pmd_t *pmdp, struct folio *folio, struct page *page) { struct mm_struct *mm =3D vma->vm_mm; struct vm_fault vmf =3D { @@ -1477,10 +1477,11 @@ static int set_huge_pmd(struct vm_area_struct *vma,= unsigned long addr, return SCAN_SUCCEED; } =20 -static int try_collapse_pte_mapped_thp(struct mm_struct *mm, unsigned long= addr, - bool install_pmd) +static enum scan_result try_collapse_pte_mapped_thp(struct mm_struct *mm, = unsigned long addr, + bool install_pmd) { - int nr_mapped_ptes =3D 0, result =3D SCAN_FAIL; + enum scan_result result =3D SCAN_FAIL; + int nr_mapped_ptes =3D 0; unsigned int nr_batch_ptes; struct mmu_notifier_range range; bool notified =3D false; @@ -1862,9 +1863,9 @@ static void retract_page_tables(struct address_space = *mapping, pgoff_t pgoff) * + unlock old pages * + unlock and free huge page; */ -static int collapse_file(struct mm_struct *mm, unsigned long addr, - struct file *file, pgoff_t start, - struct collapse_control *cc) +static enum scan_result collapse_file(struct mm_struct *mm, unsigned long = addr, + struct file *file, pgoff_t start, + struct collapse_control *cc) { struct address_space *mapping =3D file->f_mapping; struct page *dst; @@ -1872,7 +1873,8 @@ static int collapse_file(struct mm_struct *mm, unsign= ed long addr, pgoff_t index =3D 0, end =3D start + HPAGE_PMD_NR; LIST_HEAD(pagelist); XA_STATE_ORDER(xas, &mapping->i_pages, start, HPAGE_PMD_ORDER); - int nr_none =3D 0, result =3D SCAN_SUCCEED; + enum scan_result result =3D SCAN_SUCCEED; + int nr_none =3D 0; bool is_shmem =3D shmem_file(file); =20 VM_BUG_ON(!IS_ENABLED(CONFIG_READ_ONLY_THP_FOR_FS) && !is_shmem); @@ -2293,16 +2295,16 @@ static int collapse_file(struct mm_struct *mm, unsi= gned long addr, return result; } =20 -static int hpage_collapse_scan_file(struct mm_struct *mm, unsigned long ad= dr, - struct file *file, pgoff_t start, - struct collapse_control *cc) +static enum scan_result hpage_collapse_scan_file(struct mm_struct *mm, uns= igned long addr, + struct file *file, pgoff_t start, + struct collapse_control *cc) { struct folio *folio =3D NULL; struct address_space *mapping =3D file->f_mapping; XA_STATE(xas, &mapping->i_pages, start); int present, swap; int node =3D NUMA_NO_NODE; - int result =3D SCAN_SUCCEED; + enum scan_result result =3D SCAN_SUCCEED; =20 present =3D 0; swap =3D 0; @@ -2400,7 +2402,7 @@ static int hpage_collapse_scan_file(struct mm_struct = *mm, unsigned long addr, return result; } =20 -static unsigned int khugepaged_scan_mm_slot(unsigned int pages, int *resul= t, +static unsigned int khugepaged_scan_mm_slot(unsigned int pages, enum scan_= result *result, struct collapse_control *cc) __releases(&khugepaged_mm_lock) __acquires(&khugepaged_mm_lock) @@ -2561,7 +2563,7 @@ static void khugepaged_do_scan(struct collapse_contro= l *cc) unsigned int progress =3D 0, pass_through_head =3D 0; unsigned int pages =3D READ_ONCE(khugepaged_pages_to_scan); bool wait =3D true; - int result =3D SCAN_SUCCEED; + enum scan_result result =3D SCAN_SUCCEED; =20 lru_add_drain_all(); =20 @@ -2774,7 +2776,8 @@ int madvise_collapse(struct vm_area_struct *vma, unsi= gned long start, struct collapse_control *cc; struct mm_struct *mm =3D vma->vm_mm; unsigned long hstart, hend, addr; - int thps =3D 0, last_fail =3D SCAN_FAIL; + enum scan_result last_fail =3D SCAN_FAIL; + int thps =3D 0; bool mmap_locked =3D true; =20 BUG_ON(vma->vm_start > start); @@ -2796,7 +2799,7 @@ int madvise_collapse(struct vm_area_struct *vma, unsi= gned long start, =20 for (addr =3D hstart; addr < hend; addr +=3D HPAGE_PMD_SIZE) { bool retried =3D false; - int result =3D SCAN_FAIL; + enum scan_result result =3D SCAN_FAIL; =20 if (!mmap_locked) { retry: --=20 2.43.0