Re: [PATCH 02/32] libxfs: fix missing filetype updates to xfs_dir2.c

Eric Sandeen
2013-10-08T22:53:41Z

On 9/29/13 10:15 PM, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> They were missed in the original patch that was committed.

I'm not sure it's required in lookup or remove, at least today
nothing looks at it on those paths, does it?

Anyway, matches kernel, at worst the last 2 are harmless:

Reviewed-by: Eric Sandeen <sandeen@redhat.com>

> Signed-off-by: Dave Chinner <dchinner@redhat.com>
> ---
> libxfs/xfs_dir2.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/libxfs/xfs_dir2.c b/libxfs/xfs_dir2.c
> index 010c701..96a3c1d 100644
> --- a/libxfs/xfs_dir2.c
> +++ b/libxfs/xfs_dir2.c
> @@ -194,6 +194,7 @@ xfs_dir_createname(
> memset(&args, 0, sizeof(xfs_da_args_t));
> args.name = name->name;
> args.namelen = name->len;
> + args.filetype = name->type;
> args.hashval = dp->i_mount->m_dirnameops->hashname(name);
> args.inumber = inum;
> args.dp = dp;
> @@ -268,6 +269,7 @@ xfs_dir_lookup(
> memset(&args, 0, sizeof(xfs_da_args_t));
> args.name = name->name;
> args.namelen = name->len;
> + args.filetype = name->type;
> args.hashval = dp->i_mount->m_dirnameops->hashname(name);
> args.dp = dp;
> args.whichfork = XFS_DATA_FORK;
> @@ -323,6 +325,7 @@ xfs_dir_removename(
> memset(&args, 0, sizeof(xfs_da_args_t));
> args.name = name->name;
> args.namelen = name->len;
> + args.filetype = name->type;
> args.hashval = dp->i_mount->m_dirnameops->hashname(name);
> args.inumber = ino;
> args.dp = dp;
>