diff options
author | Josef Bacik <jbacik@fb.com> | 2015-02-03 07:50:16 -0800 |
---|---|---|
committer | Chris Mason <clm@fb.com> | 2015-04-10 14:04:47 -0700 |
commit | 1262133b8d6f10f5ca7621cd4cf65ddf6254126a (patch) | |
tree | ac508ea4fc6d1e9b394ac2bdfee04eb20f5fc930 /fs/btrfs/transaction.c | |
parent | 28ed1345a50491d78e1454ad4005dc5d3557a69e (diff) | |
download | op-kernel-dev-1262133b8d6f10f5ca7621cd4cf65ddf6254126a.zip op-kernel-dev-1262133b8d6f10f5ca7621cd4cf65ddf6254126a.tar.gz |
Btrfs: account for crcs in delayed ref processing
As we delete large extents, we end up doing huge amounts of COW in order
to delete the corresponding crcs. This adds accounting so that we keep
track of that space and flushing of delayed refs so that we don't build
up too much delayed crc work.
This helps limit the delayed work that must be done at commit time and
tries to avoid ENOSPC aborts because the crcs eat all the global
reserves.
Signed-off-by: Chris Mason <clm@fb.com>
Diffstat (limited to 'fs/btrfs/transaction.c')
-rw-r--r-- | fs/btrfs/transaction.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/btrfs/transaction.c b/fs/btrfs/transaction.c index ba831ee..8b9eea8 100644 --- a/fs/btrfs/transaction.c +++ b/fs/btrfs/transaction.c @@ -64,6 +64,9 @@ void btrfs_put_transaction(struct btrfs_transaction *transaction) if (atomic_dec_and_test(&transaction->use_count)) { BUG_ON(!list_empty(&transaction->list)); WARN_ON(!RB_EMPTY_ROOT(&transaction->delayed_refs.href_root)); + if (transaction->delayed_refs.pending_csums) + printk(KERN_ERR "pending csums is %llu\n", + transaction->delayed_refs.pending_csums); while (!list_empty(&transaction->pending_chunks)) { struct extent_map *em; @@ -223,6 +226,7 @@ loop: cur_trans->delayed_refs.href_root = RB_ROOT; atomic_set(&cur_trans->delayed_refs.num_entries, 0); cur_trans->delayed_refs.num_heads_ready = 0; + cur_trans->delayed_refs.pending_csums = 0; cur_trans->delayed_refs.num_heads = 0; cur_trans->delayed_refs.flushing = 0; cur_trans->delayed_refs.run_delayed_start = 0; |