From 37688e00ed3e8c603778ee5271bb12ac11a32249 Mon Sep 17 00:00:00 2001 From: Shivank Garg Date: Thu, 8 May 2025 06:00:33 +0000 Subject: [PATCH] jfs: remove redundant NULL check in __metapage_migrate_folio() The NULL check for 'mp' is unnecessary since metapage_migrate_folio() already verifies src->private exists before calling this function. Reported-by: kernel test robot Reported-by: Dan Carpenter Closes: https://lore.kernel.org/r/202505071850.XaOkcCkX-lkp@intel.com/ Fixes: 708c4552aa37 ("jfs: implement migrate_folio for jfs_metapage_aops") Signed-off-by: Shivank Garg --- fs/jfs/jfs_metapage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/jfs/jfs_metapage.c b/fs/jfs/jfs_metapage.c index 7e23db142b27..9029cd216912 100644 --- a/fs/jfs/jfs_metapage.c +++ b/fs/jfs/jfs_metapage.c @@ -238,7 +238,7 @@ static int __metapage_migrate_folio(struct address_space *mapping, int rc; mp = folio_to_mp(src, 0); - if (mp && metapage_locked(mp)) + if (metapage_locked(mp)) return -EAGAIN; rc = filemap_migrate_folio(mapping, dst, src, mode); -- 2.34.1