Re: [PATCH v2 00/21] btrfs: support idmapped mounts
From: Josef Bacik <josef@toxicpanda.com>
Date: 2021-07-19 15:32:55
On 7/19/21 7:10 AM, Christian Brauner wrote:
From: Christian Brauner <redacted> Hey everyone, This series enables the creation of idmapped mounts on btrfs. On the list of filesystems btrfs was pretty high-up and requested quite often from userspace (cf. [1]). This series requires just a few changes to the vfs for specific lookup helpers that btrfs relies on to perform permission checking when looking up an inode. The changes are required to port some other filesystem as well. The conversion of the necessary btrfs internals was fairly straightforward. No invasive changes were needed. I've decided to split up the patchset into very small individual patches. This hopefully makes the series more readable and fairly easy to review. The overall changeset is quite small. All non-filesystem wide ioctls that peform permission checking based on inodes can be supported on idmapped mounts. There are really just a few restrictions. This should really only affect the deletion of subvolumes by subvolume id which can be used to delete any subvolume in the filesystem even though the caller might not even be able to see the subvolume under their mount. Other than that behavior on idmapped and non-idmapped mounts is identical for all enabled ioctls. The changeset has an associated new testsuite specific to btrfs. The core vfs operations that btrfs implements are covered by the generic idmapped mount testsuite. For the ioctls a new testsuite was added. It is sent alongside this patchset for ease of review but will very likely be merged independent of it. All patches are based on v5.14-rc2. The series can be pulled from: https://git.kernel.org/brauner/h/fs.idmapped.btrfs https://github.com/brauner/linux/tree/fs.idmapped.btrfs The xfstests can be pulled from: https://git.kernel.org/brauner/xfstests-dev/h/fs.idmapped.btrfs https://github.com/brauner/xfstests/tree/fs.idmapped.btrfs Note, the new btrfs xfstests patch is on top of a branch of mine containing a few more preliminary patches. So if you want to run the tests, please simply pull the branch and build from there. The series has been tested with xfstests including the newly added btrfs specific test. All tests pass. There were three unrelated failures that I observed: btrfs/219, btrfs/2020 and btrfs/235. All three also fail on earlier kernels without the patch series applied. Thanks! Christian
Thanks for this work Christian, you can add Reviewed-by: Josef Bacik <josef@toxicpanda.com> to the series. Josef