Revert "nfs: ignore SB_RDONLY when remounting nfs"

[ Upstream commit 400fa37afbb11a601c204b72af0f0e5bc2db695c ]

This reverts commit 80c4de6ab44c14e910117a02f2f8241ffc6ec54a.

Silently ignoring the "ro" and "rw" mount options causes user confusion,
and regressions.

Reported-by: Alkis Georgopoulos<alkisg@gmail.com>
Cc: Li Lingfeng <lilingfeng3@huawei.com>
Fixes: 80c4de6ab44c ("nfs: ignore SB_RDONLY when remounting nfs")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/fs/nfs/super.c b/fs/nfs/super.c
index cc70800b9a4b20056c6f5f90ecaddb4b4acec61e..aa11a6dcf6ce717c6c7ea5ddfabe5e05087bd6b8 100644 (file)
--- a/fs/nfs/super.c
+++ b/fs/nfs/super.c
@@ -1017,16 +1017,6 @@ int nfs_reconfigure(struct fs_context *fc)
 
        sync_filesystem(sb);
 
-       /*
-        * The SB_RDONLY flag has been removed from the superblock during
-        * mounts to prevent interference between different filesystems.
-        * Similarly, it is also necessary to ignore the SB_RDONLY flag
-        * during reconfiguration; otherwise, it may also result in the
-        * creation of redundant superblocks when mounting a directory with
-        * different rw and ro flags multiple times.
-        */
-       fc->sb_flags_mask &= ~SB_RDONLY;
-
        /*
         * Userspace mount programs that send binary options generally send
         * them populated with default values. We have no way to know which