Re: [PATCH 1/2] simplify xfs_vn_listxattr

[Timothy Shimmin <tes@sgi.com>]

Christoph Hellwig wrote:

>  
>  	/*
>  	 * Then add the two synthetic ACL attributes.
> @@ -265,7 +316,7 @@ xfs_vn_listxattr(struct dentry *dentry, 
>  	if (xfs_acl_vhasacl_access(inode)) {
>  		error = list_one_attr(POSIX_ACL_XATTR_ACCESS,
>  				strlen(POSIX_ACL_XATTR_ACCESS) + 1,
> -				data, size, &result);
> +				data, size, &context.count);
>  		if (error)
>  			return error;
>  	}
> @@ -273,10 +324,10 @@ xfs_vn_listxattr(struct dentry *dentry, 
>  	if (xfs_acl_vhasacl_default(inode)) {
>  		error = list_one_attr(POSIX_ACL_XATTR_DEFAULT,
>  				strlen(POSIX_ACL_XATTR_DEFAULT) + 1,
> -				data, size, &result);
> +				data, size, &context.count);
>  		if (error)

Need to fix param type.

re:

static int
list_one_attr(const char *name, const size_t len, void *data,
                size_t size, ssize_t *result)

typedef struct xfs_attr_list_context {
        struct xfs_inode                *dp;            /* inode */
        struct attrlist_cursor_kern     *cursor;        /* position in list */
        char                            *alist;         /* output buffer */
        int                             seen_enough;    /* T/F: seen enough of list? */
        int                             count;          /* num used entries */

=> mismatch type on &context.count (int) and list_one_attr's result
param (ssize_t => long)

--Tim