Attachment 147289 Details for Bug 227249 – patch to add some missing inode revals

[patch] patch to add some missing inode revals

nfs-missing-revalidate.patch (text/plain), 1.84 KB, created by Jeff Layton on 2007-02-03 23:22:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeff Layton

Created: 2007-02-03 23:22:16 UTC

Size: 1.84 KB

patch

obsolete

>commit 5ba7cc4801ae0fe74b6e0160f008521ae71d9f5d
>Author: Trond Myklebust <Trond.Myklebust@netapp.com>
>Date:   Sat Dec 3 15:20:17 2005 -0500
>
>    NFS: Fix post-op attribute revalidation...
>    
>      - Missing nfs_mark_for_revalidate in nfs_proc_link()
>      - Missing nfs_mark_for_revalidate in nfs_rename()
>    
>     Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
>
>diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c
>index 7370583..c0d1a21 100644
>--- a/fs/nfs/dir.c
>+++ b/fs/nfs/dir.c
>@@ -1287,6 +1287,7 @@ dentry->d_parent->d_name.name, dentry->d_name.name);
> 		nfs_begin_data_update(dentry->d_inode);
> 		error = NFS_PROTO(dir)->rename(dir, &dentry->d_name,
> 				dir, &qsilly);
>+		nfs_mark_for_revalidate(dentry->d_inode);
> 		nfs_end_data_update(dentry->d_inode);
> 	} else
> 		error = NFS_PROTO(dir)->rename(dir, &dentry->d_name,
>@@ -1334,6 +1335,7 @@ static int nfs_safe_remove(struct dentry *dentry)
> 		/* The VFS may want to delete this inode */
> 		if (error == 0)
> 			inode->i_nlink--;
>+		nfs_mark_for_revalidate(inode);
> 		nfs_end_data_update(inode);
> 	} else
> 		error = NFS_PROTO(dir)->remove(dir, &dentry->d_name);
>@@ -1556,6 +1558,7 @@ go_ahead:
> 	nfs_begin_data_update(old_inode);
> 	error = NFS_PROTO(old_dir)->rename(old_dir, &old_dentry->d_name,
> 					   new_dir, &new_dentry->d_name);
>+	nfs_mark_for_revalidate(old_inode);
> 	nfs_end_data_update(old_inode);
> 	nfs_end_data_update(new_dir);
> 	nfs_end_data_update(old_dir);
>diff --git a/fs/nfs/proc.c b/fs/nfs/proc.c
>index a48a003..e1e3ca5 100644
>--- a/fs/nfs/proc.c
>+++ b/fs/nfs/proc.c
>@@ -375,6 +375,7 @@ nfs_proc_link(struct inode *inode, struct inode *dir, struct qstr *name)
> 
> 	dprintk("NFS call  link %s\n", name->name);
> 	status = rpc_call(NFS_CLIENT(inode), NFSPROC_LINK, &arg, NULL, 0);
>+	nfs_mark_for_revalidate(inode);
> 	nfs_mark_for_revalidate(dir);
> 	dprintk("NFS reply link: %d\n", status);
> 	return status;

Actions: View | Diff

Attachments on bug 227249: 147289 | 148245