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 44270C3ABBF for ; Wed, 7 May 2025 11:50:12 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0468C6B008C; Wed, 7 May 2025 07:50:10 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id F39696B0092; Wed, 7 May 2025 07:50:09 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D8A656B0093; Wed, 7 May 2025 07:50:09 -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 B63766B008C for ; Wed, 7 May 2025 07:50:09 -0400 (EDT) Received: from smtpin19.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 1D73E801EB for ; Wed, 7 May 2025 11:50:11 +0000 (UTC) X-FDA: 83415943422.19.08F9794 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2085.outbound.protection.outlook.com [40.107.237.85]) by imf09.hostedemail.com (Postfix) with ESMTP id 630B314000A for ; Wed, 7 May 2025 11:50:08 +0000 (UTC) Authentication-Results: imf09.hostedemail.com; dkim=pass header.d=amd.com header.s=selector1 header.b=hgSFRQbb; dmarc=pass (policy=quarantine) header.from=amd.com; spf=pass (imf09.hostedemail.com: domain of shivankg@amd.com designates 40.107.237.85 as permitted sender) smtp.mailfrom=shivankg@amd.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1746618608; a=rsa-sha256; cv=pass; b=qRvVp03ArbbBwtzV9XjicmZP6ZW2wMfQn8rHq6g0QoVrrPMRGtZYuHbHDwvAhCZvYlzLse u0sB2HlF2+2jWlTfKvzI2z8AE6GfWBtvkNtoDqj7iiz8TkBHwTU9I3IoCb4y1UTlkk0JQ5 nFHwGcm/09VvFLYACcp4pPEZWf3Pfsg= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1746618608; 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=yXdXI2nyVkK/bD8BGKmQ8mOSGxcYCNzlrdwgVvS8s40=; b=M7dhexJV+5Sd7lCuEDSRS7+7OZpzDXlSX+JCwZ+EBwVr9fpodc6rHltLra3u8YiWPlQoyw qstyEMT1gD+tQYAv8kfK34J3Tj+w06m5LnVRpbUAV65+1AywJKQvmkf9fCrd1xlCFNFafz WEm94HNH+q9uelZeJ5BBl7Rs1TF1RmA= ARC-Authentication-Results: i=2; imf09.hostedemail.com; dkim=pass header.d=amd.com header.s=selector1 header.b=hgSFRQbb; dmarc=pass (policy=quarantine) header.from=amd.com; spf=pass (imf09.hostedemail.com: domain of shivankg@amd.com designates 40.107.237.85 as permitted sender) smtp.mailfrom=shivankg@amd.com; arc=pass ("microsoft.com:s=arcselector10001:i=1") ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=KbgSl/OYVYz/QWT2ymVhaBMJu+y2h91XVb1t0X+Gwp3S10eu7NlbXx3qGEpap4q5ohGCBVUMqsJIoaIQP4TegYBIWia3in849R4bAavGoqqPKRR+cYMo2w9rQVRo3N91hHPSBtJu7PXW6oYcV1eZaofkzYw7Wfv+/HaaV1l6rD1hMAS7hmo0i+JjKGSaY8xrafZwBAowaslFvXBT5nrbjzb1d+jQ4fHWAuPJlHw2lq0wuXNH+3wTaJzKiN0GWYp8UW76uDbP4Aiu60Hm1XTwwYsit34vEBJNioqtt+fJbdFchPkVoN6Mz2vu3COP6q1DeVMQ0tvdTnh3OItsmPwApA== 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=yXdXI2nyVkK/bD8BGKmQ8mOSGxcYCNzlrdwgVvS8s40=; b=KgacoN4QbYhUOJbHVVOjUDhFtEexqThY6azOcB5X1HYG3Jq+LoQguAkOVKfXZOV6QwC8C04sT1ADmlA/BUy8IcwwIPma2fCYCeOuYeCPatzoD+mzCPLm2KWGrz2Z8rj9uxEHGYa+Db3pJFr9K5UUSKIyUTC47yzyzGo7mma3EbGOXDxvq+eUi5fB70K5LoJCbZC4MopFD+jTzP6uwx5VMHxwXg1R/hkgo+nzc/i1ikrnZ9Wsx3iIm9NKZy6n0N/LMfSYm1XBq3vsGld+Zp3hYP04tU4wkXr+RlYEh7H0b4G1Hj9i1i4l/1rZM1nnw/Ma0sKLmJypvbOFnG4q0aV+cA== 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=yXdXI2nyVkK/bD8BGKmQ8mOSGxcYCNzlrdwgVvS8s40=; b=hgSFRQbb7XDmGY/jmfPo2VZig0r8xERaqaI3uFWbcsMroBis13rhzfjFK2Q1OKJsnwhLy1geJwS9IflKD6RWRGt/6pH0K9hivTjHhJiHzJ9BYeX+2E3Y/I4RU4WuQmBTSXsu17kr1JActAiIlooWdRKh3oB3DKIiisQygeqZzKM= Received: from CH2PR12MB4262.namprd12.prod.outlook.com (2603:10b6:610:af::8) by IA1PR12MB6577.namprd12.prod.outlook.com (2603:10b6:208:3a3::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8678.30; Wed, 7 May 2025 11:49: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%5]) with mapi id 15.20.8699.026; Wed, 7 May 2025 11:49:58 +0000 Message-ID: <21037c25-b85c-48fa-bdac-27cb3be2ccdb@amd.com> Date: Wed, 7 May 2025 17:19:52 +0530 User-Agent: Mozilla Thunderbird Subject: Re: [linux-next:master 7893/8235] fs/jfs/jfs_metapage.c:245 __metapage_migrate_folio() error: we previously assumed 'mp' could be null (see line 235) To: Dan Carpenter , oe-kbuild@lists.linux.dev Cc: lkp@intel.com, oe-kbuild-all@lists.linux.dev, Andrew Morton , Linux Memory Management List References: <202505071850.XaOkcCkX-lkp@intel.com> Content-Language: en-US From: Shivank Garg In-Reply-To: <202505071850.XaOkcCkX-lkp@intel.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: PN3PR01CA0089.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c01:9a::7) To CH2PR12MB4262.namprd12.prod.outlook.com (2603:10b6:610:af::8) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4262:EE_|IA1PR12MB6577:EE_ X-MS-Office365-Filtering-Correlation-Id: 6838dd8b-3b35-4502-ae90-08dd8d5d4a95 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|366016|1800799024|7053199007; X-Microsoft-Antispam-Message-Info: =?utf-8?B?YytPTW5rS2dxbDBlbzFTZXE2UiszN0lGa2ZMZ0sxKzhDVTE0d2ZtUFlqMWE0?= =?utf-8?B?Q1NZMk9oL0ZBTUNTQ1lNWGkvM1NINDhSL1Fmem1LYXgvN1o3cGtBL2VhbDZI?= =?utf-8?B?RnRySlA3MG9IYXdSVndvZDZMREhZclo5dk02eXZwMXU3ZStLaXhrZXlaZlJ4?= =?utf-8?B?Vm9WVXJHazFSZjI4RjhHZFZ2eVVtcVRnUUFETTA2Sy9LbVpGdGVKOTlTQzRJ?= =?utf-8?B?WXpmZ2RwTFFJSDFRMXBJN0w2Vm51RktrUXZDRU1MTVpxamI1ZDRSbDRMY3Zr?= =?utf-8?B?MzlIY0hsK3I2K2V5Mk02REhOQmdBdUIza3BkWCtTcFg5MnZ6Q1RsT3lGcWV5?= =?utf-8?B?RnlrRVpja0FYYVJoZ2dTaTRPMFZoamRJcjVVQmVXVlFxOXlhUHFXd1lNcEhw?= =?utf-8?B?WkJEOGZPc2wyRWpBQnMxTTNGdkdQeXNwak9OYi96RlJHYlNSdXRFUHJoNzBB?= =?utf-8?B?NXoybDJ4YkRDRGFrbGVsNVV4YWVwZ0o2WFkxeHpDY1NPeVBhcm9FTUgrNE9O?= =?utf-8?B?dEgxSk1FcGNtZUxLN2JHUG5FSFBzL0pheDE0QUNKTkpnN2hoclhJSnZBVUdu?= =?utf-8?B?SkNCMzdIR0xYZmN1ZnlORkdKOWFvcXp5d2pSOHdIRWQ0dkNwRzI5RlJZMVRi?= =?utf-8?B?aHcyQ3kzNlpHM0hmM0VjZkRGSlZTT3lnK3laa3ZoNEF5NTBkVHo4cHI5THpR?= =?utf-8?B?MkRTYkVlZERUK0NNU3FjUDRWNU9TOFhnRUZKZk9iYW15Uy81ZmpnbUh3OERi?= =?utf-8?B?bm9YMmVKN05RbW1FSDZ0NFZ6ZHR6UnhUZE91VllFM2ZsQ3d6dkxEbVFEZVVv?= =?utf-8?B?UWpvOXNjMHE4cmY5aFQ5SUl1c0VnTjQ5VTBzcnVkM1Q3eVV1TGp4cDFqZ1Zu?= =?utf-8?B?VW1VMnVFL0laL3FqMm84OElibDByMDlpMzRyRXhFODlzWFRKUWxaMHN0WGd2?= =?utf-8?B?MU5IMkJaa0NncGNwemhzeXI1a1lQRkRnMkpSSWFHdTkzcThUWWFSY0ZFWjJr?= =?utf-8?B?eUl0bEt4QkVwTXJtTmVnWVFuYlkzR3BOVU9vWHdyVTZDRmxVVUZXenUvYjg3?= =?utf-8?B?VDhjV2NGazhaQVl6dnMycWRFVUNEbFl2NlBvVGlrZ2lyTUlsSWM3N3RyWHFm?= =?utf-8?B?T3ZDM3NJQXUyM0hxTGxqUll5SkxlUkVyYldoTzJ4YVZjdkg0elZpQnhCREJ4?= =?utf-8?B?WWtUTFZRc2t5ZXkwdy9oMTBwcU9aMXJlT0dVYUdidjcwdHp4V1M5cGliREFE?= =?utf-8?B?dWhDNVo2ZC9jcWxPcWhHVTdNVTNKaWpjSmMvTVhQK2xuZ0R6ajZIWHlQcUht?= =?utf-8?B?SW1KSUR0ejJPZzFja0s5amVGWHUzNUJyNlFQUVRvc1dlZW1ZREZOMTJHS2Rs?= =?utf-8?B?VVNab0dpRnJLclY3eTFqWnp6ZmVFR2h5bllFK2M4bUtDRFJkSUFpYWRycEZS?= =?utf-8?B?UWEzMERvL2c0dlFtaVg2QW5sanBkQnVScGRJb1A5MjVEbGlyTGI1MVlDZUxV?= =?utf-8?B?SUdqTEhVUjRIRGx2S0Zva2hYSFVkdExCTGNUVnQ3MS9JZm5CTnlZS3ZFbnNE?= =?utf-8?B?UUhlQ3J0d3hIaFBadzlRV3JZelFCWWgvK3RQSlVUaFE2VFM5YXBTVDZYQzk2?= =?utf-8?B?enBFcHdReitnNlErMUhSeDdsUjlVVm9BM3ozaW9zRkRzM1FPRTZBdGwyUTZX?= =?utf-8?B?emwwYkFqUUE3ZDFJQjA1cmwvU3J4dVFlempHbzZUcDBnMm8yNWgrenh0b2ln?= =?utf-8?B?NDdlalNMcHpuMVhJYUprazFUS3BHaTVuekhpZVpiUHNWZzBCREZ0VWhRMTEx?= =?utf-8?B?OGxsSjl2UDlscEZRb2c3eHFMckhvNUZ3MkVkZXBGeFBKbm9YdzZDR3VjZWFQ?= =?utf-8?Q?bMGHUTgYwMqs2?= 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)(376014)(366016)(1800799024)(7053199007);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Mms1VWQ3OU1lVlVtZW51RXZYVWRycktKZTQyS2txQXc4bCtkdC9USFVwTnhI?= =?utf-8?B?VFlsL0JzOWpvV1JOakJHRmR2QTdWMk5MMjk5QXRKRk5HN1J6SC9kSTBzaDFp?= =?utf-8?B?MTFURWhEUjcxSWRkbW9iZGtuMzEzZnR2bWd3TURGMk5xeEVtOE9KT2ZiaG1D?= =?utf-8?B?MGtXbEt4b3dsK1FpOTZhY3hOYkcvaStxWXRmWmNLazlocVR2dFRHdHlFYUFY?= =?utf-8?B?VUFnU1JEVnpBejdGbThkZEM1RzE0bGRLMEZOVFJ6Y0ZqaStWcEp6YkJmNlNw?= =?utf-8?B?Tm81RFlEV0JDWUZRSlVwVkQ4bEVNNEk3dHpFVkVQTmRJeXFKeitzbW5ScElT?= =?utf-8?B?anNNZzFXSUhmd0dpbTJlRGVpeUEwVHMzQU1pTGVsUEhUeUx6TVVWUEllV05X?= =?utf-8?B?a2YzSTZlYnZMaEs2TTdXb3haVUwxcGJESjY0NTBFYmNRbndVbVhHN29RWUVG?= =?utf-8?B?MWJUYnVITnZjRDBNbGdPZWJUQTc0MzNVTU55aWVoTFJsemk5TGdLNlN2Y2hh?= =?utf-8?B?b09hNTUvQkp5RVZDdFc0SjB3aHljclpCZWsvb1QxMW1DeFpUdW1yZ2Nvc1hs?= =?utf-8?B?OVlOTHJXeDZTUTlaMHZMVFJPTEFWOGVUUG1ZanZ3UzFJd1dCdW0reU4wT3NW?= =?utf-8?B?WmlRR0RUSDhRQWU1d0tudER6SzU4SGpJUGVoTHowaW9LMDhFWE95WGtGaCtC?= =?utf-8?B?M3l5cVJEdlRVaXBIb0UvdFJUVXRQVy9wL0VEMkl0OEdLRExKdlovK0gyaVdB?= =?utf-8?B?cnpQUXYzM05Yc3RtdVVvTXVtK2pwT3RJTFUxN09ycTk1WXNVbWtTVjJkcUtC?= =?utf-8?B?YURTZ0dJQVYvY1NJSC9iR1A4MTlHNDJkazY5d2VoVVV5b2loeERsUFFrZWlO?= =?utf-8?B?OFNWa1ZpdFkvbGRhUjdmUWhXTW5na3hlRm9SVVZjWlB0czB4c2x0dXk5SW9Q?= =?utf-8?B?ZDM2bjlhMTN0Wm9CNUNmcWhJNktHVWpRcjF2dzBWaVFzQkllM1JBS2NUVFBR?= =?utf-8?B?eXBFZy9CcTRzcGZleW4ycnRvcllETmJDTzRGcGdGbmdudTRvOWdKemZ0RXRo?= =?utf-8?B?R01GblpjSXJZTnVZWWIwRlZHTHl4NWhEL0FPVHl4VUJmSlp4UWMyYVlOSW1T?= =?utf-8?B?RmJoNWFFdVlZVVdhNk1Zc1QwaXBZOFNvZmVCdVhvd1k2NGxQdExwci9DT3di?= =?utf-8?B?Z0tLRlNGQWV1dnNPbXIyeTcrVDl1L1NZakFPcGdONWYrMGc3T2ZRaU5yU1lT?= =?utf-8?B?LzVSVHduMW1iQjVMSWpMN3NUOERIeWhDbkdsWFpvRERBUVZaQmJMcDNOSzYv?= =?utf-8?B?OGdiVUNPbmw5dy9zSFZodFk5OVdLcjJockZGbVBuZEFoZ0o1SjNxdVFmc241?= =?utf-8?B?UHZHTWx1VnZzaFdaRlVqVXVrV1hpTDFKbGpsVy9mRVFON3k1K01KWE0rRmhl?= =?utf-8?B?OFZVVnY0M08ySE5ISGNiMDhYUzNLRHRIcHdIVGRwTEd0QzNybEFla29GMVhB?= =?utf-8?B?dUI3RUZQemF1QXhZRlNpeWtUcUgvWVc2ajN2SHN0N3RiblJKT0pjU1JHNklC?= =?utf-8?B?dVYzSkpINmZWaFpFTk8yR2xMaU1qWUZ3eStqSTdTekxMMnJ2L1M1Zzl6VTBa?= =?utf-8?B?cjVjYWZSN3FSeW94RTV5MG9tUmlRWW9Ua0pHdjdZRUFLT3BxMGg5WTVwU0Fp?= =?utf-8?B?RlV5S1B4eVR6TjVWeU9rbDBtNnhGcGdCSDZnUzJmVFBSajN2d1g0RU5zYmpz?= =?utf-8?B?OE9xK1pQK1VqRFV6VGdZaXpjcWRyMFpjVVdwelVFU2RxWlMzOWlZU29pS2tH?= =?utf-8?B?d3pCNkV2cWhzMDVKVFh3aEZReDlrbEo4MXBBQlFFdUV0bUUyVDJoMVRkTVpL?= =?utf-8?B?dnVWSVNXbUVnR0o1Uk1WY3p1V215dFZQZU02NlRrK2w1VVdqTXd0VmowdW9a?= =?utf-8?B?UkRkcnFieDlYOWxQSHdwaWxVbUg2Uzd4VEZ1Ylg1NExzT2tkNytkeWNQVmE5?= =?utf-8?B?YlJPdUorVXkrR0hHYk1FdTZvME12VmhZdkFTQTdTSTkyN1NvcUVyZXJHM3NK?= =?utf-8?B?dUdXWVVVQnNSeGxBdWR0cjNHdEEvY3M0c3UzQ1o3NmViUjAzQVAyRWlJV2ls?= =?utf-8?Q?Xr4xpcFx2L9hSqV5hoAChqVty?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6838dd8b-3b35-4502-ae90-08dd8d5d4a95 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4262.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 May 2025 11:49:58.3283 (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: pFG8L97waoXaI0ZV0yCZ3xQ4XTvRdTK0snEgcNOrFaZLBjxGSFYtWL7pgqqlfTQ8HFCca1FZ7MMDCAz4ViEnag== X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB6577 X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: 630B314000A X-Stat-Signature: 3ha4nib9961ybfdrfx1xj7kktz9dotqk X-Rspam-User: X-HE-Tag: 1746618608-285928 X-HE-Meta: U2FsdGVkX18zGl+qDPb3lsuH6KT7ipFEVurknUqPk5UavRlcs5tnal7YJGRCuGkG00HImZn1QaF8nqt3ZLOK25ewzq+p/dIfP7vocSfpfnr3bbiq0+VXmGYnBI9huzQurtE54bvBm/PwrYn5zti8FWmt0WrAI0S0ih8sN6ncRjRbdYLjZJRAl2KEdIg/gORVToTXE+UreE9ZUDCJK1x/kFbgGxffOzzThllOBWMzDMh58iaRmIRjaYqji6aBRObPWUxDnrNY11yn4kCkMO0WxmDKT26OALCoUKuWsic7IDpyXFn1SCxVLHms2R4zGCETyUHk3ypfK+bcRxUkaM1zf1RK+1uPqs0w2GNTxKx9fccbfgXU5gLaG8cKzycyj60jtNC0ZTyLhXhvKp4IJ4j1E7iNPAoAIFGFmgBDld1RJAqMZe8PQLhUm3Ukx3BOXmw6Lt/t1CB//t+ocxVYF1pPaYvqJqFkbUmBKfq635F0sJ2X1k5fYaIE3c6dtiFb2xC+FU/rlKD/3VtJ+8no/Pwk8EMYXNYWucn+i9bjxWgJSwxecdl8oiGfVhSHLE4+VnEHKP+rSfGfhBuCmUXrcOr6SVlZdgpZ21jqjyrCgOZ2ZgSDMtcOK8/SHOpV32BhD8HCt982SQsXQQ8ZFbMEVkzUTShGNizwjpZr2hQ8OPuih1WTamlDKlBH8LgNOj+by9+FJNAAlEdx3YNu9tu5W1Oeubf33o0xcOYfZyemvmgr6SjojNFM0TDrrDaK/sW9quZ62BZYroKd6Nd22BWDxn25PiSoFHjoaLfyWY/gkWXUmiXbE7dvMANebeM3riJ4yDzjGUljAu2uqG9Zjei5y9xxajOjChwkTNvx+bUJkMxGM2ZIpiOEB8RwXK1svuOMXpmpIfdc7QbqC/njc+cGNZc5HAv6eIbGkKk3UMrQmnbJ4aJRi5fL/bpiobTbx3P27mNprhX3VNLVW5hjM/9cO2N SwGlRB8W nzyzj 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 5/7/2025 4:41 PM, Dan Carpenter wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master > head: 08710e696081d58163c8078e0e096be6d35c5fad > commit: 39ed4d1a0e03ce3ac2145ee7ef0714c78bae9c61 [7893/8235] jfs: implement migrate_folio for jfs_metapage_aops > config: i386-randconfig-141-20250502 (https://download.01.org/0day-ci/archive/20250507/202505071850.XaOkcCkX-lkp@intel.com/config) > compiler: clang version 20.1.2 (https://github.com/llvm/llvm-project 58df0ef89dd64126512e4ee27b4ac3fd8ddf6247) > > If you fix the issue in a separate patch/commit (i.e. not just a new version of > the same patch/commit), kindly add following tags > | Reported-by: kernel test robot > | Reported-by: Dan Carpenter > | Closes: https://lore.kernel.org/r/202505071850.XaOkcCkX-lkp@intel.com/ > > smatch warnings: > fs/jfs/jfs_metapage.c:245 __metapage_migrate_folio() error: we previously assumed 'mp' could be null (see line 235) > > vim +/mp +245 fs/jfs/jfs_metapage.c > > 39ed4d1a0e03ce Shivank Garg 2025-04-30 227 static int __metapage_migrate_folio(struct address_space *mapping, struct folio *dst, > 39ed4d1a0e03ce Shivank Garg 2025-04-30 228 struct folio *src, enum migrate_mode mode) > 39ed4d1a0e03ce Shivank Garg 2025-04-30 229 { > 39ed4d1a0e03ce Shivank Garg 2025-04-30 230 struct metapage *mp; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 231 int page_offset; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 232 int rc; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 233 > 39ed4d1a0e03ce Shivank Garg 2025-04-30 234 mp = folio_to_mp(src, 0); > 39ed4d1a0e03ce Shivank Garg 2025-04-30 @235 if (mp && metapage_locked(mp)) > ^^ > If mp is NULL > > 39ed4d1a0e03ce Shivank Garg 2025-04-30 236 return -EAGAIN; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 237 > 39ed4d1a0e03ce Shivank Garg 2025-04-30 238 rc = filemap_migrate_folio(mapping, dst, src, mode); > 39ed4d1a0e03ce Shivank Garg 2025-04-30 239 if (rc != MIGRATEPAGE_SUCCESS) > 39ed4d1a0e03ce Shivank Garg 2025-04-30 240 return rc; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 241 > 39ed4d1a0e03ce Shivank Garg 2025-04-30 242 if (unlikely(insert_metapage(dst, mp))) > 39ed4d1a0e03ce Shivank Garg 2025-04-30 243 return -EAGAIN; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 244 > 39ed4d1a0e03ce Shivank Garg 2025-04-30 @245 page_offset = mp->data - folio_address(src); > ^^ > Then this will crash. > > 39ed4d1a0e03ce Shivank Garg 2025-04-30 246 mp->data = folio_address(dst) + page_offset; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 247 mp->folio = dst; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 248 remove_metapage(src, mp); > 39ed4d1a0e03ce Shivank Garg 2025-04-30 249 > 39ed4d1a0e03ce Shivank Garg 2025-04-30 250 return MIGRATEPAGE_SUCCESS; > 39ed4d1a0e03ce Shivank Garg 2025-04-30 251 } > Hi Dan, This is a false positive. In metapage_migrate_folio(), it checks if (!src->private) and only calls __metapage_migrate_folio() if src->private is non-NULL static int metapage_migrate_folio(struct address_space *mapping, struct folio *dst, struct folio *src, enum migrate_mode mode) { ... if (!src->private) return filemap_migrate_folio(mapping, dst, src, mode); ... ... return __metapage_migrate_folio(mapping, dst, src, mode); } Then __metapage_migrate_folio() calls mp = folio_to_mp(src, 0), which returns src->private. Which should not be NULL as previously checked. I think we can skip the mp != NULL check from > 39ed4d1a0e03ce Shivank Garg 2025-04-30 @235 if (mp && metapage_locked(mp)) It seem redundant here. Thanks, Shivank