Re: [PATCH v4 4/7] smb: client: block cache=ro and cache=singleclient on remount

[Meetakshi Setiya]

Reviewed-by: Meetakshi Setiya <msetiya@xxxxxxxxxxxxx>

On Mon, Jun 15, 2026 at 10:54 PM <rajasimandalos@xxxxxxxxx> wrote:
>
> From: Rajasi Mandal <rajasimandal@xxxxxxxxxxxxx>
>
> cache=ro and cache=singleclient are mount-time environment declarations
> where the admin promises that the share is read-only or exclusively
> accessed.  The client bypasses server-based coherency (oplocks/leases)
> and caches aggressively based on this promise.
>
> These modes were intentionally excluded from smb3_update_mnt_flags()
> when it was introduced in commit 2d39f50c2b15 ("cifs: move update of
> flags into a separate function") — only cache=strict, cache=none and
> cache=loose were made reconfigurable.  However, remount currently
> silently accepts cache=ro and cache=singleclient without actually
> applying them, which is confusing.
>
> Add explicit checks in smb3_verify_reconfigure_ctx() to reject
> attempts to change these options during remount with a clear error
> message.
>
> Signed-off-by: Rajasi Mandal <rajasimandal@xxxxxxxxxxxxx>
> Reviewed-by: Meetakshi Setiya <msetiya@xxxxxxxxxxxxx>
> ---
>  fs/smb/client/fs_context.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/fs/smb/client/fs_context.c b/fs/smb/client/fs_context.c
> index bfbea0648f6a..659fb0adb528 100644
> --- a/fs/smb/client/fs_context.c
> +++ b/fs/smb/client/fs_context.c
> @@ -1211,6 +1211,15 @@ static int smb3_verify_reconfigure_ctx(struct fs_context *fc,
>                 cifs_errorf(fc, "can not change rdma during remount\n");
>                 return -EINVAL;
>         }
> +       /* init default: cache_ro = false, cache_rw = false (i.e. cache=strict) */
> +       if (new_ctx->cache_ro != old_ctx->cache_ro) {
> +               cifs_errorf(fc, "can not change cache=ro during remount\n");
> +               return -EINVAL;
> +       }
> +       if (new_ctx->cache_rw != old_ctx->cache_rw) {
> +               cifs_errorf(fc, "can not change cache=singleclient during remount\n");
> +               return -EINVAL;
> +       }
>
>         return 0;
>  }
> --
> 2.43.0
>