ext4: fix warning message in ext4_enable_quotas()
authorJunichi Uekawa <uekawa@google.com>
Sun, 29 Jul 2018 19:51:52 +0000 (15:51 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Sun, 29 Jul 2018 19:51:52 +0000 (15:51 -0400)
Output the warning message before we clobber type and be -1 all the time.
The error message would now be

[    1.519791] EXT4-fs warning (device vdb): ext4_enable_quotas:5402:
Failed to enable quota tracking (type=0, err=-3). Please run e2fsck to fix.

Signed-off-by: Junichi Uekawa <uekawa@google.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Reviewed-by: Andreas Dilger <adilger@dilger.ca>
fs/ext4/super.c

index 67f4310edd0abe4eb961eec84609033eff5baee8..3e64d8d51ac4a16c6b980f4702d8d107a39c34b0 100644 (file)
@@ -5686,13 +5686,13 @@ static int ext4_enable_quotas(struct super_block *sb)
                                DQUOT_USAGE_ENABLED |
                                (quota_mopt[type] ? DQUOT_LIMITS_ENABLED : 0));
                        if (err) {
-                               for (type--; type >= 0; type--)
-                                       dquot_quota_off(sb, type);
-
                                ext4_warning(sb,
                                        "Failed to enable quota tracking "
                                        "(type=%d, err=%d). Please run "
                                        "e2fsck to fix.", type, err);
+                               for (type--; type >= 0; type--)
+                                       dquot_quota_off(sb, type);
+
                                return err;
                        }
                }