Merge branch 'nfs-for-2.6.37' of git://git.linux-nfs.org/projects/trondmy/nfs-2.6
* 'nfs-for-2.6.37' of git://git.linux-nfs.org/projects/trondmy/nfs-2.6: (67 commits) SUNRPC: Cleanup duplicate assignment in rpcauth_refreshcred nfs: fix unchecked value Ask for time_delta during fsinfo probe Revalidate caches on lock SUNRPC: After calling xprt_release(), we must restart from call_reserve NFSv4: Fix up the 'dircount' hint in encode_readdir NFSv4: Clean up nfs4_decode_dirent NFSv4: nfs4_decode_dirent must clear entry->fattr->valid NFSv4: Fix a regression in decode_getfattr NFSv4: Fix up decode_attr_filehandle() to handle the case of empty fh pointer NFS: Ensure we check all allocation return values in new readdir code NFS: Readdir plus in v4 NFS: introduce generic decode_getattr function NFS: check xdr_decode for errors NFS: nfs_readdir_filler catch all errors NFS: readdir with vmapped pages NFS: remove page size checking code NFS: decode_dirent should use an xdr_stream SUNRPC: Add a helper function xdr_inline_peek NFS: remove readdir plus limit ...
This commit is contained in:
@@ -551,7 +551,7 @@ static int nfs_vm_page_mkwrite(struct vm_area_struct *vma, struct vm_fault *vmf)
|
||||
struct file *filp = vma->vm_file;
|
||||
struct dentry *dentry = filp->f_path.dentry;
|
||||
unsigned pagelen;
|
||||
int ret = -EINVAL;
|
||||
int ret = VM_FAULT_NOPAGE;
|
||||
struct address_space *mapping;
|
||||
|
||||
dfprintk(PAGECACHE, "NFS: vm_page_mkwrite(%s/%s(%ld), offset %lld)\n",
|
||||
@@ -567,21 +567,20 @@ static int nfs_vm_page_mkwrite(struct vm_area_struct *vma, struct vm_fault *vmf)
|
||||
if (mapping != dentry->d_inode->i_mapping)
|
||||
goto out_unlock;
|
||||
|
||||
ret = 0;
|
||||
pagelen = nfs_page_length(page);
|
||||
if (pagelen == 0)
|
||||
goto out_unlock;
|
||||
|
||||
ret = nfs_flush_incompatible(filp, page);
|
||||
if (ret != 0)
|
||||
goto out_unlock;
|
||||
ret = VM_FAULT_LOCKED;
|
||||
if (nfs_flush_incompatible(filp, page) == 0 &&
|
||||
nfs_updatepage(filp, page, 0, pagelen) == 0)
|
||||
goto out;
|
||||
|
||||
ret = nfs_updatepage(filp, page, 0, pagelen);
|
||||
ret = VM_FAULT_SIGBUS;
|
||||
out_unlock:
|
||||
if (!ret)
|
||||
return VM_FAULT_LOCKED;
|
||||
unlock_page(page);
|
||||
return VM_FAULT_SIGBUS;
|
||||
out:
|
||||
return ret;
|
||||
}
|
||||
|
||||
static const struct vm_operations_struct nfs_file_vm_ops = {
|
||||
@@ -684,7 +683,8 @@ static ssize_t nfs_file_splice_write(struct pipe_inode_info *pipe,
|
||||
return ret;
|
||||
}
|
||||
|
||||
static int do_getlk(struct file *filp, int cmd, struct file_lock *fl)
|
||||
static int
|
||||
do_getlk(struct file *filp, int cmd, struct file_lock *fl, int is_local)
|
||||
{
|
||||
struct inode *inode = filp->f_mapping->host;
|
||||
int status = 0;
|
||||
@@ -699,7 +699,7 @@ static int do_getlk(struct file *filp, int cmd, struct file_lock *fl)
|
||||
if (nfs_have_delegation(inode, FMODE_READ))
|
||||
goto out_noconflict;
|
||||
|
||||
if (NFS_SERVER(inode)->flags & NFS_MOUNT_NONLM)
|
||||
if (is_local)
|
||||
goto out_noconflict;
|
||||
|
||||
status = NFS_PROTO(inode)->lock(filp, cmd, fl);
|
||||
@@ -726,7 +726,8 @@ static int do_vfs_lock(struct file *file, struct file_lock *fl)
|
||||
return res;
|
||||
}
|
||||
|
||||
static int do_unlk(struct file *filp, int cmd, struct file_lock *fl)
|
||||
static int
|
||||
do_unlk(struct file *filp, int cmd, struct file_lock *fl, int is_local)
|
||||
{
|
||||
struct inode *inode = filp->f_mapping->host;
|
||||
int status;
|
||||
@@ -741,15 +742,24 @@ static int do_unlk(struct file *filp, int cmd, struct file_lock *fl)
|
||||
* If we're signalled while cleaning up locks on process exit, we
|
||||
* still need to complete the unlock.
|
||||
*/
|
||||
/* Use local locking if mounted with "-onolock" */
|
||||
if (!(NFS_SERVER(inode)->flags & NFS_MOUNT_NONLM))
|
||||
/*
|
||||
* Use local locking if mounted with "-onolock" or with appropriate
|
||||
* "-olocal_lock="
|
||||
*/
|
||||
if (!is_local)
|
||||
status = NFS_PROTO(inode)->lock(filp, cmd, fl);
|
||||
else
|
||||
status = do_vfs_lock(filp, fl);
|
||||
return status;
|
||||
}
|
||||
|
||||
static int do_setlk(struct file *filp, int cmd, struct file_lock *fl)
|
||||
static int
|
||||
is_time_granular(struct timespec *ts) {
|
||||
return ((ts->tv_sec == 0) && (ts->tv_nsec <= 1000));
|
||||
}
|
||||
|
||||
static int
|
||||
do_setlk(struct file *filp, int cmd, struct file_lock *fl, int is_local)
|
||||
{
|
||||
struct inode *inode = filp->f_mapping->host;
|
||||
int status;
|
||||
@@ -762,20 +772,31 @@ static int do_setlk(struct file *filp, int cmd, struct file_lock *fl)
|
||||
if (status != 0)
|
||||
goto out;
|
||||
|
||||
/* Use local locking if mounted with "-onolock" */
|
||||
if (!(NFS_SERVER(inode)->flags & NFS_MOUNT_NONLM))
|
||||
/*
|
||||
* Use local locking if mounted with "-onolock" or with appropriate
|
||||
* "-olocal_lock="
|
||||
*/
|
||||
if (!is_local)
|
||||
status = NFS_PROTO(inode)->lock(filp, cmd, fl);
|
||||
else
|
||||
status = do_vfs_lock(filp, fl);
|
||||
if (status < 0)
|
||||
goto out;
|
||||
|
||||
/*
|
||||
* Make sure we clear the cache whenever we try to get the lock.
|
||||
* Revalidate the cache if the server has time stamps granular
|
||||
* enough to detect subsecond changes. Otherwise, clear the
|
||||
* cache to prevent missing any changes.
|
||||
*
|
||||
* This makes locking act as a cache coherency point.
|
||||
*/
|
||||
nfs_sync_mapping(filp->f_mapping);
|
||||
if (!nfs_have_delegation(inode, FMODE_READ))
|
||||
nfs_zap_caches(inode);
|
||||
if (!nfs_have_delegation(inode, FMODE_READ)) {
|
||||
if (is_time_granular(&NFS_SERVER(inode)->time_delta))
|
||||
__nfs_revalidate_inode(NFS_SERVER(inode), inode);
|
||||
else
|
||||
nfs_zap_caches(inode);
|
||||
}
|
||||
out:
|
||||
return status;
|
||||
}
|
||||
@@ -787,6 +808,7 @@ static int nfs_lock(struct file *filp, int cmd, struct file_lock *fl)
|
||||
{
|
||||
struct inode *inode = filp->f_mapping->host;
|
||||
int ret = -ENOLCK;
|
||||
int is_local = 0;
|
||||
|
||||
dprintk("NFS: lock(%s/%s, t=%x, fl=%x, r=%lld:%lld)\n",
|
||||
filp->f_path.dentry->d_parent->d_name.name,
|
||||
@@ -800,6 +822,9 @@ static int nfs_lock(struct file *filp, int cmd, struct file_lock *fl)
|
||||
if (__mandatory_lock(inode) && fl->fl_type != F_UNLCK)
|
||||
goto out_err;
|
||||
|
||||
if (NFS_SERVER(inode)->flags & NFS_MOUNT_LOCAL_FCNTL)
|
||||
is_local = 1;
|
||||
|
||||
if (NFS_PROTO(inode)->lock_check_bounds != NULL) {
|
||||
ret = NFS_PROTO(inode)->lock_check_bounds(fl);
|
||||
if (ret < 0)
|
||||
@@ -807,11 +832,11 @@ static int nfs_lock(struct file *filp, int cmd, struct file_lock *fl)
|
||||
}
|
||||
|
||||
if (IS_GETLK(cmd))
|
||||
ret = do_getlk(filp, cmd, fl);
|
||||
ret = do_getlk(filp, cmd, fl, is_local);
|
||||
else if (fl->fl_type == F_UNLCK)
|
||||
ret = do_unlk(filp, cmd, fl);
|
||||
ret = do_unlk(filp, cmd, fl, is_local);
|
||||
else
|
||||
ret = do_setlk(filp, cmd, fl);
|
||||
ret = do_setlk(filp, cmd, fl, is_local);
|
||||
out_err:
|
||||
return ret;
|
||||
}
|
||||
@@ -821,6 +846,9 @@ out_err:
|
||||
*/
|
||||
static int nfs_flock(struct file *filp, int cmd, struct file_lock *fl)
|
||||
{
|
||||
struct inode *inode = filp->f_mapping->host;
|
||||
int is_local = 0;
|
||||
|
||||
dprintk("NFS: flock(%s/%s, t=%x, fl=%x)\n",
|
||||
filp->f_path.dentry->d_parent->d_name.name,
|
||||
filp->f_path.dentry->d_name.name,
|
||||
@@ -829,14 +857,17 @@ static int nfs_flock(struct file *filp, int cmd, struct file_lock *fl)
|
||||
if (!(fl->fl_flags & FL_FLOCK))
|
||||
return -ENOLCK;
|
||||
|
||||
if (NFS_SERVER(inode)->flags & NFS_MOUNT_LOCAL_FLOCK)
|
||||
is_local = 1;
|
||||
|
||||
/* We're simulating flock() locks using posix locks on the server */
|
||||
fl->fl_owner = (fl_owner_t)filp;
|
||||
fl->fl_start = 0;
|
||||
fl->fl_end = OFFSET_MAX;
|
||||
|
||||
if (fl->fl_type == F_UNLCK)
|
||||
return do_unlk(filp, cmd, fl);
|
||||
return do_setlk(filp, cmd, fl);
|
||||
return do_unlk(filp, cmd, fl, is_local);
|
||||
return do_setlk(filp, cmd, fl, is_local);
|
||||
}
|
||||
|
||||
/*
|
||||
|
Reference in New Issue
Block a user