From: Al Viro <viro@zeniv.linux.org.uk>
Date: Fri, 4 Mar 2011 06:14:55 +0000 (-0500)
Subject: omfs: rename() needs to mark old_inode dirty after ctime update
X-Git-Tag: v2.6.39-rc1~420^2~3
X-Git-Url: http://git.cascardo.eti.br/?a=commitdiff_plain;h=013e4f4a285d8c7d952d8d7be9f10783a85b5d3c;p=cascardo%2Flinux.git

omfs: rename() needs to mark old_inode dirty after ctime update

we *do* mark it dirty before, but it doesn't guarantee that we
don't get preempted just before assignment to ->i_ctime, with
inode getting written out before we get CPU back...

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Bob Copeland <me@bobcopeland.com>
---

diff --git a/fs/omfs/dir.c b/fs/omfs/dir.c
index 393f3f659da7..9990fc856959 100644
--- a/fs/omfs/dir.c
+++ b/fs/omfs/dir.c
@@ -423,6 +423,7 @@ static int omfs_rename(struct inode *old_dir, struct dentry *old_dentry,
 		goto out;
 
 	old_inode->i_ctime = CURRENT_TIME_SEC;
+	mark_inode_dirty(old_inode);
 out:
 	return err;
 }