Revert "Unionfs: Check remount options for being NULL"
authorErez_Zadok <ezk@cs.sunysb.edu>
Fri, 27 Apr 2007 16:35:44 +0000 (12:35 -0400)
committerErez Zadok <ezk@cs.sunysb.edu>
Sat, 16 Aug 2014 02:25:59 +0000 (22:25 -0400)
This reverts commit 49491b290f0f791a531316c6197437547985537b.

fs/unionfs/super.c

index 199b4777319155d62f148b1367b70f3b9e66451b..10520c0fbf39c6c6b8d0d316f1a307e6f92ded22 100644 (file)
@@ -430,12 +430,11 @@ static int unionfs_remount_fs(struct super_block *sb, int *flags,
        unionfs_write_lock(sb);
 
        /*
-        * The VFS will take care of "ro" and "rw" flags, and we can safely
-        * ignore MS_SILENT, but anything else left over is an error.  So we
-        * need to check if any other flags may have been passed (none are
-        * allowed/supported as of now).
+        * The VFS will take care of "ro" and "rw" flags, so anything else
+        * is an error.  So we need to check if any other flags may have
+        * been passed (none are allowed/supported as of now).
         */
-       if ((*flags & ~(MS_RDONLY | MS_SILENT)) != 0) {
+       if ((*flags & ~MS_RDONLY) != 0) {
                printk(KERN_WARNING
                       "unionfs: remount flags 0x%x unsupported\n", *flags);
                err = -EINVAL;
@@ -737,8 +736,7 @@ out_no_change:
        i = atomic_inc_return(&UNIONFS_SB(sb)->generation);
        atomic_set(&UNIONFS_D(sb->s_root)->generation, i);
        atomic_set(&UNIONFS_I(sb->s_root->d_inode)->generation, i);
-       if (!(*flags & MS_SILENT)) 
-               printk("unionfs: new generation number %d\n", i);
+       printk("unionfs: new generation number %d\n", i);
        err = 0;                /* reset to success */
 
        /*