Re: [PATCH] btrfs: accept zero for balance usage filter
On Mon, Feb 11, 2013 at 01:33:52PM +0100, David Sterba wrote: The condition can be relaxed to accept also 0 which will delete unoccupied chunks and does not need space for the actual data relocation. Until there is an automatic empty chunk reclaim, we can use this as a last resort option under enospc. CC: Ilya Dryomov idryo...@gmail.com Signed-off-by: David Sterba dste...@suse.cz Josef, please don't pull this one into btrfs-next, it's been deprecated by Btrfs: allow for selecting only completely empty chunks from me. David and I discussed this on IRC, he is OK with it. Thanks, Ilya -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] btrfs: accept zero for balance usage filter
On Tue, Feb 12, 2013 at 06:36:17PM +0200, Ilya Dryomov wrote: Josef, please don't pull this one into btrfs-next, it's been deprecated by Btrfs: allow for selecting only completely empty chunks from me. David and I discussed this on IRC, he is OK with it. Ack. david -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] btrfs: accept zero for balance usage filter
The condition can be relaxed to accept also 0 which will delete unoccupied chunks and does not need space for the actual data relocation. Until there is an automatic empty chunk reclaim, we can use this as a last resort option under enospc. CC: Ilya Dryomov idryo...@gmail.com Signed-off-by: David Sterba dste...@suse.cz --- Also needs progs update, but is not required for the kernel side. fs/btrfs/volumes.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index 5cbb7f4..fb871d0 100644 --- a/fs/btrfs/volumes.c +++ b/fs/btrfs/volumes.c @@ -2623,7 +2623,10 @@ static int chunk_usage_filter(struct btrfs_fs_info *fs_info, u64 chunk_offset, user_thresh = div_factor_fine(cache-key.offset, bargs-usage); - if (chunk_used user_thresh) + pr_debug(btrfs: chunk_used %llu, user_thresh %llu\n, + (unsigned long long)chunk_used, + (unsigned long long)user_thresh); + if (chunk_used = user_thresh) ret = 0; btrfs_put_block_group(cache); -- 1.7.9 -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html