When filesystem is unmounted while there is still some recovery work
going on, it can happen that quotas get disabled before quota recovery
is complete resulting in failed quota recovery and inconsistent quota
accounting. Move disabling of recovery in ocfs2_dismount_volume() before
disabling of quotas to fix this race.
Signed-off-by: Jan Kara <j...@suse.cz>
---
fs/ocfs2/super.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c
index ffa4952d432b..14c3d5ee6e24 100644
--- a/fs/ocfs2/super.c
+++ b/fs/ocfs2/super.c
@@ -1904,6 +1904,13 @@ static void ocfs2_dismount_volume(struct super_block
*sb, int mnt_err)
/* Orphan scan should be stopped as early as possible */
ocfs2_orphan_scan_stop(osb);
+ /*
+ * This will disable recovery and flush any recovery work. This needs
+ * to happen before disabling quotas as quota recovery needs quotas
+ * enabled.
+ */
+ ocfs2_recovery_exit(osb);
+
ocfs2_disable_quotas(osb);
/* All dquots should be freed by now */
@@ -1915,9 +1922,6 @@ static void ocfs2_dismount_volume(struct super_block *sb,
int mnt_err)
ocfs2_truncate_log_shutdown(osb);
- /* This will disable recovery and flush any recovery work. */
- ocfs2_recovery_exit(osb);
-
ocfs2_journal_shutdown(osb);
ocfs2_sync_blockdev(sb);
--
2.13.6
_______________________________________________
Ocfs2-devel mailing list
Ocfs2-devel@oss.oracle.com
https://oss.oracle.com/mailman/listinfo/ocfs2-devel
