Re: [PATCH 3/4] ceph: fix symlink inode operations

[Alex Elder <elder@inktank.com>]

On 04/12/2013 03:11 AM, Yan, Zheng wrote:
> From: "Yan, Zheng" <zheng.z.yan@intel.com>
> 
> add getattr/setattr and xattrs related methods.

Looks good.

Reviewed-by: Alex Elder <elder@inktank.com>

> Signed-off-by: Yan, Zheng <zheng.z.yan@intel.com>
> ---
>  fs/ceph/inode.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/fs/ceph/inode.c b/fs/ceph/inode.c
> index 5d730d4..d5cad38 100644
> --- a/fs/ceph/inode.c
> +++ b/fs/ceph/inode.c
> @@ -1523,6 +1523,12 @@ static void *ceph_sym_follow_link(struct dentry *dentry, struct nameidata *nd)
>  static const struct inode_operations ceph_symlink_iops = {
>  	.readlink = generic_readlink,
>  	.follow_link = ceph_sym_follow_link,
> +	.setattr = ceph_setattr,
> +	.getattr = ceph_getattr,
> +	.setxattr = ceph_setxattr,
> +	.getxattr = ceph_getxattr,
> +	.listxattr = ceph_listxattr,
> +	.removexattr = ceph_removexattr,
>  };
>  
>  /*
>