NFSv4: Send the delegation stateid for SETATTR calls
In the case where we hold a delegation stateid, use that in for inside SETATTR calls. Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
This commit is contained in:
parent
f25bc34967
commit
3e4f6290ca
3 changed files with 33 additions and 14 deletions
|
@ -421,3 +421,22 @@ void nfs_delegation_reap_unclaimed(struct nfs4_client *clp)
|
||||||
nfs_free_delegation(delegation);
|
nfs_free_delegation(delegation);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
int nfs4_copy_delegation_stateid(nfs4_stateid *dst, struct inode *inode)
|
||||||
|
{
|
||||||
|
struct nfs4_client *clp = NFS_SERVER(inode)->nfs4_state;
|
||||||
|
struct nfs_inode *nfsi = NFS_I(inode);
|
||||||
|
struct nfs_delegation *delegation;
|
||||||
|
int res = 0;
|
||||||
|
|
||||||
|
if (nfsi->delegation_state == 0)
|
||||||
|
return 0;
|
||||||
|
spin_lock(&clp->cl_lock);
|
||||||
|
delegation = nfsi->delegation;
|
||||||
|
if (delegation != NULL) {
|
||||||
|
memcpy(dst->data, delegation->stateid.data, sizeof(dst->data));
|
||||||
|
res = 1;
|
||||||
|
}
|
||||||
|
spin_unlock(&clp->cl_lock);
|
||||||
|
return res;
|
||||||
|
}
|
||||||
|
|
|
@ -41,6 +41,7 @@ void nfs_delegation_reap_unclaimed(struct nfs4_client *clp);
|
||||||
int nfs4_proc_delegreturn(struct inode *inode, struct rpc_cred *cred, const nfs4_stateid *stateid);
|
int nfs4_proc_delegreturn(struct inode *inode, struct rpc_cred *cred, const nfs4_stateid *stateid);
|
||||||
int nfs4_open_delegation_recall(struct dentry *dentry, struct nfs4_state *state);
|
int nfs4_open_delegation_recall(struct dentry *dentry, struct nfs4_state *state);
|
||||||
int nfs4_lock_delegation_recall(struct nfs4_state *state, struct file_lock *fl);
|
int nfs4_lock_delegation_recall(struct nfs4_state *state, struct file_lock *fl);
|
||||||
|
int nfs4_copy_delegation_stateid(nfs4_stateid *dst, struct inode *inode);
|
||||||
|
|
||||||
static inline int nfs_have_delegation(struct inode *inode, int flags)
|
static inline int nfs_have_delegation(struct inode *inode, int flags)
|
||||||
{
|
{
|
||||||
|
|
|
@ -1018,12 +1018,12 @@ static struct nfs4_state *nfs4_do_open(struct inode *dir, struct dentry *dentry,
|
||||||
return res;
|
return res;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int _nfs4_do_setattr(struct nfs_server *server, struct nfs_fattr *fattr,
|
static int _nfs4_do_setattr(struct inode *inode, struct nfs_fattr *fattr,
|
||||||
struct nfs_fh *fhandle, struct iattr *sattr,
|
struct iattr *sattr, struct nfs4_state *state)
|
||||||
struct nfs4_state *state)
|
|
||||||
{
|
{
|
||||||
|
struct nfs_server *server = NFS_SERVER(inode);
|
||||||
struct nfs_setattrargs arg = {
|
struct nfs_setattrargs arg = {
|
||||||
.fh = fhandle,
|
.fh = NFS_FH(inode),
|
||||||
.iap = sattr,
|
.iap = sattr,
|
||||||
.server = server,
|
.server = server,
|
||||||
.bitmask = server->attr_bitmask,
|
.bitmask = server->attr_bitmask,
|
||||||
|
@ -1042,7 +1042,9 @@ static int _nfs4_do_setattr(struct nfs_server *server, struct nfs_fattr *fattr,
|
||||||
|
|
||||||
nfs_fattr_init(fattr);
|
nfs_fattr_init(fattr);
|
||||||
|
|
||||||
if (state != NULL) {
|
if (nfs4_copy_delegation_stateid(&arg.stateid, inode)) {
|
||||||
|
/* Use that stateid */
|
||||||
|
} else if (state != NULL) {
|
||||||
msg.rpc_cred = state->owner->so_cred;
|
msg.rpc_cred = state->owner->so_cred;
|
||||||
nfs4_copy_stateid(&arg.stateid, state, current->files);
|
nfs4_copy_stateid(&arg.stateid, state, current->files);
|
||||||
} else
|
} else
|
||||||
|
@ -1054,16 +1056,15 @@ static int _nfs4_do_setattr(struct nfs_server *server, struct nfs_fattr *fattr,
|
||||||
return status;
|
return status;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int nfs4_do_setattr(struct nfs_server *server, struct nfs_fattr *fattr,
|
static int nfs4_do_setattr(struct inode *inode, struct nfs_fattr *fattr,
|
||||||
struct nfs_fh *fhandle, struct iattr *sattr,
|
struct iattr *sattr, struct nfs4_state *state)
|
||||||
struct nfs4_state *state)
|
|
||||||
{
|
{
|
||||||
|
struct nfs_server *server = NFS_SERVER(inode);
|
||||||
struct nfs4_exception exception = { };
|
struct nfs4_exception exception = { };
|
||||||
int err;
|
int err;
|
||||||
do {
|
do {
|
||||||
err = nfs4_handle_exception(server,
|
err = nfs4_handle_exception(server,
|
||||||
_nfs4_do_setattr(server, fattr, fhandle, sattr,
|
_nfs4_do_setattr(inode, fattr, sattr, state),
|
||||||
state),
|
|
||||||
&exception);
|
&exception);
|
||||||
} while (exception.retry);
|
} while (exception.retry);
|
||||||
return err;
|
return err;
|
||||||
|
@ -1504,8 +1505,7 @@ nfs4_proc_setattr(struct dentry *dentry, struct nfs_fattr *fattr,
|
||||||
if (ctx != NULL)
|
if (ctx != NULL)
|
||||||
state = ctx->state;
|
state = ctx->state;
|
||||||
|
|
||||||
status = nfs4_do_setattr(NFS_SERVER(inode), fattr,
|
status = nfs4_do_setattr(inode, fattr, sattr, state);
|
||||||
NFS_FH(inode), sattr, state);
|
|
||||||
if (status == 0)
|
if (status == 0)
|
||||||
nfs_setattr_update_inode(inode, sattr);
|
nfs_setattr_update_inode(inode, sattr);
|
||||||
if (ctx != NULL)
|
if (ctx != NULL)
|
||||||
|
@ -1824,8 +1824,7 @@ nfs4_proc_create(struct inode *dir, struct dentry *dentry, struct iattr *sattr,
|
||||||
d_instantiate(dentry, igrab(state->inode));
|
d_instantiate(dentry, igrab(state->inode));
|
||||||
if (flags & O_EXCL) {
|
if (flags & O_EXCL) {
|
||||||
struct nfs_fattr fattr;
|
struct nfs_fattr fattr;
|
||||||
status = nfs4_do_setattr(NFS_SERVER(dir), &fattr,
|
status = nfs4_do_setattr(state->inode, &fattr, sattr, state);
|
||||||
NFS_FH(state->inode), sattr, state);
|
|
||||||
if (status == 0)
|
if (status == 0)
|
||||||
nfs_setattr_update_inode(state->inode, sattr);
|
nfs_setattr_update_inode(state->inode, sattr);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue