[Cluster-devel] [PATCH] GFS2: Hash the negative dentry during inode lookup
Steven Whitehouse
swhiteho at redhat.com
Thu Sep 11 12:42:13 UTC 2014
Hi,
On 10/09/14 19:09, Benjamin Coddington wrote:
> Fix a regression introduced by:
> 6d4ade986f9c8df31e68 GFS2: Add atomic_open support
> where an early return misses d_splice_alias() which had been
> adding the negative dentry.
>
> Signed-off-by: Benjamin Coddington <bcodding at redhat.com>
> ---
> fs/gfs2/inode.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c
> index e62e594..9317ddc 100644
> --- a/fs/gfs2/inode.c
> +++ b/fs/gfs2/inode.c
> @@ -840,8 +840,10 @@ static struct dentry *__gfs2_lookup(struct inode *dir, struct dentry *dentry,
> int error;
>
> inode = gfs2_lookupi(dir, &dentry->d_name, 0);
> - if (!inode)
> + if (inode == NULL) {
> + d_add(dentry, NULL);
> return NULL;
> + }
> if (IS_ERR(inode))
> return ERR_CAST(inode);
>
Now in the GFS2 -nmw tree, and I've added Bob's signed-off-by too. Thanks,
Steve.
More information about the Cluster-devel
mailing list