[PATCH 4/8] btrfs: cleanup the target logic in __btrfs_block_rsv_release

[Josef Bacik]

This works for all callers already, but if we wanted to use the helper
for the global_block_rsv it would end up trying to refill itself.  Fix
the logic to be able to be used no matter which block rsv is passed in
to this helper.

Signed-off-by: Josef Bacik <jo...@toxicpanda.com>
---
 fs/btrfs/extent-tree.c | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c
index d6aff56337aa..6995edf887e1 100644
--- a/fs/btrfs/extent-tree.c
+++ b/fs/btrfs/extent-tree.c
@@ -4684,12 +4684,20 @@ u64 __btrfs_block_rsv_release(struct btrfs_fs_info 
*fs_info,
 {
        struct btrfs_block_rsv *global_rsv = &fs_info->global_block_rsv;
        struct btrfs_block_rsv *delayed_rsv = &fs_info->delayed_refs_rsv;
-       struct btrfs_block_rsv *target = delayed_rsv;
+       struct btrfs_block_rsv *target = NULL;
 
-       if (target->full || target == block_rsv)
+       /*
+        * If we are the delayed_rsv then push to the global rsv, otherwise dump
+        * into the delayed rsv if it is not full.
+        */
+       if (block_rsv == delayed_rsv) {
                target = global_rsv;
+       } else if (block_rsv != global_rsv) {
+               if (!delayed_rsv->full)
+                       target = delayed_rsv;
+       }
 
-       if (block_rsv->space_info != target->space_info)
+       if (target && block_rsv->space_info != target->space_info)
                target = NULL;
 
        return block_rsv_release_bytes(fs_info, block_rsv, target, num_bytes,
-- 
2.14.3