All of lore.kernel.org
 help / color / mirror / Atom feed
From: Omar Sandoval <osandov@osandov.com>
To: linux-btrfs@vger.kernel.org
Cc: kernel-team@fb.com
Subject: [PATCH v2 00/16] btrfs: inode creation cleanups and fixes
Date: Wed,  9 Mar 2022 17:31:30 -0800	[thread overview]
Message-ID: <cover.1646875648.git.osandov@fb.com> (raw)

From: Omar Sandoval <osandov@fb.com>

This series contains several cleanups and fixes for our inode creation
codepaths. The main motivation for this is preparation for fscrypt
support (in particular, setting up the fscrypt context and encrypted
names at inode creation time). But, it also reduces a lot of code
duplication and fixes some minor bugs, so it's worth getting in ahead of
time.

Patches 1-3 are small fixes. Patches 5-12 are small cleanups. Patches
13-16 are the bulk of the change.

Based on misc-next.

Changes since v1 [1]:

- Split the big final patch into patches 3 and 13-16.
- Added Sweet Tea's reviewed-by to the remaining patches.
- Rebased on latest misc-next.

Thanks!

1: https://lore.kernel.org/linux-btrfs/cover.1646348486.git.osandov@fb.com/

Omar Sandoval (16):
  btrfs: reserve correct number of items for unlink and rmdir
  btrfs: reserve correct number of items for rename
  btrfs: fix anon_dev leak in create_subvol()
  btrfs: get rid of btrfs_add_nondir()
  btrfs: remove unnecessary btrfs_i_size_write(0) calls
  btrfs: remove unnecessary inode_set_bytes(0) call
  btrfs: remove unnecessary set_nlink() in btrfs_create_subvol_root()
  btrfs: remove unused mnt_userns parameter from __btrfs_set_acl
  btrfs: remove redundant name and name_len parameters to create_subvol
  btrfs: don't pass parent objectid to btrfs_new_inode() explicitly
  btrfs: move btrfs_get_free_objectid() call into btrfs_new_inode()
  btrfs: set inode flags earlier in btrfs_new_inode()
  btrfs: allocate inode outside of btrfs_new_inode()
  btrfs: factor out common part of btrfs_{mknod,create,mkdir}()
  btrfs: reserve correct number of items for inode creation
  btrfs: move common inode creation code into btrfs_create_new_inode()

 fs/btrfs/acl.c   |  39 +-
 fs/btrfs/ctree.h |  37 +-
 fs/btrfs/inode.c | 942 +++++++++++++++++++++++------------------------
 fs/btrfs/ioctl.c | 176 ++++-----
 fs/btrfs/props.c |  40 +-
 fs/btrfs/props.h |   4 -
 6 files changed, 579 insertions(+), 659 deletions(-)

-- 
2.35.1


             reply	other threads:[~2022-03-10  1:31 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-10  1:31 Omar Sandoval [this message]
2022-03-10  1:31 ` [PATCH v2 01/16] btrfs: reserve correct number of items for unlink and rmdir Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 02/16] btrfs: reserve correct number of items for rename Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 03/16] btrfs: fix anon_dev leak in create_subvol() Omar Sandoval
2022-03-11 15:42   ` Sweet Tea Dorminy
2022-03-12  0:29     ` Omar Sandoval
2022-03-12  1:43       ` Sweet Tea Dorminy
2022-03-10  1:31 ` [PATCH v2 04/16] btrfs: get rid of btrfs_add_nondir() Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 05/16] btrfs: remove unnecessary btrfs_i_size_write(0) calls Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 06/16] btrfs: remove unnecessary inode_set_bytes(0) call Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 07/16] btrfs: remove unnecessary set_nlink() in btrfs_create_subvol_root() Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 08/16] btrfs: remove unused mnt_userns parameter from __btrfs_set_acl Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 09/16] btrfs: remove redundant name and name_len parameters to create_subvol Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 10/16] btrfs: don't pass parent objectid to btrfs_new_inode() explicitly Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 11/16] btrfs: move btrfs_get_free_objectid() call into btrfs_new_inode() Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 12/16] btrfs: set inode flags earlier in btrfs_new_inode() Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 13/16] btrfs: allocate inode outside of btrfs_new_inode() Omar Sandoval
2022-03-11 17:11   ` Sweet Tea Dorminy
2022-03-12  0:41     ` Omar Sandoval
2022-03-14 14:43       ` Sweet Tea Dorminy
2022-03-14 23:33   ` Filipe Manana
2022-03-15  0:16     ` Omar Sandoval
2022-03-15  1:14       ` Omar Sandoval
2022-03-15  8:01         ` Qu Wenruo
2022-03-10  1:31 ` [PATCH v2 14/16] btrfs: factor out common part of btrfs_{mknod,create,mkdir}() Omar Sandoval
2022-03-11 17:43   ` Sweet Tea Dorminy
2022-03-10  1:31 ` [PATCH v2 15/16] btrfs: reserve correct number of items for inode creation Omar Sandoval
2022-03-11 17:56   ` Sweet Tea Dorminy
2022-03-12  0:45     ` Omar Sandoval
2022-03-10  1:31 ` [PATCH v2 16/16] btrfs: move common inode creation code into btrfs_create_new_inode() Omar Sandoval
2022-03-11 18:04   ` Sweet Tea Dorminy
2022-03-14 12:50 ` [PATCH v2 00/16] btrfs: inode creation cleanups and fixes David Sterba
2022-03-14 18:42   ` Omar Sandoval
2022-03-14 19:27     ` David Sterba
2022-03-14 19:55       ` Omar Sandoval

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1646875648.git.osandov@fb.com \
    --to=osandov@osandov.com \
    --cc=kernel-team@fb.com \
    --cc=linux-btrfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.