fs/sysv: remove another useless write_super call
authorArtem Bityutskiy <artem.bityutskiy@linux.intel.com>
Tue, 3 Jul 2012 13:43:27 +0000 (16:43 +0300)
committerAl Viro <viro@zeniv.linux.org.uk>
Sun, 22 Jul 2012 19:58:11 +0000 (23:58 +0400)
We do not need to call 'sysv_write_super()' from 'sysv_remount()',
because VFS has called 'sysv_sync_fs()' before calling '->remount()'.
So remove it. Remove also '(un)lock_super()' which obvioulsy is becoming
useless in this function.

Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/sysv/inode.c

index af13d1342f233e937b64e4f41ea95ef0b1085e6d..f20ffe32b3982873702f27b035a124daf12a5bf7 100644 (file)
@@ -68,12 +68,9 @@ static void sysv_write_super(struct super_block *sb)
 static int sysv_remount(struct super_block *sb, int *flags, char *data)
 {
        struct sysv_sb_info *sbi = SYSV_SB(sb);
-       lock_super(sb);
+
        if (sbi->s_forced_ro)
                *flags |= MS_RDONLY;
-       if (*flags & MS_RDONLY)
-               sysv_write_super(sb);
-       unlock_super(sb);
        return 0;
 }