diff mbox series

[14/42] btrfs: reset max_extent_size properly

Message ID	20181012193256.13735-15-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: linux-btrfs@vger.kernel.org, kernel-team@fb.com Subject: [PATCH 14/42] btrfs: reset max_extent_size properly Date: Fri, 12 Oct 2018 15:32:28 -0400 Message-Id: <20181012193256.13735-15-josef@toxicpanda.com> In-Reply-To: <20181012193256.13735-1-josef@toxicpanda.com> References: <20181012193256.13735-1-josef@toxicpanda.com> Sender: linux-btrfs-owner@vger.kernel.org Precedence: bulk
Series	My current patch queue \| expand [00/42,v5] 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. 12, 2018, 7:32 p.m. UTC

If we use up our block group before allocating a new one we'll easily
get a max_extent_size that's set really really low, which will result in
a lot of fragmentation.  We need to make sure we're resetting the
max_extent_size when we add a new chunk or add new space.

Reviewed-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: Josef Bacik <josef@toxicpanda.com>
---
 fs/btrfs/extent-tree.c | 2 ++
 1 file changed, 2 insertions(+)

diff mbox series

Patch

diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c
index 640ec640b21c..2b1704331d21 100644
--- a/fs/btrfs/extent-tree.c
+++ b/fs/btrfs/extent-tree.c
@@ -4573,6 +4573,7 @@  static int do_chunk_alloc(struct btrfs_trans_handle *trans, u64 flags,
 			goto out;
 	} else {
 		ret = 1;
+		space_info->max_extent_size = 0;
 	}
 
 	space_info->force_alloc = CHUNK_ALLOC_NO_FORCE;
@@ -6671,6 +6672,7 @@  static int btrfs_free_reserved_bytes(struct btrfs_block_group_cache *cache,
 		space_info->bytes_readonly += num_bytes;
 	cache->reserved -= num_bytes;
 	space_info->bytes_reserved -= num_bytes;
+	space_info->max_extent_size = 0;
 
 	if (delalloc)
 		cache->delalloc_bytes -= num_bytes;