Commit ee1ac541 authored by Jan Kara's avatar Jan Kara

quota: Remove dqonoff_mutex from dquot_scan_active()

All callers of dquot_scan_active() now hold s_umount so we can rely on
that lock to protect us against quota state changes.
Signed-off-by: default avatarJan Kara <jack@suse.cz>
parent 2a64f80e
...@@ -572,7 +572,8 @@ int dquot_scan_active(struct super_block *sb, ...@@ -572,7 +572,8 @@ int dquot_scan_active(struct super_block *sb,
struct dquot *dquot, *old_dquot = NULL; struct dquot *dquot, *old_dquot = NULL;
int ret = 0; int ret = 0;
mutex_lock(&sb_dqopt(sb)->dqonoff_mutex); WARN_ON_ONCE(!rwsem_is_locked(&sb->s_umount));
spin_lock(&dq_list_lock); spin_lock(&dq_list_lock);
list_for_each_entry(dquot, &inuse_list, dq_inuse) { list_for_each_entry(dquot, &inuse_list, dq_inuse) {
if (!test_bit(DQ_ACTIVE_B, &dquot->dq_flags)) if (!test_bit(DQ_ACTIVE_B, &dquot->dq_flags))
...@@ -603,7 +604,6 @@ int dquot_scan_active(struct super_block *sb, ...@@ -603,7 +604,6 @@ int dquot_scan_active(struct super_block *sb,
spin_unlock(&dq_list_lock); spin_unlock(&dq_list_lock);
out: out:
dqput(old_dquot); dqput(old_dquot);
mutex_unlock(&sb_dqopt(sb)->dqonoff_mutex);
return ret; return ret;
} }
EXPORT_SYMBOL(dquot_scan_active); EXPORT_SYMBOL(dquot_scan_active);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment