Commit 29dfeb0b authored Jul 27, 2019 by Al Viro

hypfs: don't bother with d_delete()



If that's not the last reference, d_delete() will do d_drop().
If it is, dput() immediately after it will unhash the sucker
anyway, since ->d_delete() the method is always_delete_dentry().

IOW, there's no point trying to turn it into a negative hashed
dentry - it won't stick around anyway.  Just d_drop() it and be
done with that.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

parent 84a2bd39

arch/s390/hypfs/inode.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -76,7 +76,7 @@ static void hypfs_remove(struct dentry *dentry)
		else
		simple_unlink(d_inode(parent), dentry);
		}
		d_delete(dentry);
		d_drop(dentry);
		dput(dentry);
		inode_unlock(d_inode(parent));
		}

Admin message