From: Sage Weil <sage@newdream.net>
To: hch@infradead.org, viro@ZenIV.linux.org.uk
Cc: linux-fsdevel@vger.kernel.org, Sage Weil <sage@newdream.net>,
Eric Van Hensbergen <ericvh@gmail.com>,
Ron Minnich <rminnich@sandia.gov>,
Latchesar Ionkov <lucho@ionkov.net>,
v9fs-developer@lists.sourceforge.net
Subject: [PATCH 18/26] 9p: remove unnecessary dentry_unhash on rmdir, dir rename
Date: Fri, 27 May 2011 13:42:11 -0700 [thread overview]
Message-ID: <1306528939-30636-19-git-send-email-sage@newdream.net> (raw)
In-Reply-To: <1306528939-30636-1-git-send-email-sage@newdream.net>
9p has no problems with references to unlinked directories.
CC: Eric Van Hensbergen <ericvh@gmail.com>
CC: Ron Minnich <rminnich@sandia.gov>
CC: Latchesar Ionkov <lucho@ionkov.net>
CC: v9fs-developer@lists.sourceforge.net
Signed-off-by: Sage Weil <sage@newdream.net>
---
fs/9p/vfs_inode.c | 4 ----
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c
index 8d7f3e6..7f6c677 100644
--- a/fs/9p/vfs_inode.c
+++ b/fs/9p/vfs_inode.c
@@ -814,7 +814,6 @@ int v9fs_vfs_unlink(struct inode *i, struct dentry *d)
int v9fs_vfs_rmdir(struct inode *i, struct dentry *d)
{
- dentry_unhash(d);
return v9fs_remove(i, d, 1);
}
@@ -840,9 +839,6 @@ v9fs_vfs_rename(struct inode *old_dir, struct dentry *old_dentry,
struct p9_fid *newdirfid;
struct p9_wstat wstat;
- if (new_dentry->d_inode && S_ISDIR(new_dentry->d_inode->i_mode))
- dentry_unhash(new_dentry);
-
P9_DPRINTK(P9_DEBUG_VFS, "\n");
retval = 0;
old_inode = old_dentry->d_inode;
--
1.7.0
next prev parent reply other threads:[~2011-05-27 20:41 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-27 20:41 [PATCH 00/26] Some final dentry_unhash cleanup Sage Weil
2011-05-27 20:41 ` [PATCH 01/26] bfs: remove unnecessary dentry_unhash on dir rename Sage Weil
2011-05-27 20:41 ` [PATCH 02/26] sysv: remove unnecessary dentry_unhash from rmdir, " Sage Weil
2011-05-27 20:41 ` [PATCH 03/26] jffs2: " Sage Weil
2011-05-27 20:41 ` [PATCH 04/26] jfs: " Sage Weil
2011-05-27 20:41 ` [PATCH 05/26] logfs: " Sage Weil
[not found] ` <1306528939-30636-1-git-send-email-sage-BnTBU8nroG7k1uMJSBkQmQ@public.gmane.org>
2011-05-27 20:41 ` [PATCH 06/26] nilfs2: " Sage Weil
2011-05-27 20:42 ` [PATCH 07/26] ubifs: " Sage Weil
2011-05-27 20:42 ` Sage Weil
2011-05-27 20:42 ` [PATCH 08/26] ufs: " Sage Weil
2011-05-27 20:42 ` [PATCH 09/26] reiserfs: " Sage Weil
2011-05-27 20:42 ` [PATCH 10/26] udf: " Sage Weil
2011-05-30 9:43 ` Jan Kara
2011-05-27 20:42 ` [PATCH 11/26] omfs: remove unnecessary dentry_unhash on rmdir, dir rneame Sage Weil
2011-06-05 0:42 ` Bob Copeland
2011-05-27 20:42 ` [PATCH 12/26] hfs: remove unnecessary dentry_unhash on rmdir, dir rename Sage Weil
2011-05-27 20:42 ` [PATCH 13/26] hfsplus: " Sage Weil
2011-05-27 20:42 ` [PATCH 14/26] hostfs: " Sage Weil
2011-05-27 20:42 ` [uml-devel] " Sage Weil
2011-05-27 20:42 ` [PATCH 15/26] ecryptfs: " Sage Weil
2011-05-27 20:42 ` [PATCH 16/26] ncpfs: document dentry_unhash usage Sage Weil
2011-05-27 20:42 ` [PATCH 17/26] ncpfs: fix rename over directory with dangling references Sage Weil
2011-05-27 20:42 ` Sage Weil [this message]
2011-05-27 20:42 ` [PATCH 19/26] affs: remove unnecessary dentry_unhash on rmdir, dir rename Sage Weil
2011-05-27 20:42 ` [PATCH 20/26] afs: " Sage Weil
2011-05-27 20:42 ` [PATCH 21/26] coda: " Sage Weil
2011-05-27 20:42 ` [PATCH 22/26] fuse: " Sage Weil
2011-05-27 20:42 ` [PATCH 23/26] minix: " Sage Weil
2011-05-27 20:42 ` [PATCH 24/26] hpfs: " Sage Weil
2011-05-27 20:42 ` [PATCH 25/26] fat: " Sage Weil
2011-05-28 6:37 ` OGAWA Hirofumi
2011-05-27 20:42 ` [PATCH 26/26] configfs: " Sage Weil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1306528939-30636-19-git-send-email-sage@newdream.net \
--to=sage@newdream.net \
--cc=ericvh@gmail.com \
--cc=hch@infradead.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=lucho@ionkov.net \
--cc=rminnich@sandia.gov \
--cc=v9fs-developer@lists.sourceforge.net \
--cc=viro@ZenIV.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.