diff mbox series

[21/42] btrfs: reset max_extent_size on clear in a bitmap

Message ID	20181011195431.3441-22-josef@toxicpanda.com (mailing list archive)
State	New, archived
Headers	show Return-Path: <linux-btrfs-owner@kernel.org> From: Josef Bacik <josef@toxicpanda.com> To: kernel-team@fb.com, linux-btrfs@vger.kernel.org Cc: Josef Bacik <jbacik@fb.com> Subject: [PATCH 21/42] btrfs: reset max_extent_size on clear in a bitmap Date: Thu, 11 Oct 2018 15:54:10 -0400 Message-Id: <20181011195431.3441-22-josef@toxicpanda.com> In-Reply-To: <20181011195431.3441-1-josef@toxicpanda.com> References: <20181011195431.3441-1-josef@toxicpanda.com> Sender: linux-btrfs-owner@vger.kernel.org Precedence: bulk
Series	My current patch queue \| expand [00/42,v4] My current patch queue [01/42] btrfs: add btrfs_delete_ref_head helper [02/42] btrfs: add cleanup_ref_head_accounting helper [03/42] btrfs: cleanup extent_op handling [04/42] btrfs: only track ref_heads in delayed_ref_updates [05/42] btrfs: only count ref heads run in __btrfs_run_delayed_refs [06/42] btrfs: introduce delayed_refs_rsv [07/42] btrfs: check if free bgs for commit [08/42] btrfs: dump block_rsv whe dumping space info [09/42] btrfs: release metadata before running delayed refs [10/42] btrfs: protect space cache inode alloc with nofs [11/42] btrfs: fix truncate throttling [12/42] btrfs: don't use global rsv for chunk allocation [13/42] btrfs: add ALLOC_CHUNK_FORCE to the flushing code [14/42] btrfs: reset max_extent_size properly [15/42] btrfs: don't enospc all tickets on flush failure [16/42] btrfs: loop in inode_rsv_refill [17/42] btrfs: run delayed iputs before committing [18/42] btrfs: move the dio_sem higher up the callchain [19/42] btrfs: set max_extent_size properly [20/42] btrfs: don't use ctl->free_space for max_extent_size [21/42] btrfs: reset max_extent_size on clear in a bitmap [22/42] btrfs: only run delayed refs if we're committing [23/42] btrfs: make sure we create all new bgs [24/42] btrfs: assert on non-empty delayed iputs [25/42] btrfs: pass delayed_refs_root to btrfs_delayed_ref_lock [26/42] btrfs: make btrfs_destroy_delayed_refs use btrfs_delayed_ref_lock [27/42] btrfs: make btrfs_destroy_delayed_refs use btrfs_delete_ref_head [28/42] btrfs: handle delayed ref head accounting cleanup in abort [29/42] btrfs: call btrfs_create_pending_block_groups unconditionally [30/42] btrfs: just delete pending bgs if we are aborted [31/42] btrfs: cleanup pending bgs on transaction abort [32/42] btrfs: only free reserved extent if we didn't insert it [33/42] btrfs: fix insert_reserved error handling [34/42] btrfs: wait on ordered extents on abort cleanup [35/42] MAINTAINERS: update my email address for btrfs [36/42] btrfs: wait on caching when putting the bg cache [37/42] btrfs: wakeup cleaner thread when adding delayed iput [38/42] btrfs: be more explicit about allowed flush states [39/42] btrfs: replace cleaner_delayed_iput_mutex with a waitqueue [40/42] btrfs: drop min_size from evict_refill_and_join [41/42] btrfs: reserve extra space during evict() [42/42] btrfs: don't run delayed_iputs in commit

Commit Message

Josef Bacik Oct. 11, 2018, 7:54 p.m. UTC

From: Josef Bacik <jbacik@fb.com>

We need to clear the max_extent_size when we clear bits from a bitmap
since it could have been from the range that contains the
max_extent_size.

Reviewed-by: Liu Bo <bo.liu@linux.alibaba.com>
Signed-off-by: Josef Bacik <jbacik@fb.com>
---
 fs/btrfs/free-space-cache.c | 2 ++
 1 file changed, 2 insertions(+)

diff mbox series

Patch

diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c
index 2e96ee7da3ec..d2a863a2ee24 100644
--- a/fs/btrfs/free-space-cache.c
+++ b/fs/btrfs/free-space-cache.c
@@ -1687,6 +1687,8 @@  static inline void __bitmap_clear_bits(struct btrfs_free_space_ctl *ctl,
 	bitmap_clear(info->bitmap, start, count);
 
 	info->bytes -= bytes;
+	if (info->max_extent_size > ctl->unit)
+		info->max_extent_size = 0;
 }
 
 static void bitmap_clear_bits(struct btrfs_free_space_ctl *ctl,