提交 7e355b83 编写于 作者: J Josef Bacik

Btrfs: if we have a lot of pinned space, commit the transaction

Mitch kept hitting a panic because he was getting ENOSPC.  One of my previous
patches makes it so we are much better at not allocating new metadata chunks.
Unfortunately coupled with the overcommit patch this works us into a bit of a
problem if we are removing a bunch of space and end up chewing up all of our
space with pinned extents.  We can allocate chunks fine and overflow is ok, but
the only way to reclaim this space is to commit the transaction.  So if we go to
overcommit, first check and see how much pinned space we have.  If we have more
than 80% of the free space chewed up with pinned extents, just commit the
transaction, this will free up enough space for our reservation and we won't
have this problem anymore.  With this patch Mitch's test doesn't blow up
anymore.  Thanks,
Reported-and-tested-by: NMitch Harder <mitch.harder@sabayonlinux.org>
Signed-off-by: NJosef Bacik <josef@redhat.com>
上级 36ba022a
...@@ -3510,6 +3510,20 @@ static int reserve_metadata_bytes(struct btrfs_root *root, ...@@ -3510,6 +3510,20 @@ static int reserve_metadata_bytes(struct btrfs_root *root,
u64 profile = btrfs_get_alloc_profile(root, 0); u64 profile = btrfs_get_alloc_profile(root, 0);
u64 avail; u64 avail;
/*
* If we have a lot of space that's pinned, don't bother doing
* the overcommit dance yet and just commit the transaction.
*/
avail = (space_info->total_bytes - space_info->bytes_used) * 8;
do_div(avail, 10);
if (space_info->bytes_pinned >= avail && flush && !trans &&
!committed) {
space_info->flush = 1;
flushing = true;
spin_unlock(&space_info->lock);
goto commit;
}
spin_lock(&root->fs_info->free_chunk_lock); spin_lock(&root->fs_info->free_chunk_lock);
avail = root->fs_info->free_chunk_space; avail = root->fs_info->free_chunk_space;
...@@ -3581,6 +3595,7 @@ static int reserve_metadata_bytes(struct btrfs_root *root, ...@@ -3581,6 +3595,7 @@ static int reserve_metadata_bytes(struct btrfs_root *root,
if (trans) if (trans)
goto out; goto out;
commit:
ret = -ENOSPC; ret = -ENOSPC;
if (committed) if (committed)
goto out; goto out;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册