f2fs: disable extent_cache for fcollapse/finsert inodes
[cascardo/linux.git] / fs / f2fs / file.c
index f4c0086..72e52cd 100644 (file)
@@ -81,7 +81,8 @@ static int f2fs_vm_page_mkwrite(struct vm_area_struct *vma,
                zero_user_segment(page, offset, PAGE_SIZE);
        }
        set_page_dirty(page);
-       SetPageUptodate(page);
+       if (!PageUptodate(page))
+               SetPageUptodate(page);
 
        trace_f2fs_vm_page_mkwrite(page, DATA);
 mapped:
@@ -171,22 +172,16 @@ static void try_to_fix_pino(struct inode *inode)
        fi->xattr_ver = 0;
        if (file_wrong_pino(inode) && inode->i_nlink == 1 &&
                        get_parent_ino(inode, &pino)) {
-               fi->i_pino = pino;
+               f2fs_i_pino_write(inode, pino);
                file_got_pino(inode);
-               up_write(&fi->i_sem);
-
-               mark_inode_dirty_sync(inode);
-               f2fs_write_inode(inode, NULL);
-       } else {
-               up_write(&fi->i_sem);
        }
+       up_write(&fi->i_sem);
 }
 
 static int f2fs_do_sync_file(struct file *file, loff_t start, loff_t end,
                                                int datasync, bool atomic)
 {
        struct inode *inode = file->f_mapping->host;
-       struct f2fs_inode_info *fi = F2FS_I(inode);
        struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
        nid_t ino = inode->i_ino;
        int ret = 0;
@@ -204,9 +199,9 @@ static int f2fs_do_sync_file(struct file *file, loff_t start, loff_t end,
 
        /* if fdatasync is triggered, let's do in-place-update */
        if (datasync || get_dirty_pages(inode) <= SM_I(sbi)->min_fsync_blocks)
-               set_inode_flag(fi, FI_NEED_IPU);
+               set_inode_flag(inode, FI_NEED_IPU);
        ret = filemap_write_and_wait_range(inode->i_mapping, start, end);
-       clear_inode_flag(fi, FI_NEED_IPU);
+       clear_inode_flag(inode, FI_NEED_IPU);
 
        if (ret) {
                trace_f2fs_sync_file_exit(inode, need_cp, datasync, ret);
@@ -214,7 +209,7 @@ static int f2fs_do_sync_file(struct file *file, loff_t start, loff_t end,
        }
 
        /* if the inode is dirty, let's recover all the time */
-       if (!datasync) {
+       if (!datasync && !f2fs_skip_inode_update(inode)) {
                f2fs_write_inode(inode, NULL);
                goto go_write;
        }
@@ -222,14 +217,14 @@ static int f2fs_do_sync_file(struct file *file, loff_t start, loff_t end,
        /*
         * if there is no written data, don't waste time to write recovery info.
         */
-       if (!is_inode_flag_set(fi, FI_APPEND_WRITE) &&
+       if (!is_inode_flag_set(inode, FI_APPEND_WRITE) &&
                        !exist_written_data(sbi, ino, APPEND_INO)) {
 
                /* it may call write_inode just prior to fsync */
                if (need_inode_page_update(sbi, ino))
                        goto go_write;
 
-               if (is_inode_flag_set(fi, FI_UPDATE_WRITE) ||
+               if (is_inode_flag_set(inode, FI_UPDATE_WRITE) ||
                                exist_written_data(sbi, ino, UPDATE_INO))
                        goto flush_out;
                goto out;
@@ -239,9 +234,9 @@ go_write:
         * Both of fdatasync() and fsync() are able to be recovered from
         * sudden-power-off.
         */
-       down_read(&fi->i_sem);
+       down_read(&F2FS_I(inode)->i_sem);
        need_cp = need_do_checkpoint(inode);
-       up_read(&fi->i_sem);
+       up_read(&F2FS_I(inode)->i_sem);
 
        if (need_cp) {
                /* all the dirty node pages should be flushed for POR */
@@ -252,12 +247,12 @@ go_write:
                 * will be used only for fsynced inodes after checkpoint.
                 */
                try_to_fix_pino(inode);
-               clear_inode_flag(fi, FI_APPEND_WRITE);
-               clear_inode_flag(fi, FI_UPDATE_WRITE);
+               clear_inode_flag(inode, FI_APPEND_WRITE);
+               clear_inode_flag(inode, FI_UPDATE_WRITE);
                goto out;
        }
 sync_nodes:
-       ret = fsync_node_pages(sbi, ino, &wbc, atomic);
+       ret = fsync_node_pages(sbi, inode, &wbc, atomic);
        if (ret)
                goto out;
 
@@ -268,7 +263,7 @@ sync_nodes:
        }
 
        if (need_inode_block_update(sbi, ino)) {
-               mark_inode_dirty_sync(inode);
+               f2fs_mark_inode_dirty_sync(inode);
                f2fs_write_inode(inode, NULL);
                goto sync_nodes;
        }
@@ -279,10 +274,10 @@ sync_nodes:
 
        /* once recovery info is written, don't need to tack this */
        remove_ino_entry(sbi, ino, APPEND_INO);
-       clear_inode_flag(fi, FI_APPEND_WRITE);
+       clear_inode_flag(inode, FI_APPEND_WRITE);
 flush_out:
        remove_ino_entry(sbi, ino, UPDATE_INO);
-       clear_inode_flag(fi, FI_UPDATE_WRITE);
+       clear_inode_flag(inode, FI_UPDATE_WRITE);
        ret = f2fs_issue_flush(sbi);
        f2fs_update_time(sbi, REQ_TIME);
 out:
@@ -360,7 +355,7 @@ static loff_t f2fs_seek_block(struct file *file, loff_t offset, int whence)
 
        for (; data_ofs < isize; data_ofs = (loff_t)pgofs << PAGE_SHIFT) {
                set_new_dnode(&dn, inode, NULL, NULL, 0);
-               err = get_dnode_of_data(&dn, pgofs, LOOKUP_NODE_RA);
+               err = get_dnode_of_data(&dn, pgofs, LOOKUP_NODE);
                if (err && err != -ENOENT) {
                        goto fail;
                } else if (err == -ENOENT) {
@@ -487,8 +482,7 @@ int truncate_data_blocks_range(struct dnode_of_data *dn, int count)
                set_data_blkaddr(dn);
                invalidate_blocks(sbi, blkaddr);
                if (dn->ofs_in_node == 0 && IS_INODE(dn->node_page))
-                       clear_inode_flag(F2FS_I(dn->inode),
-                                               FI_FIRST_BLOCK_WRITTEN);
+                       clear_inode_flag(dn->inode, FI_FIRST_BLOCK_WRITTEN);
                nr_free++;
        }
 
@@ -502,7 +496,6 @@ int truncate_data_blocks_range(struct dnode_of_data *dn, int count)
                                                        dn->inode) + ofs;
                f2fs_update_extent_cache_range(dn, fofs, 0, len);
                dec_valid_block_count(sbi, dn->inode, nr_free);
-               sync_inode_page(dn);
        }
        dn->ofs_in_node = ofs;
 
@@ -616,7 +609,7 @@ free_partial:
        return err;
 }
 
-int f2fs_truncate(struct inode *inode, bool lock)
+int f2fs_truncate(struct inode *inode)
 {
        int err;
 
@@ -633,12 +626,12 @@ int f2fs_truncate(struct inode *inode, bool lock)
                        return err;
        }
 
-       err = truncate_blocks(inode, i_size_read(inode), lock);
+       err = truncate_blocks(inode, i_size_read(inode), true);
        if (err)
                return err;
 
        inode->i_mtime = inode->i_ctime = CURRENT_TIME;
-       mark_inode_dirty(inode);
+       f2fs_mark_inode_dirty_sync(inode);
        return 0;
 }
 
@@ -654,7 +647,6 @@ int f2fs_getattr(struct vfsmount *mnt,
 #ifdef CONFIG_F2FS_FS_POSIX_ACL
 static void __setattr_copy(struct inode *inode, const struct iattr *attr)
 {
-       struct f2fs_inode_info *fi = F2FS_I(inode);
        unsigned int ia_valid = attr->ia_valid;
 
        if (ia_valid & ATTR_UID)
@@ -675,7 +667,7 @@ static void __setattr_copy(struct inode *inode, const struct iattr *attr)
 
                if (!in_group_p(inode->i_gid) && !capable(CAP_FSETID))
                        mode &= ~S_ISGID;
-               set_acl_inode(fi, mode);
+               set_acl_inode(inode, mode);
        }
 }
 #else
@@ -685,7 +677,6 @@ static void __setattr_copy(struct inode *inode, const struct iattr *attr)
 int f2fs_setattr(struct dentry *dentry, struct iattr *attr)
 {
        struct inode *inode = d_inode(dentry);
-       struct f2fs_inode_info *fi = F2FS_I(inode);
        int err;
 
        err = inode_change_ok(inode, attr);
@@ -699,7 +690,7 @@ int f2fs_setattr(struct dentry *dentry, struct iattr *attr)
 
                if (attr->ia_size <= i_size_read(inode)) {
                        truncate_setsize(inode, attr->ia_size);
-                       err = f2fs_truncate(inode, true);
+                       err = f2fs_truncate(inode);
                        if (err)
                                return err;
                        f2fs_balance_fs(F2FS_I_SB(inode), true);
@@ -724,13 +715,13 @@ int f2fs_setattr(struct dentry *dentry, struct iattr *attr)
 
        if (attr->ia_valid & ATTR_MODE) {
                err = posix_acl_chmod(inode, get_inode_mode(inode));
-               if (err || is_inode_flag_set(fi, FI_ACL_MODE)) {
-                       inode->i_mode = fi->i_acl_mode;
-                       clear_inode_flag(fi, FI_ACL_MODE);
+               if (err || is_inode_flag_set(inode, FI_ACL_MODE)) {
+                       inode->i_mode = F2FS_I(inode)->i_acl_mode;
+                       clear_inode_flag(inode, FI_ACL_MODE);
                }
        }
 
-       mark_inode_dirty(inode);
+       f2fs_mark_inode_dirty_sync(inode);
        return err;
 }
 
@@ -859,79 +850,189 @@ static int punch_hole(struct inode *inode, loff_t offset, loff_t len)
        return ret;
 }
 
-static int __exchange_data_block(struct inode *inode, pgoff_t src,
-                                       pgoff_t dst, bool full)
+static int __read_out_blkaddrs(struct inode *inode, block_t *blkaddr,
+                               int *do_replace, pgoff_t off, pgoff_t len)
 {
        struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
        struct dnode_of_data dn;
-       block_t new_addr;
-       bool do_replace = false;
-       int ret;
+       int ret, done, i;
 
+next_dnode:
        set_new_dnode(&dn, inode, NULL, NULL, 0);
-       ret = get_dnode_of_data(&dn, src, LOOKUP_NODE_RA);
+       ret = get_dnode_of_data(&dn, off, LOOKUP_NODE_RA);
        if (ret && ret != -ENOENT) {
                return ret;
        } else if (ret == -ENOENT) {
-               new_addr = NULL_ADDR;
-       } else {
-               new_addr = dn.data_blkaddr;
-               if (!is_checkpointed_data(sbi, new_addr)) {
+               if (dn.max_level == 0)
+                       return -ENOENT;
+               done = min((pgoff_t)ADDRS_PER_BLOCK - dn.ofs_in_node, len);
+               blkaddr += done;
+               do_replace += done;
+               goto next;
+       }
+
+       done = min((pgoff_t)ADDRS_PER_PAGE(dn.node_page, inode) -
+                                                       dn.ofs_in_node, len);
+       for (i = 0; i < done; i++, blkaddr++, do_replace++, dn.ofs_in_node++) {
+               *blkaddr = datablock_addr(dn.node_page, dn.ofs_in_node);
+               if (!is_checkpointed_data(sbi, *blkaddr)) {
+
+                       if (test_opt(sbi, LFS)) {
+                               f2fs_put_dnode(&dn);
+                               return -ENOTSUPP;
+                       }
+
                        /* do not invalidate this block address */
                        f2fs_update_data_blkaddr(&dn, NULL_ADDR);
-                       do_replace = true;
+                       *do_replace = 1;
                }
-               f2fs_put_dnode(&dn);
        }
+       f2fs_put_dnode(&dn);
+next:
+       len -= done;
+       off += done;
+       if (len)
+               goto next_dnode;
+       return 0;
+}
 
-       if (new_addr == NULL_ADDR)
-               return full ? truncate_hole(inode, dst, dst + 1) : 0;
+static int __roll_back_blkaddrs(struct inode *inode, block_t *blkaddr,
+                               int *do_replace, pgoff_t off, int len)
+{
+       struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
+       struct dnode_of_data dn;
+       int ret, i;
 
-       if (do_replace) {
-               struct page *ipage = get_node_page(sbi, inode->i_ino);
-               struct node_info ni;
+       for (i = 0; i < len; i++, do_replace++, blkaddr++) {
+               if (*do_replace == 0)
+                       continue;
 
-               if (IS_ERR(ipage)) {
-                       ret = PTR_ERR(ipage);
-                       goto err_out;
+               set_new_dnode(&dn, inode, NULL, NULL, 0);
+               ret = get_dnode_of_data(&dn, off + i, LOOKUP_NODE_RA);
+               if (ret) {
+                       dec_valid_block_count(sbi, inode, 1);
+                       invalidate_blocks(sbi, *blkaddr);
+               } else {
+                       f2fs_update_data_blkaddr(&dn, *blkaddr);
                }
+               f2fs_put_dnode(&dn);
+       }
+       return 0;
+}
 
-               set_new_dnode(&dn, inode, ipage, NULL, 0);
-               ret = f2fs_reserve_block(&dn, dst);
-               if (ret)
-                       goto err_out;
+static int __clone_blkaddrs(struct inode *src_inode, struct inode *dst_inode,
+                       block_t *blkaddr, int *do_replace,
+                       pgoff_t src, pgoff_t dst, pgoff_t len, bool full)
+{
+       struct f2fs_sb_info *sbi = F2FS_I_SB(src_inode);
+       pgoff_t i = 0;
+       int ret;
 
-               truncate_data_blocks_range(&dn, 1);
+       while (i < len) {
+               if (blkaddr[i] == NULL_ADDR && !full) {
+                       i++;
+                       continue;
+               }
 
-               get_node_info(sbi, dn.nid, &ni);
-               f2fs_replace_block(sbi, &dn, dn.data_blkaddr, new_addr,
-                               ni.version, true, false);
-               f2fs_put_dnode(&dn);
-       } else {
-               struct page *psrc, *pdst;
+               if (do_replace[i] || blkaddr[i] == NULL_ADDR) {
+                       struct dnode_of_data dn;
+                       struct node_info ni;
+                       size_t new_size;
+                       pgoff_t ilen;
+
+                       set_new_dnode(&dn, dst_inode, NULL, NULL, 0);
+                       ret = get_dnode_of_data(&dn, dst + i, ALLOC_NODE);
+                       if (ret)
+                               return ret;
 
-               psrc = get_lock_data_page(inode, src, true);
-               if (IS_ERR(psrc))
-                       return PTR_ERR(psrc);
-               pdst = get_new_data_page(inode, NULL, dst, true);
-               if (IS_ERR(pdst)) {
+                       get_node_info(sbi, dn.nid, &ni);
+                       ilen = min((pgoff_t)
+                               ADDRS_PER_PAGE(dn.node_page, dst_inode) -
+                                               dn.ofs_in_node, len - i);
+                       do {
+                               dn.data_blkaddr = datablock_addr(dn.node_page,
+                                                               dn.ofs_in_node);
+                               truncate_data_blocks_range(&dn, 1);
+
+                               if (do_replace[i]) {
+                                       f2fs_i_blocks_write(src_inode,
+                                                               1, false);
+                                       f2fs_i_blocks_write(dst_inode,
+                                                               1, true);
+                                       f2fs_replace_block(sbi, &dn, dn.data_blkaddr,
+                                       blkaddr[i], ni.version, true, false);
+
+                                       do_replace[i] = 0;
+                               }
+                               dn.ofs_in_node++;
+                               i++;
+                               new_size = (dst + i) << PAGE_SHIFT;
+                               if (dst_inode->i_size < new_size)
+                                       f2fs_i_size_write(dst_inode, new_size);
+                       } while ((do_replace[i] || blkaddr[i] == NULL_ADDR) && --ilen);
+
+                       f2fs_put_dnode(&dn);
+               } else {
+                       struct page *psrc, *pdst;
+
+                       psrc = get_lock_data_page(src_inode, src + i, true);
+                       if (IS_ERR(psrc))
+                               return PTR_ERR(psrc);
+                       pdst = get_new_data_page(dst_inode, NULL, dst + i,
+                                                               true);
+                       if (IS_ERR(pdst)) {
+                               f2fs_put_page(psrc, 1);
+                               return PTR_ERR(pdst);
+                       }
+                       f2fs_copy_page(psrc, pdst);
+                       set_page_dirty(pdst);
+                       f2fs_put_page(pdst, 1);
                        f2fs_put_page(psrc, 1);
-                       return PTR_ERR(pdst);
-               }
-               f2fs_copy_page(psrc, pdst);
-               set_page_dirty(pdst);
-               f2fs_put_page(pdst, 1);
-               f2fs_put_page(psrc, 1);
 
-               return truncate_hole(inode, src, src + 1);
+                       ret = truncate_hole(src_inode, src + i, src + i + 1);
+                       if (ret)
+                               return ret;
+                       i++;
+               }
        }
        return 0;
+}
 
-err_out:
-       if (!get_dnode_of_data(&dn, src, LOOKUP_NODE)) {
-               f2fs_update_data_blkaddr(&dn, new_addr);
-               f2fs_put_dnode(&dn);
+static int __exchange_data_block(struct inode *src_inode,
+                       struct inode *dst_inode, pgoff_t src, pgoff_t dst,
+                       int len, bool full)
+{
+       block_t *src_blkaddr;
+       int *do_replace;
+       int ret;
+
+       src_blkaddr = f2fs_kvzalloc(sizeof(block_t) * len, GFP_KERNEL);
+       if (!src_blkaddr)
+               return -ENOMEM;
+
+       do_replace = f2fs_kvzalloc(sizeof(int) * len, GFP_KERNEL);
+       if (!do_replace) {
+               kvfree(src_blkaddr);
+               return -ENOMEM;
        }
+
+       ret = __read_out_blkaddrs(src_inode, src_blkaddr, do_replace, src, len);
+       if (ret)
+               goto roll_back;
+
+       ret = __clone_blkaddrs(src_inode, dst_inode, src_blkaddr,
+                                       do_replace, src, dst, len, full);
+       if (ret)
+               goto roll_back;
+
+       kvfree(src_blkaddr);
+       kvfree(do_replace);
+       return 0;
+
+roll_back:
+       __roll_back_blkaddrs(src_inode, src_blkaddr, do_replace, src, len);
+       kvfree(src_blkaddr);
+       kvfree(do_replace);
        return ret;
 }
 
@@ -939,16 +1040,15 @@ static int f2fs_do_collapse(struct inode *inode, pgoff_t start, pgoff_t end)
 {
        struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
        pgoff_t nrpages = (i_size_read(inode) + PAGE_SIZE - 1) / PAGE_SIZE;
-       int ret = 0;
+       int ret;
 
-       for (; end < nrpages; start++, end++) {
-               f2fs_balance_fs(sbi, true);
-               f2fs_lock_op(sbi);
-               ret = __exchange_data_block(inode, end, start, true);
-               f2fs_unlock_op(sbi);
-               if (ret)
-                       break;
-       }
+       f2fs_balance_fs(sbi, true);
+       f2fs_lock_op(sbi);
+
+       f2fs_drop_extent_tree(inode);
+
+       ret = __exchange_data_block(inode, inode, end, start, nrpages - end, true);
+       f2fs_unlock_op(sbi);
        return ret;
 }
 
@@ -992,7 +1092,7 @@ static int f2fs_collapse_range(struct inode *inode, loff_t offset, loff_t len)
 
        ret = truncate_blocks(inode, new_size, true);
        if (!ret)
-               i_size_write(inode, new_size);
+               f2fs_i_size_write(inode, new_size);
 
        return ret;
 }
@@ -1128,11 +1228,8 @@ static int f2fs_zero_range(struct inode *inode, loff_t offset, loff_t len,
        }
 
 out:
-       if (!(mode & FALLOC_FL_KEEP_SIZE) && i_size_read(inode) < new_size) {
-               i_size_write(inode, new_size);
-               mark_inode_dirty(inode);
-               update_inode_page(inode);
-       }
+       if (!(mode & FALLOC_FL_KEEP_SIZE) && i_size_read(inode) < new_size)
+               f2fs_i_size_write(inode, new_size);
 
        return ret;
 }
@@ -1140,7 +1237,7 @@ out:
 static int f2fs_insert_range(struct inode *inode, loff_t offset, loff_t len)
 {
        struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
-       pgoff_t pg_start, pg_end, delta, nrpages, idx;
+       pgoff_t nr, pg_start, pg_end, delta, idx;
        loff_t new_size;
        int ret = 0;
 
@@ -1175,14 +1272,20 @@ static int f2fs_insert_range(struct inode *inode, loff_t offset, loff_t len)
        pg_start = offset >> PAGE_SHIFT;
        pg_end = (offset + len) >> PAGE_SHIFT;
        delta = pg_end - pg_start;
-       nrpages = (i_size_read(inode) + PAGE_SIZE - 1) / PAGE_SIZE;
+       idx = (i_size_read(inode) + PAGE_SIZE - 1) / PAGE_SIZE;
+
+       while (!ret && idx > pg_start) {
+               nr = idx - pg_start;
+               if (nr > delta)
+                       nr = delta;
+               idx -= nr;
 
-       for (idx = nrpages - 1; idx >= pg_start && idx != -1; idx--) {
                f2fs_lock_op(sbi);
-               ret = __exchange_data_block(inode, idx, idx + delta, false);
+               f2fs_drop_extent_tree(inode);
+
+               ret = __exchange_data_block(inode, inode, idx,
+                                       idx + delta, nr, false);
                f2fs_unlock_op(sbi);
-               if (ret)
-                       break;
        }
 
        /* write out all moved pages, if possible */
@@ -1190,7 +1293,7 @@ static int f2fs_insert_range(struct inode *inode, loff_t offset, loff_t len)
        truncate_pagecache(inode, offset);
 
        if (!ret)
-               i_size_write(inode, new_size);
+               f2fs_i_size_write(inode, new_size);
        return ret;
 }
 
@@ -1238,11 +1341,8 @@ static int expand_inode_data(struct inode *inode, loff_t offset,
                new_size = ((loff_t)pg_end << PAGE_SHIFT) + off_end;
        }
 
-       if (!(mode & FALLOC_FL_KEEP_SIZE) && i_size_read(inode) < new_size) {
-               i_size_write(inode, new_size);
-               mark_inode_dirty(inode);
-               update_inode_page(inode);
-       }
+       if (!(mode & FALLOC_FL_KEEP_SIZE) && i_size_read(inode) < new_size)
+               f2fs_i_size_write(inode, new_size);
 
        return ret;
 }
@@ -1285,7 +1385,7 @@ static long f2fs_fallocate(struct file *file, int mode,
 
        if (!ret) {
                inode->i_mtime = inode->i_ctime = CURRENT_TIME;
-               mark_inode_dirty(inode);
+               f2fs_mark_inode_dirty_sync(inode);
                f2fs_update_time(F2FS_I_SB(inode), REQ_TIME);
        }
 
@@ -1310,10 +1410,10 @@ static int f2fs_release_file(struct inode *inode, struct file *filp)
        if (f2fs_is_atomic_file(inode))
                drop_inmem_pages(inode);
        if (f2fs_is_volatile_file(inode)) {
-               clear_inode_flag(F2FS_I(inode), FI_VOLATILE_FILE);
-               set_inode_flag(F2FS_I(inode), FI_DROP_CACHE);
+               clear_inode_flag(inode, FI_VOLATILE_FILE);
+               set_inode_flag(inode, FI_DROP_CACHE);
                filemap_fdatawrite(inode->i_mapping);
-               clear_inode_flag(F2FS_I(inode), FI_DROP_CACHE);
+               clear_inode_flag(inode, FI_DROP_CACHE);
        }
        return 0;
 }
@@ -1376,9 +1476,8 @@ static int f2fs_ioc_setflags(struct file *filp, unsigned long arg)
        fi->i_flags = flags;
        inode_unlock(inode);
 
-       f2fs_set_inode_flags(inode);
        inode->i_ctime = CURRENT_TIME;
-       mark_inode_dirty(inode);
+       f2fs_set_inode_flags(inode);
 out:
        mnt_drop_write_file(filp);
        return ret;
@@ -1412,7 +1511,7 @@ static int f2fs_ioc_start_atomic_write(struct file *filp)
        if (ret)
                goto out;
 
-       set_inode_flag(F2FS_I(inode), FI_ATOMIC_FILE);
+       set_inode_flag(inode, FI_ATOMIC_FILE);
        f2fs_update_time(F2FS_I_SB(inode), REQ_TIME);
 
        if (!get_dirty_pages(inode))
@@ -1423,7 +1522,7 @@ static int f2fs_ioc_start_atomic_write(struct file *filp)
                                        inode->i_ino, get_dirty_pages(inode));
        ret = filemap_write_and_wait_range(inode->i_mapping, 0, LLONG_MAX);
        if (ret)
-               clear_inode_flag(F2FS_I(inode), FI_ATOMIC_FILE);
+               clear_inode_flag(inode, FI_ATOMIC_FILE);
 out:
        inode_unlock(inode);
        mnt_drop_write_file(filp);
@@ -1448,10 +1547,10 @@ static int f2fs_ioc_commit_atomic_write(struct file *filp)
                goto err_out;
 
        if (f2fs_is_atomic_file(inode)) {
-               clear_inode_flag(F2FS_I(inode), FI_ATOMIC_FILE);
+               clear_inode_flag(inode, FI_ATOMIC_FILE);
                ret = commit_inmem_pages(inode);
                if (ret) {
-                       set_inode_flag(F2FS_I(inode), FI_ATOMIC_FILE);
+                       set_inode_flag(inode, FI_ATOMIC_FILE);
                        goto err_out;
                }
        }
@@ -1484,7 +1583,7 @@ static int f2fs_ioc_start_volatile_write(struct file *filp)
        if (ret)
                goto out;
 
-       set_inode_flag(F2FS_I(inode), FI_VOLATILE_FILE);
+       set_inode_flag(inode, FI_VOLATILE_FILE);
        f2fs_update_time(F2FS_I_SB(inode), REQ_TIME);
 out:
        inode_unlock(inode);
@@ -1538,7 +1637,7 @@ static int f2fs_ioc_abort_volatile_write(struct file *filp)
        if (f2fs_is_atomic_file(inode))
                drop_inmem_pages(inode);
        if (f2fs_is_volatile_file(inode)) {
-               clear_inode_flag(F2FS_I(inode), FI_VOLATILE_FILE);
+               clear_inode_flag(inode, FI_VOLATILE_FILE);
                ret = f2fs_do_sync_file(filp, 0, LLONG_MAX, 0, true);
        }
 
@@ -1871,7 +1970,7 @@ do_map:
                        continue;
                }
 
-               set_inode_flag(F2FS_I(inode), FI_DO_DEFRAG);
+               set_inode_flag(inode, FI_DO_DEFRAG);
 
                idx = map.m_lblk;
                while (idx < map.m_lblk + map.m_len && cnt < blk_per_seg) {
@@ -1896,14 +1995,14 @@ do_map:
                if (idx < pg_end && cnt < blk_per_seg)
                        goto do_map;
 
-               clear_inode_flag(F2FS_I(inode), FI_DO_DEFRAG);
+               clear_inode_flag(inode, FI_DO_DEFRAG);
 
                err = filemap_fdatawrite(inode->i_mapping);
                if (err)
                        goto out;
        }
 clear_out:
-       clear_inode_flag(F2FS_I(inode), FI_DO_DEFRAG);
+       clear_inode_flag(inode, FI_DO_DEFRAG);
 out:
        inode_unlock(inode);
        if (!err)