ext4: convert last user of ext4_mark_super_dirty() to ext4_handle_dirty_super()

The last user of ext4_mark_super_dirty() in ext4_file_open() is so
rare it can well be modifying the superblock properly by journalling
the change.  Change it and get rid of ext4_mark_super_dirty() as it's
not needed anymore.

Artem: small amendments.
Artem: tested using xfstests for both journalled and non-journalled ext4.

Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Tested-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>

1 files changed, 13 insertions, 1 deletions

diff --git a/fs/ext4/file.c b/fs/ext4/file.c
index 1c81509f5bd9..f77e795fed65 100644
--- a/fs/ext4/file.c
+++ b/fs/ext4/file.c
@@ -244,9 +244,21 @@ static int ext4_file_open(struct inode * inode, struct file * filp)
                 path.dentry = mnt->mnt_root;
                 cp = d_path(&path, buf, sizeof(buf));
                 if (!IS_ERR(cp)) {
+                        handle_t *handle;
+                        int err;
+
+                        handle = ext4_journal_start_sb(sb, 1);
+                        if (IS_ERR(handle))
+                                return PTR_ERR(handle);
+                        err = ext4_journal_get_write_access(handle, sbi->s_sbh);
+                        if (err) {
+                                ext4_journal_stop(handle);
+                                return err;
+                        }
                         strlcpy(sbi->s_es->s_last_mounted, cp,
                                 sizeof(sbi->s_es->s_last_mounted));
-                        ext4_mark_super_dirty(sb);
+                        ext4_handle_dirty_super(handle, sb);
+                        ext4_journal_stop(handle);
                 }
         }
         /*