git://ftp.safe.ca / safe / jmp / linux-2.6 / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 2722610) | patch
nfs: clean up sillyrenaming in nfs_rename()
authorMiklos Szeredi <mszeredi@suse.cz>
Thu, 3 Dec 2009 20:58:56 +0000 (15:58 -0500)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Thu, 3 Dec 2009 20:58:56 +0000 (15:58 -0500)
commit24e93025ee434a58d35e5abb283c5bcc9a13e477
treef187100fdb63afb537e209c62e9c6bb5a95fc46btree | snapshot
parent27226104e60964f21717e0f452cecd45c85a64c6commit | diff
nfs: clean up sillyrenaming in nfs_rename()

The d_instantiate(new_dentry, NULL) is superfluous, the dentry is
already negative.  Rehashing this dummy dentry isn't needed either,
d_move() works fine on an unhashed target.

The re-checking for busy after a failed nfs_sillyrename() is bogus
too: new_dentry->d_count < 2 would be a bug here.

Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/dir.c diff | blob | history
Full containers within linux kernel