quota: Fix bogus warning in dquot_disable()
authorJan Kara <jack@suse.cz>
Mon, 19 Dec 2016 13:01:39 +0000 (14:01 +0100)
committerJan Kara <jack@suse.cz>
Mon, 19 Dec 2016 13:01:39 +0000 (14:01 +0100)
commit2700e6067c72a99d1b7037692da0145ac44623c4
tree0eba2f2c772188890a2841751487a9d490795541
parent5716863e0f8251d3360d4cbfc0e44e08007075df
quota: Fix bogus warning in dquot_disable()

dquot_disable() was warning when sb_has_quota_loaded() was true when
invalidating page cache for quota files. The thinking behind this
warning was that we must have raced with somebody else turning quotas on
and this should not happen because all places modifying quota state must
hold s_umount exclusively now. However sb_has_quota_loaded() can be also
true at this point when we are just suspending quotas on remount
read-only. Just restore the behavior to situation before commit
c3b004460d77 ("quota: Remove dqonoff_mutex") which introduced the
warning.

The code in dquot_disable() can be further simplified with the new
locking of quota state changes however let's leave that to a separate
commit that can get more testing exposure.

Fixes: c3b004460d77bf3f980d877be539016f2df4df12
Signed-off-by: Jan Kara <jack@suse.cz>
fs/quota/dquot.c