Btrfs: do not free the dirty bytes from the trans block rsv on cleanup
authorJosef Bacik <jbacik@fusionio.com>
Mon, 7 Oct 2013 14:45:07 +0000 (10:45 -0400)
committerChris Mason <chris.mason@fusionio.com>
Tue, 12 Nov 2013 02:55:58 +0000 (21:55 -0500)
The transactions should be cleaning up their reservations on failure, this just
causes us to have warnings on unmount because we go negative by free'ing
reservations that have already been free'ed.  Thanks,

Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
fs/btrfs/disk-io.c

index d205bddc7776f5bc48622b628096eab6d7a72c10..fdc75ab094838a242cbec897927497b60f391be0 100644 (file)
@@ -4046,8 +4046,6 @@ void btrfs_cleanup_one_transaction(struct btrfs_transaction *cur_trans,
        btrfs_destroy_ordered_operations(cur_trans, root);
 
        btrfs_destroy_delayed_refs(cur_trans, root);
-       btrfs_block_rsv_release(root, &root->fs_info->trans_block_rsv,
-                               cur_trans->dirty_pages.dirty_bytes);
 
        cur_trans->state = TRANS_STATE_COMMIT_START;
        wake_up(&root->fs_info->transaction_blocked_wait);