Josef Bacik wrote: > 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 <[email protected]>
Reviewed-by: Li Zefan <[email protected]> > --- > 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 */ -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
