Skip to content

Commit f2db710

Browse files
chaseyuJaegeuk Kim
authored andcommitted
f2fs: fix to clear dirty inode in f2fs_evict_inode()
As Yanming reported in bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=215904 The kernel message is shown below: kernel BUG at fs/f2fs/inode.c:825! Call Trace: evict+0x282/0x4e0 __dentry_kill+0x2b2/0x4d0 shrink_dentry_list+0x17c/0x4f0 shrink_dcache_parent+0x143/0x1e0 do_one_tree+0x9/0x30 shrink_dcache_for_umount+0x51/0x120 generic_shutdown_super+0x5c/0x3a0 kill_block_super+0x90/0xd0 kill_f2fs_super+0x225/0x310 deactivate_locked_super+0x78/0xc0 cleanup_mnt+0x2b7/0x480 task_work_run+0xc8/0x150 exit_to_user_mode_prepare+0x14a/0x150 syscall_exit_to_user_mode+0x1d/0x40 do_syscall_64+0x48/0x90 The root cause is: inode node and dnode node share the same nid, so during f2fs_evict_inode(), dnode node truncation will invalidate its NAT entry, so when truncating inode node, it fails due to invalid NAT entry, result in inode is still marked as dirty, fix this issue by clearing dirty for inode and setting SBI_NEED_FSCK flag in filesystem. output from dump.f2fs: [print_node_info: 354] Node ID [0xf:15] is inode i_nid[0] [0x f : 15] Cc: stable@vger.kernel.org Reported-by: Ming Yan <yanming@tju.edu.cn> Signed-off-by: Chao Yu <chao.yu@oppo.com> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
1 parent 7f262f7 commit f2db710

1 file changed

Lines changed: 15 additions & 1 deletion

File tree

fs/f2fs/inode.c

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -796,8 +796,22 @@ void f2fs_evict_inode(struct inode *inode)
796796
f2fs_lock_op(sbi);
797797
err = f2fs_remove_inode_page(inode);
798798
f2fs_unlock_op(sbi);
799-
if (err == -ENOENT)
799+
if (err == -ENOENT) {
800800
err = 0;
801+
802+
/*
803+
* in fuzzed image, another node may has the same
804+
* block address as inode's, if it was truncated
805+
* previously, truncation of inode node will fail.
806+
*/
807+
if (is_inode_flag_set(inode, FI_DIRTY_INODE)) {
808+
f2fs_warn(F2FS_I_SB(inode),
809+
"f2fs_evict_inode: inconsistent node id, ino:%lu",
810+
inode->i_ino);
811+
f2fs_inode_synced(inode);
812+
set_sbi_flag(sbi, SBI_NEED_FSCK);
813+
}
814+
}
801815
}
802816

803817
/* give more chances, if ENOMEM case */

0 commit comments

Comments
 (0)