Re: [PATCH 11/16] btrfs: kill btrfs_fs_info::volume_mutex

2018-04-13 Thread David Sterba
On Fri, Apr 13, 2018 at 01:30:22PM +0800, Anand Jain wrote: > > > > @@ -4569,8 +4560,10 @@ static long btrfs_ioctl_balance(struct file *file, > > void __user *arg) > > /* this is either (2) or (3) */ > > if (!atomic_read(_info->balance_running)) { > >

Re: [PATCH 11/16] btrfs: kill btrfs_fs_info::volume_mutex

2018-04-12 Thread Anand Jain
@@ -4569,8 +4560,10 @@ static long btrfs_ioctl_balance(struct file *file, void __user *arg) /* this is either (2) or (3) */ if (!atomic_read(_info->balance_running)) { mutex_unlock(_info->balance_mutex); - if

Re: [PATCH 11/16] btrfs: kill btrfs_fs_info::volume_mutex

2018-04-09 Thread Anand Jain
On 04/04/2018 02:34 AM, David Sterba wrote: Mutual exclusion of device add/rm and balance was done by the volume mutex up to version 3.7. The commit 5ac00addc7ac091109 ("Btrfs: disallow mutually exclusive admin operations from user mode") added a bit that essentially tracked the same

[PATCH 11/16] btrfs: kill btrfs_fs_info::volume_mutex

2018-04-03 Thread David Sterba
Mutual exclusion of device add/rm and balance was done by the volume mutex up to version 3.7. The commit 5ac00addc7ac091109 ("Btrfs: disallow mutually exclusive admin operations from user mode") added a bit that essentially tracked the same information. The status bit has an advantage over a