diff options
Diffstat (limited to 'fs/jffs2/dir.c')
-rw-r--r-- | fs/jffs2/dir.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/fs/jffs2/dir.c b/fs/jffs2/dir.c index d211b8e18566..30c4c9ebb693 100644 --- a/fs/jffs2/dir.c +++ b/fs/jffs2/dir.c | |||
@@ -843,9 +843,14 @@ static int jffs2_rename (struct inode *old_dir_i, struct dentry *old_dentry, | |||
843 | 843 | ||
844 | pr_notice("%s(): Link succeeded, unlink failed (err %d). You now have a hard link\n", | 844 | pr_notice("%s(): Link succeeded, unlink failed (err %d). You now have a hard link\n", |
845 | __func__, ret); | 845 | __func__, ret); |
846 | /* Might as well let the VFS know */ | 846 | /* |
847 | d_instantiate(new_dentry, d_inode(old_dentry)); | 847 | * We can't keep the target in dcache after that. |
848 | ihold(d_inode(old_dentry)); | 848 | * For one thing, we can't afford dentry aliases for directories. |
849 | * For another, if there was a victim, we _can't_ set new inode | ||
850 | * for that sucker and we have to trigger mount eviction - the | ||
851 | * caller won't do it on its own since we are returning an error. | ||
852 | */ | ||
853 | d_invalidate(new_dentry); | ||
849 | new_dir_i->i_mtime = new_dir_i->i_ctime = ITIME(now); | 854 | new_dir_i->i_mtime = new_dir_i->i_ctime = ITIME(now); |
850 | return ret; | 855 | return ret; |
851 | } | 856 | } |