cangelog: v4 -> v5 ... Merge 2nd and 3rd patches and update commit log No code change in total v3 -> v4 ... Reorganize patches and update commit log No code change in total v2 -> v3 ... Use if-else block instead of two if blocks and add Tested-by tag in 2nd patch v1 -> v2 ... Split the patch to hopefully make review easier
Note: I will send a xfstest if this series is merged. This series changes the behavior of rmdir(2) and allow it to delete an empty subvolume. In order so that, 1st and 2nd patch refactor btrfs_ioctl_snap_destroy() and extract the actual deletion process as btrfs_delete_subvolume() (remaining part in btrfs_ioctl_snap_destroy() is mainly permission checks). Then, 3rd patch changes btrfs_rmdir() to call this function. The required permission check is already done in vfs layer. Tomohiro Misono (3): btrfs: Move may_destroy_subvol() from ioctl.c to inode.c btrfs: Factor out the main deletion process from btrfs_ioctl_snap_destroy() btrfs: Allow rmdir(2) to delete an empty subvolume fs/btrfs/ctree.h | 5 +- fs/btrfs/inode.c | 197 ++++++++++++++++++++++++++++++++++++++++++++++++++++++- fs/btrfs/ioctl.c | 185 +-------------------------------------------------- 3 files changed, 198 insertions(+), 189 deletions(-) -- 2.14.3 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html