diff options
| author | Qu Wenruo <wqu@suse.com> | 2017-12-12 15:34:34 +0800 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2018-03-31 02:01:04 +0200 |
| commit | 8287475a20552af66b32c07704dbdbeeb898ac1f (patch) | |
| tree | a47e9a69b67d9399ae17f3ba5f093ce796ede144 /fs/btrfs/ctree.h | |
| parent | 4f5427ccce5d9cb8e2c8f98b49e744e523d246ec (diff) | |
| download | lwn-8287475a20552af66b32c07704dbdbeeb898ac1f.tar.gz lwn-8287475a20552af66b32c07704dbdbeeb898ac1f.zip | |
btrfs: qgroup: Use root::qgroup_meta_rsv_* to record qgroup meta reserved space
For quota disabled->enable case, it's possible that at reservation time
quota was not enabled so no bytes were really reserved, while at release
time, quota was enabled so we will try to release some bytes we didn't
really own.
Such situation can cause metadata reserveation underflow, for both types,
also less possible for per-trans type since quota enable will commit
transaction.
To address this, record qgroup meta reserved bytes into
root::qgroup_meta_rsv_pertrans and ::prealloc.
So at releasing time we won't free any bytes we didn't reserve.
For DATA, it's already handled by io_tree, so nothing needs to be done
there.
Signed-off-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/ctree.h')
| -rw-r--r-- | fs/btrfs/ctree.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index 7924e50cc528..0eb55825862a 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -1264,6 +1264,11 @@ struct btrfs_root { int send_in_progress; struct btrfs_subvolume_writers *subv_writers; atomic_t will_be_snapshotted; + + /* For qgroup metadata reserved space */ + spinlock_t qgroup_meta_rsv_lock; + u64 qgroup_meta_rsv_pertrans; + u64 qgroup_meta_rsv_prealloc; }; struct btrfs_file_private { |