* [RFC][PATCH] btrfs: take the last remnants of ->d_fsdata use out
@ 2018-05-13 18:03 Al Viro
2018-05-14 10:39 ` David Sterba
0 siblings, 1 reply; 2+ messages in thread
From: Al Viro @ 2018-05-13 18:03 UTC (permalink / raw)
To: linux-fsdevel; +Cc: Chris Mason, linux-btrfs
[spotted while going through ->d_fsdata handling around d_splice_alias();
don't really care which tree that goes through]
The only thing even looking at ->d_fsdata in there (since 2012)
had been kfree(dentry->d_fsdata) in btrfs_dentry_delete(). Which,
incidentally, is all btrfs_dentry_delete() does.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index e064c49c9a9a..c6b2b0a20561 100644
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -5842,11 +5842,6 @@ static int btrfs_dentry_delete(const struct dentry *dentry)
return 0;
}
-static void btrfs_dentry_release(struct dentry *dentry)
-{
- kfree(dentry->d_fsdata);
-}
-
static struct dentry *btrfs_lookup(struct inode *dir, struct dentry *dentry,
unsigned int flags)
{
@@ -10665,5 +10660,4 @@ static const struct inode_operations btrfs_symlink_inode_operations = {
const struct dentry_operations btrfs_dentry_operations = {
.d_delete = btrfs_dentry_delete,
- .d_release = btrfs_dentry_release,
};
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [RFC][PATCH] btrfs: take the last remnants of ->d_fsdata use out
2018-05-13 18:03 [RFC][PATCH] btrfs: take the last remnants of ->d_fsdata use out Al Viro
@ 2018-05-14 10:39 ` David Sterba
0 siblings, 0 replies; 2+ messages in thread
From: David Sterba @ 2018-05-14 10:39 UTC (permalink / raw)
To: Al Viro; +Cc: linux-fsdevel, Chris Mason, linux-btrfs
On Sun, May 13, 2018 at 07:03:18PM +0100, Al Viro wrote:
> [spotted while going through ->d_fsdata handling around d_splice_alias();
> don't really care which tree that goes through]
>
> The only thing even looking at ->d_fsdata in there (since 2012)
> had been kfree(dentry->d_fsdata) in btrfs_dentry_delete(). Which,
> incidentally, is all btrfs_dentry_delete() does.
Right, it's a leftover after the DCACHE_NEED_LOOKUP removal. I'll add it
to btrfs tree, thanks.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-05-14 10:42 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-13 18:03 [RFC][PATCH] btrfs: take the last remnants of ->d_fsdata use out Al Viro
2018-05-14 10:39 ` David Sterba
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).