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

Reply via email to