[Cluster-devel] [GFS2 PATCH] gfs2: getlabel support

[Andrew Price]

On 21/08/18 04:21, Abhi Das wrote:
> Add support for the GETFSLABEL ioctl in gfs2.
> I tested this patch and it works as expected.
> 
> Signed-off-by: Steve Whitehouse <swhiteho at redhat.com>
> Tested-by: Abhi Das <adas at redhat.com>

The message should probably start with a From: so that git am uses the 
correct author. Looks good otherwise.

Cheers,
Andy

> ---
>   fs/gfs2/file.c | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
> 
> diff --git a/fs/gfs2/file.c b/fs/gfs2/file.c
> index 08369c6..6510f4e 100644
> --- a/fs/gfs2/file.c
> +++ b/fs/gfs2/file.c
> @@ -314,6 +314,17 @@ static int gfs2_set_flags(struct file *filp, u32 __user *ptr)
>   	return do_gfs2_set_flags(filp, gfsflags, mask);
>   }
>   
> +static int gfs2_getlabel(struct file *filp, char __user *label)
> +{
> +	struct inode *inode = file_inode(filp);
> +	struct gfs2_sbd *sdp = GFS2_SB(inode);
> +
> +	if (copy_to_user(label, sdp->sd_sb.sb_locktable, GFS2_LOCKNAME_LEN))
> +		return -EFAULT;
> +
> +	return 0;
> +}
> +
>   static long gfs2_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
>   {
>   	switch(cmd) {
> @@ -323,7 +334,10 @@ static long gfs2_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
>   		return gfs2_set_flags(filp, (u32 __user *)arg);
>   	case FITRIM:
>   		return gfs2_fitrim(filp, (void __user *)arg);
> +	case FS_IOC_GETFSLABEL:
> +		return gfs2_getlabel(filp, (char __user *)arg);
>   	}
> +
>   	return -ENOTTY;
>   }
>   
>