NFS: Create a have_delegation rpc_op
Delegations are a v4 feature, so push them out of the generic code. Signed-off-by: Bryan Schumaker <bjschuma@netapp.com> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
This commit is contained in:
parent
a5c58892b4
commit
011e2a7fd5
10 changed files with 25 additions and 16 deletions
|
@ -47,7 +47,7 @@ void nfs_mark_delegation_referenced(struct nfs_delegation *delegation)
|
|||
*
|
||||
* Returns one if inode has the indicated delegation, otherwise zero.
|
||||
*/
|
||||
int nfs_have_delegation(struct inode *inode, fmode_t flags)
|
||||
int nfs4_have_delegation(struct inode *inode, fmode_t flags)
|
||||
{
|
||||
struct nfs_delegation *delegation;
|
||||
int ret = 0;
|
||||
|
|
|
@ -56,14 +56,9 @@ int nfs4_lock_delegation_recall(struct nfs4_state *state, struct file_lock *fl);
|
|||
bool nfs4_copy_delegation_stateid(nfs4_stateid *dst, struct inode *inode, fmode_t flags);
|
||||
|
||||
void nfs_mark_delegation_referenced(struct nfs_delegation *delegation);
|
||||
int nfs_have_delegation(struct inode *inode, fmode_t flags);
|
||||
int nfs4_have_delegation(struct inode *inode, fmode_t flags);
|
||||
|
||||
#else
|
||||
static inline int nfs_have_delegation(struct inode *inode, fmode_t flags)
|
||||
{
|
||||
return 0;
|
||||
}
|
||||
|
||||
static inline int nfs_inode_return_delegation(struct inode *inode)
|
||||
{
|
||||
nfs_wb_all(inode);
|
||||
|
@ -73,7 +68,7 @@ static inline int nfs_inode_return_delegation(struct inode *inode)
|
|||
|
||||
static inline int nfs_have_delegated_attributes(struct inode *inode)
|
||||
{
|
||||
return nfs_have_delegation(inode, FMODE_READ) &&
|
||||
return NFS_PROTO(inode)->have_delegation(inode, FMODE_READ) &&
|
||||
!(NFS_I(inode)->cache_validity & NFS_INO_REVAL_FORCED);
|
||||
}
|
||||
|
||||
|
|
|
@ -1144,7 +1144,7 @@ static int nfs_lookup_revalidate(struct dentry *dentry, struct nameidata *nd)
|
|||
goto out_bad;
|
||||
}
|
||||
|
||||
if (nfs_have_delegation(inode, FMODE_READ))
|
||||
if (NFS_PROTO(dir)->have_delegation(inode, FMODE_READ))
|
||||
goto out_set_verifier;
|
||||
|
||||
/* Force a full look up iff the parent directory has changed */
|
||||
|
|
|
@ -178,7 +178,7 @@ nfs_file_flush(struct file *file, fl_owner_t id)
|
|||
* If we're holding a write delegation, then just start the i/o
|
||||
* but don't wait for completion (or send a commit).
|
||||
*/
|
||||
if (nfs_have_delegation(inode, FMODE_WRITE))
|
||||
if (NFS_PROTO(inode)->have_delegation(inode, FMODE_WRITE))
|
||||
return filemap_fdatawrite(file->f_mapping);
|
||||
|
||||
/* Flush writes to the server and return any errors */
|
||||
|
@ -677,7 +677,7 @@ do_getlk(struct file *filp, int cmd, struct file_lock *fl, int is_local)
|
|||
}
|
||||
fl->fl_type = saved_type;
|
||||
|
||||
if (nfs_have_delegation(inode, FMODE_READ))
|
||||
if (NFS_PROTO(inode)->have_delegation(inode, FMODE_READ))
|
||||
goto out_noconflict;
|
||||
|
||||
if (is_local)
|
||||
|
@ -772,7 +772,7 @@ do_setlk(struct file *filp, int cmd, struct file_lock *fl, int is_local)
|
|||
* This makes locking act as a cache coherency point.
|
||||
*/
|
||||
nfs_sync_mapping(filp->f_mapping);
|
||||
if (!nfs_have_delegation(inode, FMODE_READ)) {
|
||||
if (!NFS_PROTO(inode)->have_delegation(inode, FMODE_READ)) {
|
||||
if (is_time_granular(&NFS_SERVER(inode)->time_delta))
|
||||
__nfs_revalidate_inode(NFS_SERVER(inode), inode);
|
||||
else
|
||||
|
|
|
@ -1457,7 +1457,7 @@ static int nfs_update_inode(struct inode *inode, struct nfs_fattr *fattr)
|
|||
if (!(S_ISREG(inode->i_mode) || S_ISDIR(inode->i_mode)
|
||||
|| S_ISLNK(inode->i_mode)))
|
||||
invalid &= ~NFS_INO_INVALID_DATA;
|
||||
if (!nfs_have_delegation(inode, FMODE_READ) ||
|
||||
if (!NFS_PROTO(inode)->have_delegation(inode, FMODE_READ) ||
|
||||
(save_cache_validity & NFS_INO_REVAL_FORCED))
|
||||
nfsi->cache_validity |= invalid;
|
||||
|
||||
|
|
|
@ -877,6 +877,11 @@ nfs3_proc_lock(struct file *filp, int cmd, struct file_lock *fl)
|
|||
return nlmclnt_proc(NFS_SERVER(inode)->nlm_host, cmd, fl);
|
||||
}
|
||||
|
||||
static int nfs3_have_delegation(struct inode *inode, fmode_t flags)
|
||||
{
|
||||
return 0;
|
||||
}
|
||||
|
||||
const struct nfs_rpc_ops nfs_v3_clientops = {
|
||||
.version = 3, /* protocol version */
|
||||
.dentry_ops = &nfs_dentry_operations,
|
||||
|
@ -921,5 +926,6 @@ const struct nfs_rpc_ops nfs_v3_clientops = {
|
|||
.lock = nfs3_proc_lock,
|
||||
.clear_acl_cache = nfs3_forget_cached_acls,
|
||||
.close_context = nfs_close_context,
|
||||
.have_delegation = nfs3_have_delegation,
|
||||
.init_client = nfs_init_client,
|
||||
};
|
||||
|
|
|
@ -294,7 +294,7 @@ static int nfs4_handle_exception(struct nfs_server *server, int errorcode, struc
|
|||
case 0:
|
||||
return 0;
|
||||
case -NFS4ERR_OPENMODE:
|
||||
if (inode && nfs_have_delegation(inode, FMODE_READ)) {
|
||||
if (inode && nfs4_have_delegation(inode, FMODE_READ)) {
|
||||
nfs_inode_return_delegation(inode);
|
||||
exception->retry = 1;
|
||||
return 0;
|
||||
|
@ -3466,7 +3466,7 @@ bool nfs4_write_need_cache_consistency_data(const struct nfs_write_data *data)
|
|||
/* Otherwise, request attributes if and only if we don't hold
|
||||
* a delegation
|
||||
*/
|
||||
return nfs_have_delegation(hdr->inode, FMODE_READ) == 0;
|
||||
return nfs4_have_delegation(hdr->inode, FMODE_READ) == 0;
|
||||
}
|
||||
|
||||
static void nfs4_proc_write_setup(struct nfs_write_data *data, struct rpc_message *msg)
|
||||
|
@ -6804,6 +6804,7 @@ const struct nfs_rpc_ops nfs_v4_clientops = {
|
|||
.clear_acl_cache = nfs4_zap_acl_attr,
|
||||
.close_context = nfs4_close_context,
|
||||
.open_context = nfs4_atomic_open,
|
||||
.have_delegation = nfs4_have_delegation,
|
||||
.init_client = nfs4_init_client,
|
||||
};
|
||||
|
||||
|
|
|
@ -734,6 +734,11 @@ static int nfs_lock_check_bounds(const struct file_lock *fl)
|
|||
return -EINVAL;
|
||||
}
|
||||
|
||||
static int nfs_have_delegation(struct inode *inode, fmode_t flags)
|
||||
{
|
||||
return 0;
|
||||
}
|
||||
|
||||
const struct nfs_rpc_ops nfs_v2_clientops = {
|
||||
.version = 2, /* protocol version */
|
||||
.dentry_ops = &nfs_dentry_operations,
|
||||
|
@ -777,5 +782,6 @@ const struct nfs_rpc_ops nfs_v2_clientops = {
|
|||
.lock = nfs_proc_lock,
|
||||
.lock_check_bounds = nfs_lock_check_bounds,
|
||||
.close_context = nfs_close_context,
|
||||
.have_delegation = nfs_have_delegation,
|
||||
.init_client = nfs_init_client,
|
||||
};
|
||||
|
|
|
@ -410,7 +410,7 @@ static void nfs_inode_add_request(struct inode *inode, struct nfs_page *req)
|
|||
nfs_lock_request(req);
|
||||
|
||||
spin_lock(&inode->i_lock);
|
||||
if (!nfsi->npages && nfs_have_delegation(inode, FMODE_WRITE))
|
||||
if (!nfsi->npages && NFS_PROTO(inode)->have_delegation(inode, FMODE_WRITE))
|
||||
inode->i_version++;
|
||||
set_bit(PG_MAPPED, &req->wb_flags);
|
||||
SetPagePrivate(req->wb_page);
|
||||
|
|
|
@ -1422,6 +1422,7 @@ struct nfs_rpc_ops {
|
|||
struct nfs_open_context *ctx,
|
||||
int open_flags,
|
||||
struct iattr *iattr);
|
||||
int (*have_delegation)(struct inode *, fmode_t);
|
||||
struct nfs_client *
|
||||
(*init_client) (struct nfs_client *, const struct rpc_timeout *,
|
||||
const char *, rpc_authflavor_t);
|
||||
|
|
Loading…
Reference in a new issue