xfs
[Top] [All Lists]

Re: [PATCH v7 1/7] xfs: create wrappers for converting kuid_t to/from ui

To: Dwight Engen <dwight.engen@xxxxxxxxxx>
Subject: Re: [PATCH v7 1/7] xfs: create wrappers for converting kuid_t to/from uid_t
From: Gao feng <gaofeng@xxxxxxxxxxxxxx>
Date: Wed, 31 Jul 2013 14:36:08 +0800
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20130729230640.5131f5cc@xxxxxxxxxx>
References: <20130729230640.5131f5cc@xxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7
On 07/30/2013 11:06 AM, Dwight Engen wrote:
> Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
> Signed-off-by: Dwight Engen <dwight.engen@xxxxxxxxxx>
> ---
>  fs/xfs/xfs_linux.h | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
> 

looks good to me, thanks!
Reviewed-by: Gao feng <gaofeng@xxxxxxxxxxxxxx>

> diff --git a/fs/xfs/xfs_linux.h b/fs/xfs/xfs_linux.h
> index 800f896..761e4c0 100644
> --- a/fs/xfs/xfs_linux.h
> +++ b/fs/xfs/xfs_linux.h
> @@ -159,6 +159,32 @@
>  #define MAX(a,b)     (max(a,b))
>  #define howmany(x, y)        (((x)+((y)-1))/(y))
>  
> +/* Kernel uid/gid conversion. These are used to convert to/from the on disk
> + * uid_t/gid_t types to the kuid_t/kgid_t types that the kernel uses 
> internally.
> + * The conversion here is type only, the value will remain the same since we
> + * are converting to the init_user_ns. The uid is later mapped to a 
> particular
> + * user namespace value when crossing the kernel/user boundary.
> + */
> +static inline __uint32_t xfs_kuid_to_uid(kuid_t uid)
> +{
> +     return from_kuid(&init_user_ns, uid);
> +}
> +
> +static inline kuid_t xfs_uid_to_kuid(__uint32_t uid)
> +{
> +     return make_kuid(&init_user_ns, uid);
> +}
> +
> +static inline __uint32_t xfs_kgid_to_gid(kgid_t gid)
> +{
> +     return from_kgid(&init_user_ns, gid);
> +}
> +
> +static inline kgid_t xfs_gid_to_kgid(__uint32_t gid)
> +{
> +     return make_kgid(&init_user_ns, gid);
> +}
> +
>  /*
>   * Various platform dependent calls that don't fit anywhere else
>   */
> 

<Prev in Thread] Current Thread [Next in Thread>