Btrfs: fix a mismerge in btrfs_balance()
authorIlya Dryomov <idryomov@gmail.com>
Wed, 6 Mar 2013 08:57:55 +0000 (01:57 -0700)
committerChris Mason <chris.mason@fusionio.com>
Thu, 7 Mar 2013 03:03:16 +0000 (22:03 -0500)
Raid56 merge (merge commit e942f88) had mistakenly removed a call to
__cancel_balance(), which resulted in balance not cleaning up after itself
after a successful finish.  (Cleanup includes switching the state, removing
the balance item and releasing mut_ex_op testnset lock.)  Bring it back.

Reported-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
fs/btrfs/volumes.c

index 9ff454df675612d9785bbef9647bd8a6f4adcdc3..6b9cff42265d06e5b26a7ed53684a5551675a930 100644 (file)
@@ -3235,6 +3235,11 @@ int btrfs_balance(struct btrfs_balance_control *bctl,
                update_ioctl_balance_args(fs_info, 0, bargs);
        }
 
+       if ((ret && ret != -ECANCELED && ret != -ENOSPC) ||
+           balance_need_close(fs_info)) {
+               __cancel_balance(fs_info);
+       }
+
        wake_up(&fs_info->balance_wait_q);
 
        return ret;