btrfs: qgroup: Remove the unnecesaary spin lock for qgroup_rescan_running
After the previous patch, qgroup_rescan_running is protected by btrfs_fs_info::qgroup_rescan_lock, thus no need for the extra spinlock. Suggested-by: Josef Bacik <josef@toxicpanda.com> Signed-off-by: Qu Wenruo <wqu@suse.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
parent
d61acbbf54
commit
daf475c915
|
@ -3238,7 +3238,6 @@ qgroup_rescan_init(struct btrfs_fs_info *fs_info, u64 progress_objectid,
|
||||||
}
|
}
|
||||||
|
|
||||||
mutex_lock(&fs_info->qgroup_rescan_lock);
|
mutex_lock(&fs_info->qgroup_rescan_lock);
|
||||||
spin_lock(&fs_info->qgroup_lock);
|
|
||||||
|
|
||||||
if (init_flags) {
|
if (init_flags) {
|
||||||
if (fs_info->qgroup_flags & BTRFS_QGROUP_STATUS_FLAG_RESCAN) {
|
if (fs_info->qgroup_flags & BTRFS_QGROUP_STATUS_FLAG_RESCAN) {
|
||||||
|
@ -3253,7 +3252,6 @@ qgroup_rescan_init(struct btrfs_fs_info *fs_info, u64 progress_objectid,
|
||||||
}
|
}
|
||||||
|
|
||||||
if (ret) {
|
if (ret) {
|
||||||
spin_unlock(&fs_info->qgroup_lock);
|
|
||||||
mutex_unlock(&fs_info->qgroup_rescan_lock);
|
mutex_unlock(&fs_info->qgroup_rescan_lock);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
@ -3264,8 +3262,6 @@ qgroup_rescan_init(struct btrfs_fs_info *fs_info, u64 progress_objectid,
|
||||||
sizeof(fs_info->qgroup_rescan_progress));
|
sizeof(fs_info->qgroup_rescan_progress));
|
||||||
fs_info->qgroup_rescan_progress.objectid = progress_objectid;
|
fs_info->qgroup_rescan_progress.objectid = progress_objectid;
|
||||||
init_completion(&fs_info->qgroup_rescan_completion);
|
init_completion(&fs_info->qgroup_rescan_completion);
|
||||||
|
|
||||||
spin_unlock(&fs_info->qgroup_lock);
|
|
||||||
mutex_unlock(&fs_info->qgroup_rescan_lock);
|
mutex_unlock(&fs_info->qgroup_rescan_lock);
|
||||||
|
|
||||||
btrfs_init_work(&fs_info->qgroup_rescan_work,
|
btrfs_init_work(&fs_info->qgroup_rescan_work,
|
||||||
|
@ -3342,9 +3338,7 @@ int btrfs_qgroup_wait_for_completion(struct btrfs_fs_info *fs_info,
|
||||||
int ret = 0;
|
int ret = 0;
|
||||||
|
|
||||||
mutex_lock(&fs_info->qgroup_rescan_lock);
|
mutex_lock(&fs_info->qgroup_rescan_lock);
|
||||||
spin_lock(&fs_info->qgroup_lock);
|
|
||||||
running = fs_info->qgroup_rescan_running;
|
running = fs_info->qgroup_rescan_running;
|
||||||
spin_unlock(&fs_info->qgroup_lock);
|
|
||||||
mutex_unlock(&fs_info->qgroup_rescan_lock);
|
mutex_unlock(&fs_info->qgroup_rescan_lock);
|
||||||
|
|
||||||
if (!running)
|
if (!running)
|
||||||
|
|
Loading…
Reference in New Issue