REPOST patch 06/38: CLIENT: change inode to dentry in ->setattr()

Kendrick M. Smith (kmsmith@umich.edu)
Wed, 14 Aug 2002 16:38:53 -0400 (EDT)


This patch changes the interface of the ->setattr() nfs_rpc_op
so that its first argument is a dentry instead of an inode.

[Explanation: The dentry is required because in NFSv4, we may
need to OPEN the file before doing the SETATTR. (This is
required if the file size is changed as part of the setattr.)
Opening the file requires making use of the containing
directory's inode.]

--- old/fs/nfs/inode.c Thu Aug 1 16:16:33 2002
+++ new/fs/nfs/inode.c Sun Aug 11 20:27:33 2002
@@ -782,7 +782,7 @@ printk("nfs_setattr: revalidate failed,
if (error)
goto out;

- error = NFS_PROTO(inode)->setattr(inode, &fattr, attr);
+ error = NFS_PROTO(inode)->setattr(dentry, &fattr, attr);
if (error)
goto out;
/*
--- old/fs/nfs/nfs3proc.c Sun Aug 11 20:26:56 2002
+++ new/fs/nfs/nfs3proc.c Sun Aug 11 20:27:33 2002
@@ -86,9 +86,10 @@ nfs3_proc_getattr(struct inode *inode, s
}

static int
-nfs3_proc_setattr(struct inode *inode, struct nfs_fattr *fattr,
+nfs3_proc_setattr(struct dentry *dentry, struct nfs_fattr *fattr,
struct iattr *sattr)
{
+ struct inode *inode = dentry->d_inode;
struct nfs3_sattrargs arg = {
fh: NFS_FH(inode),
sattr: sattr,
--- old/fs/nfs/proc.c Sun Aug 11 20:26:56 2002
+++ new/fs/nfs/proc.c Sun Aug 11 20:27:33 2002
@@ -78,9 +78,10 @@ nfs_proc_getattr(struct inode *inode, st
}

static int
-nfs_proc_setattr(struct inode *inode, struct nfs_fattr *fattr,
+nfs_proc_setattr(struct dentry *dentry, struct nfs_fattr *fattr,
struct iattr *sattr)
{
+ struct inode *inode = dentry->d_inode;
struct nfs_sattrargs arg = {
fh: NFS_FH(inode),
sattr: sattr
--- old/include/linux/nfs_xdr.h Sun Aug 11 20:26:56 2002
+++ new/include/linux/nfs_xdr.h Sun Aug 11 20:27:33 2002
@@ -297,7 +297,7 @@ struct nfs_rpc_ops {
int (*getroot) (struct nfs_server *, struct nfs_fh *,
struct nfs_fattr *);
int (*getattr) (struct inode *, struct nfs_fattr *);
- int (*setattr) (struct inode *, struct nfs_fattr *,
+ int (*setattr) (struct dentry *, struct nfs_fattr *,
struct iattr *);
int (*lookup) (struct inode *, struct qstr *,
struct nfs_fh *, struct nfs_fattr *);

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/