diff options
| author | Josef Bacik <josef@redhat.com> | 2011-03-18 15:13:42 -0400 |
|---|---|---|
| committer | Josef Bacik <josef@redhat.com> | 2011-03-21 10:25:48 -0400 |
| commit | 7d0d2e8e6b6f7da221a25238cf490a095c8c4788 (patch) | |
| tree | a2bce1c429f9efb9d3cc41390afa73839787c96c /fs | |
| parent | 22a94d44bd6876a90630338229da6c0436d46593 (diff) | |
| download | op-kernel-dev-7d0d2e8e6b6f7da221a25238cf490a095c8c4788.zip op-kernel-dev-7d0d2e8e6b6f7da221a25238cf490a095c8c4788.tar.gz | |
Btrfs: check free space in block group before searching for a cluster
The free space cluster stuff is heavy duty, so there is no sense in going
through the entire song and dance if there isn't enough space in the block group
to begin with. Thanks,
Signed-off-by: Josef Bacik <josef@redhat.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/btrfs/free-space-cache.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c index 0282033..f631df8 100644 --- a/fs/btrfs/free-space-cache.c +++ b/fs/btrfs/free-space-cache.c @@ -1999,6 +1999,16 @@ int btrfs_find_space_cluster(struct btrfs_trans_handle *trans, min_bytes = max(bytes, (bytes + empty_size) >> 2); spin_lock(&block_group->tree_lock); + + /* + * If we know we don't have enough space to make a cluster don't even + * bother doing all the work to try and find one. + */ + if (block_group->free_space < min_bytes) { + spin_unlock(&block_group->tree_lock); + return -ENOSPC; + } + spin_lock(&cluster->lock); /* someone already found a cluster, hooray */ |
