Commit 7f701508 authored by Josef Bacik's avatar Josef Bacik

Btrfs: don't increase the block_rsv's size when emergency allocating space

If we have to emergency reserve space we need to not increase the block_rsv
size, otherwise we'll leak space.  Take for instance delalloc, say we reserve
4k, and we use that 4k, and then we have to emergency allocate another 4k, we
bump the size up to 8k, however we've only accounted for 4k in reservations in
all of our supporting logic, so we'll go to free the 4k and end up having a size
of 4k, which will cause us to later not free as much space.  I saw this doing
testing where I wasn't reserving enough space for something but was still
leaking space, very frustrating.  Thanks,
Signed-off-by: default avatarJosef Bacik <josef@redhat.com>
parent 7ed49f18
...@@ -5707,9 +5707,6 @@ use_block_rsv(struct btrfs_trans_handle *trans, ...@@ -5707,9 +5707,6 @@ use_block_rsv(struct btrfs_trans_handle *trans,
ret = reserve_metadata_bytes(trans, root, block_rsv, blocksize, ret = reserve_metadata_bytes(trans, root, block_rsv, blocksize,
0); 0);
if (!ret) { if (!ret) {
spin_lock(&block_rsv->lock);
block_rsv->size += blocksize;
spin_unlock(&block_rsv->lock);
return block_rsv; return block_rsv;
} else if (ret && block_rsv != global_rsv) { } else if (ret && block_rsv != global_rsv) {
ret = block_rsv_use_bytes(global_rsv, blocksize); ret = block_rsv_use_bytes(global_rsv, blocksize);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment