Commit 162e0a16 authored by Josef Bacik's avatar Josef Bacik Committed by David Sterba

btrfs: if we're restriping, use the target restripe profile

Previously we depended on some weird behavior in our chunk allocator to
force the allocation of new stripes, so by the time we got to doing the
reduce we would usually already have a chunk with the proper target.

However that behavior causes other problems and needs to be removed.
First however we need to remove this check to only restripe if we
already have those available profiles, because if we're allocating our
first chunk it obviously will not be available.  Simply use the target
as specified, and if that fails it'll be because we're out of space.
Tested-by: default avatarHolger Hoffstätte <holger@applied-asynchrony.com>
Signed-off-by: default avatarJosef Bacik <josef@toxicpanda.com>
Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
parent 349e120e
...@@ -65,11 +65,8 @@ static u64 btrfs_reduce_alloc_profile(struct btrfs_fs_info *fs_info, u64 flags) ...@@ -65,11 +65,8 @@ static u64 btrfs_reduce_alloc_profile(struct btrfs_fs_info *fs_info, u64 flags)
spin_lock(&fs_info->balance_lock); spin_lock(&fs_info->balance_lock);
target = get_restripe_target(fs_info, flags); target = get_restripe_target(fs_info, flags);
if (target) { if (target) {
/* Pick target profile only if it's already available */ spin_unlock(&fs_info->balance_lock);
if ((flags & target) & BTRFS_EXTENDED_PROFILE_MASK) { return extended_to_chunk(target);
spin_unlock(&fs_info->balance_lock);
return extended_to_chunk(target);
}
} }
spin_unlock(&fs_info->balance_lock); spin_unlock(&fs_info->balance_lock);
......
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