Re: [PATCH V2 2/6] Btrfs: remove some unnecessary spin_lock usages

2013-03-30 Thread Arne Jansen
On 03/28/13 11:54, Wang Shilong wrote: From: Wang Shilong wangsl-f...@cn.fujitsu.com We use mutex_lock to protect all the user change operaions. So when we are calling find_qgroup_rb() to check whether qgroup exists, we don't have to hold spin_lock. Besides, when enabling/disabling

[PATCH V2 2/6] Btrfs: remove some unnecessary spin_lock usages

2013-03-28 Thread Wang Shilong
From: Wang Shilong wangsl-f...@cn.fujitsu.com We use mutex_lock to protect all the user change operaions. So when we are calling find_qgroup_rb() to check whether qgroup exists, we don't have to hold spin_lock. Besides, when enabling/disabling quota,it must be single thread when operations come