f2fs: update inode info to inode page for new file
authorYunlei He <heyunlei@huawei.com>
Wed, 3 Jan 2018 10:03:04 +0000 (18:03 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Tue, 16 Jan 2018 23:39:56 +0000 (15:39 -0800)
After checkpoint,
 1. creat a new file A ,(with dirty inode && dirty inode page && xattr info)
 2. backgroud wb write back file A inode page (without update from inode cache)
 3. fsync file A, write back inode page of file A with inode cache info
 4. sudden power off before new checkpoint

In this case, recovery process will try to recover a zero inode
page. Inline xattr flag of file A will be miss and xattr info
will be taken as blkaddr index.

Signed-off-by: Yunlei He <heyunlei@huawei.com>
Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/node.c

index 35b079d..b495a54 100644 (file)
@@ -2207,7 +2207,9 @@ void recover_inline_xattr(struct inode *inode, struct page *page)
        f2fs_bug_on(F2FS_I_SB(inode), IS_ERR(ipage));
 
        ri = F2FS_INODE(page);
-       if (!(ri->i_inline & F2FS_INLINE_XATTR)) {
+       if (ri->i_inline & F2FS_INLINE_XATTR) {
+               set_inode_flag(inode, FI_INLINE_XATTR);
+       } else {
                clear_inode_flag(inode, FI_INLINE_XATTR);
                goto update_inode;
        }