On Wed, Jul 28, 2021 at 12:08 PM Ronnie Sahlberg <lsahlber@xxxxxxxxxx> wrote: > > We lost parsing of backupuid in the switch to new mount API. > Add it back. > > Signed-off-by: Ronnie Sahlberg <lsahlber@xxxxxxxxxx> > --- > fs/cifs/fs_context.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/fs/cifs/fs_context.c b/fs/cifs/fs_context.c > index 9a59d7ff9a11..eed59bc1d913 100644 > --- a/fs/cifs/fs_context.c > +++ b/fs/cifs/fs_context.c > @@ -925,6 +925,13 @@ static int smb3_fs_context_parse_param(struct fs_context *fc, > ctx->cred_uid = uid; > ctx->cruid_specified = true; > break; > + case Opt_backupuid: > + uid = make_kuid(current_user_ns(), result.uint_32); > + if (!uid_valid(uid)) > + goto cifs_parse_mount_err; > + ctx->backupuid = uid; > + ctx->backupuid_specified = true; > + break; > case Opt_backupgid: > gid = make_kgid(current_user_ns(), result.uint_32); > if (!gid_valid(gid)) > -- > 2.30.2 > Looks good to me. Might also be worthwhile to check if we missed any other mount option? -- Regards, Shyam
