On Thu, Mar 18, 2021 at 07:27:23AM +0100, Christoph Hellwig wrote:
> > -static inline kuid_t fsuid_into_mnt(struct user_namespace *mnt_userns)
> > +static inline kuid_t idmapped_fsuid(struct user_namespace *mnt_userns)
> > {
> > return kuid_from_mnt(mnt_userns, current_fsuid());
> > }
> >
> > -static inline kgid_t fsgid_into_mnt(struct user_namespace *mnt_userns)
> > +static inline kgid_t idmapped_fsgid(struct user_namespace *mnt_userns)
> > {
> > return kgid_from_mnt(mnt_userns, current_fsgid());
> > }
>
> I'm not sure the naming is an improvement. I always think of
> identity mapped when reading it, which couldn't be further from what
> it does.. But either way comments describing what these helpers do
> would be very useful.
s/idmapped/mapped/?
