diff options
| author | Liu Bo <liubo2009@cn.fujitsu.com> | 2012-02-16 11:34:39 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.cz> | 2012-02-16 17:23:18 +0100 |
| commit | d9b0218f6cb682aa6a4ada2bfc5a25fdf3018563 (patch) | |
| tree | ea325470025698b1f9bfd8ff034d0d54ee89bb59 | |
| parent | Btrfs: kick out redundant stuff in convert_extent_bit (diff) | |
| download | linux-d9b0218f6cb682aa6a4ada2bfc5a25fdf3018563.tar.xz linux-d9b0218f6cb682aa6a4ada2bfc5a25fdf3018563.zip | |
Btrfs: fix a bug on overcommit stuff
When overcommitting, we should check the sum of pinned space and
bytes for delayed item.
Signed-off-by: Liu Bo <liubo2009@cn.fujitsu.com>
| -rw-r--r-- | fs/btrfs/extent-tree.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index 60bfe2d68547..dc083f55bcfd 100644 --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c @@ -3611,12 +3611,15 @@ static int may_commit_transaction(struct btrfs_root *root, if (space_info != delayed_rsv->space_info) return -ENOSPC; + spin_lock(&space_info->lock); spin_lock(&delayed_rsv->lock); - if (delayed_rsv->size < bytes) { + if (space_info->bytes_pinned + delayed_rsv->size < bytes) { spin_unlock(&delayed_rsv->lock); + spin_unlock(&space_info->lock); return -ENOSPC; } spin_unlock(&delayed_rsv->lock); + spin_unlock(&space_info->lock); commit: trans = btrfs_join_transaction(root); |