Merge tag 'for-f2fs-4.2' of git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk...
authorLinus Torvalds <torvalds@linux-foundation.org>
Thu, 25 Jun 2015 03:38:29 +0000 (20:38 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 25 Jun 2015 03:38:29 +0000 (20:38 -0700)
Pull f2fs updates from Jaegeuk Kim:
 "New features:
   - per-file encryption (e.g., ext4)
   - FALLOC_FL_ZERO_RANGE
   - FALLOC_FL_COLLAPSE_RANGE
   - RENAME_WHITEOUT

  Major enhancement/fixes:
   - recovery broken superblocks
   - enhance f2fs_trim_fs with a discard_map
   - fix a race condition on dentry block allocation
   - fix a deadlock during summary operation
   - fix a missing fiemap result

  .. and many minor bug fixes and clean-ups were done"

* tag 'for-f2fs-4.2' of git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs: (83 commits)
  f2fs: do not trim preallocated blocks when truncating after i_size
  f2fs crypto: add alloc_bounce_page
  f2fs crypto: fix to handle errors likewise ext4
  f2fs: drop the volatile_write flag only
  f2fs: skip committing valid superblock
  f2fs: setting discard option in parse_options()
  f2fs: fix to return exact trimmed size
  f2fs: support FALLOC_FL_INSERT_RANGE
  f2fs: hide common code in f2fs_replace_block
  f2fs: disable the discard option when device doesn't support
  f2fs crypto: remove alloc_page for bounce_page
  f2fs: fix a deadlock for summary page lock vs. sentry_lock
  f2fs crypto: clean up error handling in f2fs_fname_setup_filename
  f2fs crypto: avoid f2fs_inherit_context for symlink
  f2fs crypto: do not set encryption policy for non-directory by ioctl
  f2fs crypto: allow setting encryption policy once
  f2fs crypto: check context consistent for rename2
  f2fs: avoid duplicated code by reusing f2fs_read_end_io
  f2fs crypto: use per-inode tfm structure
  f2fs: recovering broken superblock during mount
  ...

1  2 
fs/f2fs/namei.c

diff --combined fs/f2fs/namei.c
index 71765d062914a515fc7603843cabd2b456d5a189,83e21a17497745aad4df0cbad71437756da57799..fdbae21ee8fb2ca991c2e37e35b9fdc505eb291a
@@@ -56,11 -56,18 +56,18 @@@ static struct inode *f2fs_new_inode(str
                goto out;
        }
  
-       if (f2fs_may_inline(inode))
+       /* If the directory encrypted, then we should encrypt the inode. */
+       if (f2fs_encrypted_inode(dir) && f2fs_may_encrypt(inode))
+               f2fs_set_encrypted_inode(inode);
+       if (f2fs_may_inline_data(inode))
                set_inode_flag(F2FS_I(inode), FI_INLINE_DATA);
-       if (test_opt(sbi, INLINE_DENTRY) && S_ISDIR(inode->i_mode))
+       if (f2fs_may_inline_dentry(inode))
                set_inode_flag(F2FS_I(inode), FI_INLINE_DENTRY);
  
+       stat_inc_inline_inode(inode);
+       stat_inc_inline_dir(inode);
        trace_f2fs_new_inode(inode, 0);
        mark_inode_dirty(inode);
        return inode;
@@@ -136,7 -143,6 +143,6 @@@ static int f2fs_create(struct inode *di
  
        alloc_nid_done(sbi, ino);
  
-       stat_inc_inline_inode(inode);
        d_instantiate(dentry, inode);
        unlock_new_inode(inode);
  
@@@ -155,6 -161,10 +161,10 @@@ static int f2fs_link(struct dentry *old
        struct f2fs_sb_info *sbi = F2FS_I_SB(dir);
        int err;
  
+       if (f2fs_encrypted_inode(dir) &&
+               !f2fs_is_child_context_consistent_with_parent(dir, inode))
+               return -EPERM;
        f2fs_balance_fs(sbi);
  
        inode->i_ctime = CURRENT_TIME;
@@@ -232,32 -242,34 +242,34 @@@ static struct dentry *f2fs_lookup(struc
        struct inode *inode = NULL;
        struct f2fs_dir_entry *de;
        struct page *page;
+       nid_t ino;
+       int err = 0;
  
        if (dentry->d_name.len > F2FS_NAME_LEN)
                return ERR_PTR(-ENAMETOOLONG);
  
        de = f2fs_find_entry(dir, &dentry->d_name, &page);
-       if (de) {
-               nid_t ino = le32_to_cpu(de->ino);
-               f2fs_dentry_kunmap(dir, page);
-               f2fs_put_page(page, 0);
+       if (!de)
+               return d_splice_alias(inode, dentry);
  
-               inode = f2fs_iget(dir->i_sb, ino);
-               if (IS_ERR(inode))
-                       return ERR_CAST(inode);
+       ino = le32_to_cpu(de->ino);
+       f2fs_dentry_kunmap(dir, page);
+       f2fs_put_page(page, 0);
  
-               if (f2fs_has_inline_dots(inode)) {
-                       int err;
+       inode = f2fs_iget(dir->i_sb, ino);
+       if (IS_ERR(inode))
+               return ERR_CAST(inode);
  
-                       err = __recover_dot_dentries(inode, dir->i_ino);
-                       if (err) {
-                               iget_failed(inode);
-                               return ERR_PTR(err);
-                       }
-               }
+       if (f2fs_has_inline_dots(inode)) {
+               err = __recover_dot_dentries(inode, dir->i_ino);
+               if (err)
+                       goto err_out;
        }
        return d_splice_alias(inode, dentry);
+ err_out:
+       iget_failed(inode);
+       return ERR_PTR(err);
  }
  
  static int f2fs_unlink(struct inode *dir, struct dentry *dentry)
@@@ -296,15 -308,19 +308,15 @@@ fail
        return err;
  }
  
 -static void *f2fs_follow_link(struct dentry *dentry, struct nameidata *nd)
 +static const char *f2fs_follow_link(struct dentry *dentry, void **cookie)
  {
 -      struct page *page = page_follow_link_light(dentry, nd);
 -
 -      if (IS_ERR_OR_NULL(page))
 -              return page;
 -
 -      /* this is broken symlink case */
 -      if (*nd_get_link(nd) == 0) {
 -              page_put_link(dentry, nd, page);
 -              return ERR_PTR(-ENOENT);
 +      const char *link = page_follow_link_light(dentry, cookie);
 +      if (!IS_ERR(link) && !*link) {
 +              /* this is broken symlink case */
 +              page_put_link(NULL, *cookie);
 +              link = ERR_PTR(-ENOENT);
        }
 -      return page;
 +      return link;
  }
  
  static int f2fs_symlink(struct inode *dir, struct dentry *dentry,
  {
        struct f2fs_sb_info *sbi = F2FS_I_SB(dir);
        struct inode *inode;
-       size_t symlen = strlen(symname) + 1;
+       size_t len = strlen(symname);
+       size_t p_len;
+       char *p_str;
+       struct f2fs_str disk_link = FSTR_INIT(NULL, 0);
+       struct f2fs_encrypted_symlink_data *sd = NULL;
        int err;
  
+       if (len > dir->i_sb->s_blocksize)
+               return -ENAMETOOLONG;
        f2fs_balance_fs(sbi);
  
        inode = f2fs_new_inode(dir, S_IFLNK | S_IRWXUGO);
        if (IS_ERR(inode))
                return PTR_ERR(inode);
  
-       inode->i_op = &f2fs_symlink_inode_operations;
+       if (f2fs_encrypted_inode(inode))
+               inode->i_op = &f2fs_encrypted_symlink_inode_operations;
+       else
+               inode->i_op = &f2fs_symlink_inode_operations;
        inode->i_mapping->a_ops = &f2fs_dblock_aops;
  
        f2fs_lock_op(sbi);
        if (err)
                goto out;
        f2fs_unlock_op(sbi);
-       err = page_symlink(inode, symname, symlen);
        alloc_nid_done(sbi, inode->i_ino);
  
+       if (f2fs_encrypted_inode(dir)) {
+               struct qstr istr = QSTR_INIT(symname, len);
+               err = f2fs_get_encryption_info(inode);
+               if (err)
+                       goto err_out;
+               err = f2fs_fname_crypto_alloc_buffer(inode, len, &disk_link);
+               if (err)
+                       goto err_out;
+               err = f2fs_fname_usr_to_disk(inode, &istr, &disk_link);
+               if (err < 0)
+                       goto err_out;
+               p_len = encrypted_symlink_data_len(disk_link.len) + 1;
+               if (p_len > dir->i_sb->s_blocksize) {
+                       err = -ENAMETOOLONG;
+                       goto err_out;
+               }
+               sd = kzalloc(p_len, GFP_NOFS);
+               if (!sd) {
+                       err = -ENOMEM;
+                       goto err_out;
+               }
+               memcpy(sd->encrypted_path, disk_link.name, disk_link.len);
+               sd->len = cpu_to_le16(disk_link.len);
+               p_str = (char *)sd;
+       } else {
+               p_len = len + 1;
+               p_str = (char *)symname;
+       }
+       err = page_symlink(inode, p_str, p_len);
+ err_out:
        d_instantiate(dentry, inode);
        unlock_new_inode(inode);
  
         * If the symlink path is stored into inline_data, there is no
         * performance regression.
         */
-       filemap_write_and_wait_range(inode->i_mapping, 0, symlen - 1);
+       if (!err)
+               filemap_write_and_wait_range(inode->i_mapping, 0, p_len - 1);
  
        if (IS_DIRSYNC(dir))
                f2fs_sync_fs(sbi->sb, 1);
+       kfree(sd);
+       f2fs_fname_crypto_free_buffer(&disk_link);
        return err;
  out:
        handle_failed_inode(inode);
@@@ -379,7 -445,6 +441,6 @@@ static int f2fs_mkdir(struct inode *dir
                goto out_fail;
        f2fs_unlock_op(sbi);
  
-       stat_inc_inline_dir(inode);
        alloc_nid_done(sbi, inode->i_ino);
  
        d_instantiate(dentry, inode);
        return err;
  }
  
+ static int __f2fs_tmpfile(struct inode *dir, struct dentry *dentry,
+                                       umode_t mode, struct inode **whiteout)
+ {
+       struct f2fs_sb_info *sbi = F2FS_I_SB(dir);
+       struct inode *inode;
+       int err;
+       if (!whiteout)
+               f2fs_balance_fs(sbi);
+       inode = f2fs_new_inode(dir, mode);
+       if (IS_ERR(inode))
+               return PTR_ERR(inode);
+       if (whiteout) {
+               init_special_inode(inode, inode->i_mode, WHITEOUT_DEV);
+               inode->i_op = &f2fs_special_inode_operations;
+       } else {
+               inode->i_op = &f2fs_file_inode_operations;
+               inode->i_fop = &f2fs_file_operations;
+               inode->i_mapping->a_ops = &f2fs_dblock_aops;
+       }
+       f2fs_lock_op(sbi);
+       err = acquire_orphan_inode(sbi);
+       if (err)
+               goto out;
+       err = f2fs_do_tmpfile(inode, dir);
+       if (err)
+               goto release_out;
+       /*
+        * add this non-linked tmpfile to orphan list, in this way we could
+        * remove all unused data of tmpfile after abnormal power-off.
+        */
+       add_orphan_inode(sbi, inode->i_ino);
+       f2fs_unlock_op(sbi);
+       alloc_nid_done(sbi, inode->i_ino);
+       if (whiteout) {
+               inode_dec_link_count(inode);
+               *whiteout = inode;
+       } else {
+               d_tmpfile(dentry, inode);
+       }
+       unlock_new_inode(inode);
+       return 0;
+ release_out:
+       release_orphan_inode(sbi);
+ out:
+       handle_failed_inode(inode);
+       return err;
+ }
+ static int f2fs_tmpfile(struct inode *dir, struct dentry *dentry, umode_t mode)
+ {
+       if (f2fs_encrypted_inode(dir)) {
+               int err = f2fs_get_encryption_info(dir);
+               if (err)
+                       return err;
+       }
+       return __f2fs_tmpfile(dir, dentry, mode, NULL);
+ }
+ static int f2fs_create_whiteout(struct inode *dir, struct inode **whiteout)
+ {
+       return __f2fs_tmpfile(dir, NULL, S_IFCHR | WHITEOUT_MODE, whiteout);
+ }
  static int f2fs_rename(struct inode *old_dir, struct dentry *old_dentry,
-                       struct inode *new_dir, struct dentry *new_dentry)
+                       struct inode *new_dir, struct dentry *new_dentry,
+                       unsigned int flags)
  {
        struct f2fs_sb_info *sbi = F2FS_I_SB(old_dir);
        struct inode *old_inode = d_inode(old_dentry);
        struct inode *new_inode = d_inode(new_dentry);
+       struct inode *whiteout = NULL;
        struct page *old_dir_page;
-       struct page *old_page, *new_page;
+       struct page *old_page, *new_page = NULL;
        struct f2fs_dir_entry *old_dir_entry = NULL;
        struct f2fs_dir_entry *old_entry;
        struct f2fs_dir_entry *new_entry;
        int err = -ENOENT;
  
+       if ((old_dir != new_dir) && f2fs_encrypted_inode(new_dir) &&
+               !f2fs_is_child_context_consistent_with_parent(new_dir,
+                                                       old_inode)) {
+               err = -EPERM;
+               goto out;
+       }
        f2fs_balance_fs(sbi);
  
        old_entry = f2fs_find_entry(old_dir, &old_dentry->d_name, &old_page);
                        goto out_old;
        }
  
+       if (flags & RENAME_WHITEOUT) {
+               err = f2fs_create_whiteout(old_dir, &whiteout);
+               if (err)
+                       goto out_dir;
+       }
        if (new_inode) {
  
                err = -ENOTEMPTY;
                if (old_dir_entry && !f2fs_empty_dir(new_inode))
-                       goto out_dir;
+                       goto out_whiteout;
  
                err = -ENOENT;
                new_entry = f2fs_find_entry(new_dir, &new_dentry->d_name,
                                                &new_page);
                if (!new_entry)
-                       goto out_dir;
+                       goto out_whiteout;
  
                f2fs_lock_op(sbi);
  
                if (err)
                        goto put_out_dir;
  
-               if (update_dent_inode(old_inode, &new_dentry->d_name)) {
+               if (update_dent_inode(old_inode, new_inode,
+                                               &new_dentry->d_name)) {
                        release_orphan_inode(sbi);
                        goto put_out_dir;
                }
                err = f2fs_add_link(new_dentry, old_inode);
                if (err) {
                        f2fs_unlock_op(sbi);
-                       goto out_dir;
+                       goto out_whiteout;
                }
  
                if (old_dir_entry) {
  
        down_write(&F2FS_I(old_inode)->i_sem);
        file_lost_pino(old_inode);
+       if (new_inode && file_enc_name(new_inode))
+               file_set_enc_name(old_inode);
        up_write(&F2FS_I(old_inode)->i_sem);
  
        old_inode->i_ctime = CURRENT_TIME;
  
        f2fs_delete_entry(old_entry, old_page, old_dir, NULL);
  
+       if (whiteout) {
+               whiteout->i_state |= I_LINKABLE;
+               set_inode_flag(F2FS_I(whiteout), FI_INC_LINK);
+               err = f2fs_add_link(old_dentry, whiteout);
+               if (err)
+                       goto put_out_dir;
+               whiteout->i_state &= ~I_LINKABLE;
+               iput(whiteout);
+       }
        if (old_dir_entry) {
-               if (old_dir != new_dir) {
+               if (old_dir != new_dir && !whiteout) {
                        f2fs_set_link(old_inode, old_dir_entry,
                                                old_dir_page, new_dir);
                        update_inode_page(old_inode);
  
  put_out_dir:
        f2fs_unlock_op(sbi);
-       f2fs_dentry_kunmap(new_dir, new_page);
-       f2fs_put_page(new_page, 0);
+       if (new_page) {
+               f2fs_dentry_kunmap(new_dir, new_page);
+               f2fs_put_page(new_page, 0);
+       }
+ out_whiteout:
+       if (whiteout)
+               iput(whiteout);
  out_dir:
        if (old_dir_entry) {
                f2fs_dentry_kunmap(old_inode, old_dir_page);
@@@ -581,6 -752,14 +748,14 @@@ static int f2fs_cross_rename(struct ino
        int old_nlink = 0, new_nlink = 0;
        int err = -ENOENT;
  
+       if ((f2fs_encrypted_inode(old_dir) || f2fs_encrypted_inode(new_dir)) &&
+               (old_dir != new_dir) &&
+               (!f2fs_is_child_context_consistent_with_parent(new_dir,
+                                                               old_inode) ||
+               !f2fs_is_child_context_consistent_with_parent(old_dir,
+                                                               new_inode)))
+               return -EPERM;
        f2fs_balance_fs(sbi);
  
        old_entry = f2fs_find_entry(old_dir, &old_dentry->d_name, &old_page);
  
        f2fs_lock_op(sbi);
  
-       err = update_dent_inode(old_inode, &new_dentry->d_name);
+       err = update_dent_inode(old_inode, new_inode, &new_dentry->d_name);
        if (err)
                goto out_unlock;
+       if (file_enc_name(new_inode))
+               file_set_enc_name(old_inode);
  
-       err = update_dent_inode(new_inode, &old_dentry->d_name);
+       err = update_dent_inode(new_inode, old_inode, &old_dentry->d_name);
        if (err)
                goto out_undo;
+       if (file_enc_name(old_inode))
+               file_set_enc_name(new_inode);
  
        /* update ".." directory entry info of old dentry */
        if (old_dir_entry)
                f2fs_sync_fs(sbi->sb, 1);
        return 0;
  out_undo:
-       /* Still we may fail to recover name info of f2fs_inode here */
-       update_dent_inode(old_inode, &old_dentry->d_name);
+       /*
+        * Still we may fail to recover name info of f2fs_inode here
+        * Drop it, once its name is set as encrypted
+        */
+       update_dent_inode(old_inode, old_inode, &old_dentry->d_name);
  out_unlock:
        f2fs_unlock_op(sbi);
  out_new_dir:
@@@ -719,7 -905,7 +901,7 @@@ static int f2fs_rename2(struct inode *o
                        struct inode *new_dir, struct dentry *new_dentry,
                        unsigned int flags)
  {
-       if (flags & ~(RENAME_NOREPLACE | RENAME_EXCHANGE))
+       if (flags & ~(RENAME_NOREPLACE | RENAME_EXCHANGE | RENAME_WHITEOUT))
                return -EINVAL;
  
        if (flags & RENAME_EXCHANGE) {
         * VFS has already handled the new dentry existence case,
         * here, we just deal with "RENAME_NOREPLACE" as regular rename.
         */
-       return f2fs_rename(old_dir, old_dentry, new_dir, new_dentry);
+       return f2fs_rename(old_dir, old_dentry, new_dir, new_dentry, flags);
  }
  
- static int f2fs_tmpfile(struct inode *dir, struct dentry *dentry, umode_t mode)
+ #ifdef CONFIG_F2FS_FS_ENCRYPTION
 -static void *f2fs_encrypted_follow_link(struct dentry *dentry,
 -                                              struct nameidata *nd)
++static const char *f2fs_encrypted_follow_link(struct dentry *dentry, void **cookie)
  {
-       struct f2fs_sb_info *sbi = F2FS_I_SB(dir);
-       struct inode *inode;
-       int err;
-       inode = f2fs_new_inode(dir, mode);
-       if (IS_ERR(inode))
-               return PTR_ERR(inode);
-       inode->i_op = &f2fs_file_inode_operations;
-       inode->i_fop = &f2fs_file_operations;
-       inode->i_mapping->a_ops = &f2fs_dblock_aops;
-       f2fs_lock_op(sbi);
-       err = acquire_orphan_inode(sbi);
-       if (err)
-               goto out;
-       err = f2fs_do_tmpfile(inode, dir);
-       if (err)
-               goto release_out;
-       /*
-        * add this non-linked tmpfile to orphan list, in this way we could
-        * remove all unused data of tmpfile after abnormal power-off.
-        */
-       add_orphan_inode(sbi, inode->i_ino);
-       f2fs_unlock_op(sbi);
-       alloc_nid_done(sbi, inode->i_ino);
-       stat_inc_inline_inode(inode);
-       d_tmpfile(dentry, inode);
-       unlock_new_inode(inode);
-       return 0;
+       struct page *cpage = NULL;
+       char *caddr, *paddr = NULL;
+       struct f2fs_str cstr;
+       struct f2fs_str pstr = FSTR_INIT(NULL, 0);
+       struct inode *inode = d_inode(dentry);
+       struct f2fs_encrypted_symlink_data *sd;
+       loff_t size = min_t(loff_t, i_size_read(inode), PAGE_SIZE - 1);
+       u32 max_size = inode->i_sb->s_blocksize;
+       int res;
+       res = f2fs_get_encryption_info(inode);
+       if (res)
+               return ERR_PTR(res);
+       cpage = read_mapping_page(inode->i_mapping, 0, NULL);
+       if (IS_ERR(cpage))
 -              return cpage;
++              return ERR_CAST(cpage);
+       caddr = kmap(cpage);
+       caddr[size] = 0;
+       /* Symlink is encrypted */
+       sd = (struct f2fs_encrypted_symlink_data *)caddr;
+       cstr.name = sd->encrypted_path;
+       cstr.len = le16_to_cpu(sd->len);
+       /* this is broken symlink case */
+       if (cstr.name[0] == 0 && cstr.len == 0) {
+               res = -ENOENT;
+               goto errout;
+       }
  
- release_out:
-       release_orphan_inode(sbi);
- out:
-       handle_failed_inode(inode);
-       return err;
+       if ((cstr.len + sizeof(struct f2fs_encrypted_symlink_data) - 1) >
+                                                               max_size) {
+               /* Symlink data on the disk is corrupted */
+               res = -EIO;
+               goto errout;
+       }
+       res = f2fs_fname_crypto_alloc_buffer(inode, cstr.len, &pstr);
+       if (res)
+               goto errout;
+       res = f2fs_fname_disk_to_usr(inode, NULL, &cstr, &pstr);
+       if (res < 0)
+               goto errout;
+       paddr = pstr.name;
+       /* Null-terminate the name */
+       paddr[res] = '\0';
 -      nd_set_link(nd, paddr);
+       kunmap(cpage);
+       page_cache_release(cpage);
 -      return NULL;
++      return *cookie = paddr;
+ errout:
+       f2fs_fname_crypto_free_buffer(&pstr);
+       kunmap(cpage);
+       page_cache_release(cpage);
+       return ERR_PTR(res);
  }
  
+ const struct inode_operations f2fs_encrypted_symlink_inode_operations = {
+       .readlink       = generic_readlink,
+       .follow_link    = f2fs_encrypted_follow_link,
+       .put_link       = kfree_put_link,
+       .getattr        = f2fs_getattr,
+       .setattr        = f2fs_setattr,
+       .setxattr       = generic_setxattr,
+       .getxattr       = generic_getxattr,
+       .listxattr      = f2fs_listxattr,
+       .removexattr    = generic_removexattr,
+ };
+ #endif
  const struct inode_operations f2fs_dir_inode_operations = {
        .create         = f2fs_create,
        .lookup         = f2fs_lookup,