diff --git a/fs/nfs/delegation.c b/fs/nfs/delegation.c index 98811a07745095cff8306c08b445fdb8670e8c8a..1a4a28cd3ba24e668c5bceafc54372cb420830eb 100644 --- a/fs/nfs/delegation.c +++ b/fs/nfs/delegation.c @@ -106,15 +106,17 @@ int nfs4_check_delegation(struct inode *inode, fmode_t flags) static int nfs_delegation_claim_locks(struct nfs4_state *state, const nfs4_stateid *stateid) { struct inode *inode = state->inode; + struct nfs_inode *nfsi = NFS_I(inode); struct file_lock *fl; struct file_lock_context *flctx = inode->i_flctx; struct list_head *list; int status = 0; if (flctx == NULL) - goto out; + return status; list = &flctx->flc_posix; + down_write(&nfsi->rwsem); spin_lock(&flctx->flc_lock); restart: list_for_each_entry(fl, list, fl_list) { @@ -132,6 +134,7 @@ static int nfs_delegation_claim_locks(struct nfs4_state *state, const nfs4_state } spin_unlock(&flctx->flc_lock); out: + up_write(&nfsi->rwsem); return status; } diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index 6d47a1922a1ba0fbbaf051fec02cddd3d472d2f6..29cb861ae4495d82008b131907bbe22f3782114f 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c @@ -6603,12 +6603,16 @@ static int nfs4_proc_unlck(struct nfs4_state *state, int cmd, struct file_lock * status = -ENOMEM; if (IS_ERR(seqid)) goto out; + down_read(&nfsi->rwsem); task = nfs4_do_unlck(request, nfs_file_open_context(request->fl_file), lsp, seqid); status = PTR_ERR(task); - if (IS_ERR(task)) + if (IS_ERR(task)) { + up_read(&nfsi->rwsem); goto out; + } status = rpc_wait_for_completion_task(task); rpc_put_task(task); + up_read(&nfsi->rwsem); out: request->fl_flags = fl_flags; trace_nfs4_unlock(request, state, F_SETLK, status); @@ -6945,7 +6949,9 @@ static int _nfs4_proc_setlk(struct nfs4_state *state, int cmd, struct file_lock } up_read(&nfsi->rwsem); mutex_unlock(&sp->so_delegreturn_mutex); + down_read(&nfsi->rwsem); status = _nfs4_do_setlk(state, cmd, request, NFS_LOCK_NEW); + up_read(&nfsi->rwsem); out: request->fl_flags = fl_flags; return status;