diff options
| author | Boris Burkov <boris@bur.io> | 2023-06-28 11:00:05 -0700 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2023-10-12 16:44:10 +0200 |
| commit | 6ed05643ddb166c0fddabac8ee092659006214a9 (patch) | |
| tree | d4170bc6404d703e0eeecb955cbb76322679585f /fs/btrfs/transaction.c | |
| parent | af0e2aab3b70b7844232bbce2a619ec70e049df4 (diff) | |
| download | linux-6ed05643ddb166c0fddabac8ee092659006214a9.tar.gz linux-6ed05643ddb166c0fddabac8ee092659006214a9.tar.bz2 linux-6ed05643ddb166c0fddabac8ee092659006214a9.zip | |
btrfs: create qgroup earlier in snapshot creation
Pull creating the qgroup earlier in the snapshot. This allows simple
quotas qgroups to see all the metadata writes related to the snapshot
being created and to be born with the root node accounted.
Note this has an impact on transaction commit where the qgroup creation
can do a lot of work, allocate memory and take locks. The change is done
for correctness, potential performance issues will be fixed in the
future.
Signed-off-by: Boris Burkov <boris@bur.io>
[ add note ]
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/transaction.c')
| -rw-r--r-- | fs/btrfs/transaction.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fs/btrfs/transaction.c b/fs/btrfs/transaction.c index f5c3ea01c3d6..6c24d00c3612 100644 --- a/fs/btrfs/transaction.c +++ b/fs/btrfs/transaction.c @@ -1812,6 +1812,12 @@ static noinline int create_pending_snapshot(struct btrfs_trans_handle *trans, } btrfs_release_path(path); + ret = btrfs_create_qgroup(trans, objectid); + if (ret) { + btrfs_abort_transaction(trans, ret); + goto fail; + } + /* * pull in the delayed directory update * and the delayed inode item |