On Tuesday 20 August 2013, Dwight Engen wrote: > diff --git a/arch/powerpc/platforms/cell/spufs/inode.c b/arch/powerpc/platforms/cell/spufs/inode.c > index f390042..90fb308 100644 > --- a/arch/powerpc/platforms/cell/spufs/inode.c > +++ b/arch/powerpc/platforms/cell/spufs/inode.c > @@ -620,12 +620,12 @@ spufs_parse_options(struct super_block *sb, char *options, struct inode *root) > case Opt_uid: > if (match_int(&args[0], &option)) > return 0; > - root->i_uid = option; > + root->i_uid = make_kuid(&init_user_ns, option); > break; > case Opt_gid: > if (match_int(&args[0], &option)) > return 0; > - root->i_gid = option; > + root->i_gid = make_kgid(&init_user_ns, option); > break; > case Opt_mode: > if (match_octal(&args[0], &option)) Doesn't this mean the uid/gid is taken from the initial namespace rather than from the namespace of the 'mount' process calling this? I think the logical choice would be to have the UID be the one that gets passed here in the caller's namespace. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
