diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2007-06-05 19:27:00 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2007-07-10 23:40:25 -0400 |
commit | e2f032e9ef66e33089d09452892696ea97d1dca1 (patch) | |
tree | f342e1944bc9e230730c519197832709fc18f9a6 | |
parent | aa53ed541a1fec78a78d02afc8b042d040cc080d (diff) |
NFS: nfs3_proc_create() should use nfs_post_op_update_inode()
Also get rid of a redundant call to nfs_setattr_update_inode(). The call to
nfs3_proc_setattr() already takes care of that.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
-rw-r--r-- | fs/nfs/nfs3proc.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/nfs/nfs3proc.c b/fs/nfs/nfs3proc.c index 45268d6def2e..814d886b6aa4 100644 --- a/fs/nfs/nfs3proc.c +++ b/fs/nfs/nfs3proc.c | |||
@@ -335,9 +335,7 @@ again: | |||
335 | * not sure this buys us anything (and I'd have | 335 | * not sure this buys us anything (and I'd have |
336 | * to revamp the NFSv3 XDR code) */ | 336 | * to revamp the NFSv3 XDR code) */ |
337 | status = nfs3_proc_setattr(dentry, &fattr, sattr); | 337 | status = nfs3_proc_setattr(dentry, &fattr, sattr); |
338 | if (status == 0) | 338 | nfs_post_op_update_inode(dentry->d_inode, &fattr); |
339 | nfs_setattr_update_inode(dentry->d_inode, sattr); | ||
340 | nfs_refresh_inode(dentry->d_inode, &fattr); | ||
341 | dprintk("NFS reply setattr (post-create): %d\n", status); | 339 | dprintk("NFS reply setattr (post-create): %d\n", status); |
342 | } | 340 | } |
343 | if (status != 0) | 341 | if (status != 0) |