Commit bbb495c2 authored by Josef Bacik's avatar Josef Bacik

Btrfs: don't check bytes_pinned to determine if we should commit the transaction

Before the only reason to commit the transaction to recover space in
reserve_metadata_bytes() was if there were enough pinned_bytes to satisfy our
reservation.  But now we have the delayed inode stuff which will hold it's
reservations until we commit the transaction.  So say we max out our reservation
by creating a bunch of files but don't have any pinned bytes we will ENOSPC out
early even though we could commit the transaction and get that space back.  So
now just unconditionally commit the transaction since currently there is no way
to know how much metadata space is being reserved by delayed inode stuff.
Thanks,
Signed-off-by: default avatarJosef Bacik <josef@redhat.com>
parent ed3ee9f4
...@@ -3568,17 +3568,6 @@ static int reserve_metadata_bytes(struct btrfs_root *root, ...@@ -3568,17 +3568,6 @@ static int reserve_metadata_bytes(struct btrfs_root *root,
goto again; goto again;
} }
/*
* Not enough space to be reclaimed, don't bother committing the
* transaction.
*/
spin_lock(&space_info->lock);
if (space_info->bytes_pinned < orig_bytes)
ret = -ENOSPC;
spin_unlock(&space_info->lock);
if (ret)
goto out;
ret = -EAGAIN; ret = -EAGAIN;
if (trans) if (trans)
goto out; goto out;
......
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