Commit 07103a3c authored by Josef Bacik's avatar Josef Bacik Committed by Chris Mason

Btrfs: fix free space accounting when unpinning extents

This patch fixes what I hope is the last early ENOSPC bug left.  I did not know
that pinned extents would merge into one big extent when inserted on to the
pinned extent tree, so I was adding free space to a block group that could
possibly span multiple block groups.

This is a big issue because first that space doesn't exist in that block group,
and second we won't actually use that space because there are a bunch of other
checks to make sure we're allocating within the constraints of the block group.

This patch fixes the problem by adding the btrfs_add_free_space to
btrfs_update_pinned_extents which makes sure we are adding the appropriate
amount of free space to the appropriate block group.  Thanks much to Lee Trager
for running my myriad of debug patches to help me track this problem down.
Thank you,
Signed-off-by: default avatarJosef Bacik <jbacik@redhat.com>
parent 7c2ca468
...@@ -2034,6 +2034,8 @@ int btrfs_update_pinned_extents(struct btrfs_root *root, ...@@ -2034,6 +2034,8 @@ int btrfs_update_pinned_extents(struct btrfs_root *root,
spin_unlock(&cache->lock); spin_unlock(&cache->lock);
spin_unlock(&cache->space_info->lock); spin_unlock(&cache->space_info->lock);
fs_info->total_pinned -= len; fs_info->total_pinned -= len;
if (cache->cached)
btrfs_add_free_space(cache, bytenr, len);
} }
bytenr += len; bytenr += len;
num -= len; num -= len;
...@@ -2099,7 +2101,6 @@ int btrfs_finish_extent_commit(struct btrfs_trans_handle *trans, ...@@ -2099,7 +2101,6 @@ int btrfs_finish_extent_commit(struct btrfs_trans_handle *trans,
u64 start; u64 start;
u64 end; u64 end;
int ret; int ret;
struct btrfs_block_group_cache *cache;
mutex_lock(&root->fs_info->pinned_mutex); mutex_lock(&root->fs_info->pinned_mutex);
while(1) { while(1) {
...@@ -2109,9 +2110,6 @@ int btrfs_finish_extent_commit(struct btrfs_trans_handle *trans, ...@@ -2109,9 +2110,6 @@ int btrfs_finish_extent_commit(struct btrfs_trans_handle *trans,
break; break;
btrfs_update_pinned_extents(root, start, end + 1 - start, 0); btrfs_update_pinned_extents(root, start, end + 1 - start, 0);
clear_extent_dirty(unpin, start, end, GFP_NOFS); clear_extent_dirty(unpin, start, end, GFP_NOFS);
cache = btrfs_lookup_block_group(root->fs_info, start);
if (cache->cached)
btrfs_add_free_space(cache, start, end - start + 1);
if (need_resched()) { if (need_resched()) {
mutex_unlock(&root->fs_info->pinned_mutex); mutex_unlock(&root->fs_info->pinned_mutex);
cond_resched(); cond_resched();
......
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