2019-05-19 14:08:55 +02:00
|
|
|
// SPDX-License-Identifier: GPL-2.0-only
|
2005-04-17 00:20:36 +02:00
|
|
|
/*
|
|
|
|
* linux/fs/nfs/file.c
|
|
|
|
*
|
|
|
|
* Copyright (C) 1992 Rick Sladkey
|
|
|
|
*
|
|
|
|
* Changes Copyright (C) 1994 by Florian La Roche
|
|
|
|
* - Do not copy data too often around in the kernel.
|
|
|
|
* - In nfs_file_read the return value of kmalloc wasn't checked.
|
|
|
|
* - Put in a better version of read look-ahead buffering. Original idea
|
|
|
|
* and implementation by Wai S Kok elekokws@ee.nus.sg.
|
|
|
|
*
|
|
|
|
* Expire cache on write to a file by Wai S Kok (Oct 1994).
|
|
|
|
*
|
|
|
|
* Total rewrite of read side for new NFS buffer cache.. Linus.
|
|
|
|
*
|
|
|
|
* nfs regular file handling functions
|
|
|
|
*/
|
|
|
|
|
2012-07-30 22:05:23 +02:00
|
|
|
#include <linux/module.h>
|
2005-04-17 00:20:36 +02:00
|
|
|
#include <linux/time.h>
|
|
|
|
#include <linux/kernel.h>
|
|
|
|
#include <linux/errno.h>
|
|
|
|
#include <linux/fcntl.h>
|
|
|
|
#include <linux/stat.h>
|
|
|
|
#include <linux/nfs_fs.h>
|
|
|
|
#include <linux/nfs_mount.h>
|
|
|
|
#include <linux/mm.h>
|
|
|
|
#include <linux/pagemap.h>
|
include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h
percpu.h is included by sched.h and module.h and thus ends up being
included when building most .c files. percpu.h includes slab.h which
in turn includes gfp.h making everything defined by the two files
universally available and complicating inclusion dependencies.
percpu.h -> slab.h dependency is about to be removed. Prepare for
this change by updating users of gfp and slab facilities include those
headers directly instead of assuming availability. As this conversion
needs to touch large number of source files, the following script is
used as the basis of conversion.
http://userweb.kernel.org/~tj/misc/slabh-sweep.py
The script does the followings.
* Scan files for gfp and slab usages and update includes such that
only the necessary includes are there. ie. if only gfp is used,
gfp.h, if slab is used, slab.h.
* When the script inserts a new include, it looks at the include
blocks and try to put the new include such that its order conforms
to its surrounding. It's put in the include block which contains
core kernel includes, in the same order that the rest are ordered -
alphabetical, Christmas tree, rev-Xmas-tree or at the end if there
doesn't seem to be any matching order.
* If the script can't find a place to put a new include (mostly
because the file doesn't have fitting include block), it prints out
an error message indicating which .h file needs to be added to the
file.
The conversion was done in the following steps.
1. The initial automatic conversion of all .c files updated slightly
over 4000 files, deleting around 700 includes and adding ~480 gfp.h
and ~3000 slab.h inclusions. The script emitted errors for ~400
files.
2. Each error was manually checked. Some didn't need the inclusion,
some needed manual addition while adding it to implementation .h or
embedding .c file was more appropriate for others. This step added
inclusions to around 150 files.
3. The script was run again and the output was compared to the edits
from #2 to make sure no file was left behind.
4. Several build tests were done and a couple of problems were fixed.
e.g. lib/decompress_*.c used malloc/free() wrappers around slab
APIs requiring slab.h to be added manually.
5. The script was run on all .h files but without automatically
editing them as sprinkling gfp.h and slab.h inclusions around .h
files could easily lead to inclusion dependency hell. Most gfp.h
inclusion directives were ignored as stuff from gfp.h was usually
wildly available and often used in preprocessor macros. Each
slab.h inclusion directive was examined and added manually as
necessary.
6. percpu.h was updated not to include slab.h.
7. Build test were done on the following configurations and failures
were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my
distributed build env didn't work with gcov compiles) and a few
more options had to be turned off depending on archs to make things
build (like ipr on powerpc/64 which failed due to missing writeq).
* x86 and x86_64 UP and SMP allmodconfig and a custom test config.
* powerpc and powerpc64 SMP allmodconfig
* sparc and sparc64 SMP allmodconfig
* ia64 SMP allmodconfig
* s390 SMP allmodconfig
* alpha SMP allmodconfig
* um on x86_64 SMP allmodconfig
8. percpu.h modifications were reverted so that it could be applied as
a separate patch and serve as bisection point.
Given the fact that I had only a couple of failures from tests on step
6, I'm fairly confident about the coverage of this conversion patch.
If there is a breakage, it's likely to be something in one of the arch
headers which should be easily discoverable easily on most builds of
the specific arch.
Signed-off-by: Tejun Heo <tj@kernel.org>
Guess-its-ok-by: Christoph Lameter <cl@linux-foundation.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Lee Schermerhorn <Lee.Schermerhorn@hp.com>
2010-03-24 09:04:11 +01:00
|
|
|
#include <linux/gfp.h>
|
2010-07-30 21:31:54 +02:00
|
|
|
#include <linux/swap.h>
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2016-12-24 20:46:01 +01:00
|
|
|
#include <linux/uaccess.h>
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
#include "delegation.h"
|
2007-07-22 23:09:05 +02:00
|
|
|
#include "internal.h"
|
2006-03-20 19:44:14 +01:00
|
|
|
#include "iostat.h"
|
2009-04-03 17:42:44 +02:00
|
|
|
#include "fscache.h"
|
2014-09-10 17:23:30 +02:00
|
|
|
#include "pnfs.h"
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2013-08-20 00:59:33 +02:00
|
|
|
#include "nfstrace.h"
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
#define NFSDBG_FACILITY NFSDBG_FILE
|
|
|
|
|
2009-09-27 20:29:37 +02:00
|
|
|
static const struct vm_operations_struct nfs_file_vm_ops;
|
2007-07-22 23:09:05 +02:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
/* Hack for future NFS swap support */
|
|
|
|
#ifndef IS_SWAPFILE
|
|
|
|
# define IS_SWAPFILE(inode) (0)
|
|
|
|
#endif
|
|
|
|
|
2012-07-16 22:39:15 +02:00
|
|
|
int nfs_check_flags(int flags)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
|
|
|
if ((flags & (O_APPEND | O_DIRECT)) == (O_APPEND | O_DIRECT))
|
|
|
|
return -EINVAL;
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
2012-07-30 22:05:25 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_check_flags);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Open file
|
|
|
|
*/
|
|
|
|
static int
|
|
|
|
nfs_file_open(struct inode *inode, struct file *filp)
|
|
|
|
{
|
|
|
|
int res;
|
|
|
|
|
2013-09-16 16:53:17 +02:00
|
|
|
dprintk("NFS: open file(%pD2)\n", filp);
|
2008-06-11 23:55:42 +02:00
|
|
|
|
2010-02-01 20:17:14 +01:00
|
|
|
nfs_inc_stats(inode, NFSIOS_VFSOPEN);
|
2005-04-17 00:20:36 +02:00
|
|
|
res = nfs_check_flags(filp->f_flags);
|
|
|
|
if (res)
|
|
|
|
return res;
|
|
|
|
|
2008-06-11 22:32:46 +02:00
|
|
|
res = nfs_open(inode, filp);
|
2005-04-17 00:20:36 +02:00
|
|
|
return res;
|
|
|
|
}
|
|
|
|
|
2012-07-16 22:39:15 +02:00
|
|
|
int
|
2005-04-17 00:20:36 +02:00
|
|
|
nfs_file_release(struct inode *inode, struct file *filp)
|
|
|
|
{
|
2013-09-16 16:53:17 +02:00
|
|
|
dprintk("NFS: release(%pD2)\n", filp);
|
2008-06-11 23:55:58 +02:00
|
|
|
|
2006-03-20 19:44:14 +01:00
|
|
|
nfs_inc_stats(inode, NFSIOS_VFSRELEASE);
|
2015-07-13 20:01:33 +02:00
|
|
|
nfs_file_clear_open_context(filp);
|
|
|
|
return 0;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2012-07-30 22:05:25 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_file_release);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2005-06-13 17:14:01 +02:00
|
|
|
/**
|
|
|
|
* nfs_revalidate_size - Revalidate the file size
|
2019-02-18 19:32:38 +01:00
|
|
|
* @inode: pointer to inode struct
|
|
|
|
* @filp: pointer to struct file
|
2005-06-13 17:14:01 +02:00
|
|
|
*
|
|
|
|
* Revalidates the file length. This is basically a wrapper around
|
|
|
|
* nfs_revalidate_inode() that takes into account the fact that we may
|
|
|
|
* have cached writes (in which case we don't care about the server's
|
|
|
|
* idea of what the file length is), or O_DIRECT (in which case we
|
|
|
|
* shouldn't trust the cache).
|
|
|
|
*/
|
|
|
|
static int nfs_revalidate_file_size(struct inode *inode, struct file *filp)
|
|
|
|
{
|
|
|
|
struct nfs_server *server = NFS_SERVER(inode);
|
2010-04-16 22:42:46 +02:00
|
|
|
|
2005-06-13 17:14:01 +02:00
|
|
|
if (filp->f_flags & O_DIRECT)
|
|
|
|
goto force_reval;
|
2016-12-09 00:18:38 +01:00
|
|
|
if (nfs_check_cache_invalid(inode, NFS_INO_REVAL_PAGECACHE))
|
2010-04-16 22:42:46 +02:00
|
|
|
goto force_reval;
|
|
|
|
return 0;
|
2005-06-13 17:14:01 +02:00
|
|
|
force_reval:
|
|
|
|
return __nfs_revalidate_inode(server, inode);
|
|
|
|
}
|
|
|
|
|
2012-12-18 00:59:39 +01:00
|
|
|
loff_t nfs_file_llseek(struct file *filp, loff_t offset, int whence)
|
2005-06-13 17:14:01 +02:00
|
|
|
{
|
2013-09-16 16:53:17 +02:00
|
|
|
dprintk("NFS: llseek file(%pD2, %lld, %d)\n",
|
|
|
|
filp, offset, whence);
|
2008-06-11 23:55:34 +02:00
|
|
|
|
2011-07-18 19:21:38 +02:00
|
|
|
/*
|
2012-12-18 00:59:39 +01:00
|
|
|
* whence == SEEK_END || SEEK_DATA || SEEK_HOLE => we must revalidate
|
2011-07-18 19:21:38 +02:00
|
|
|
* the cached file length
|
|
|
|
*/
|
2012-12-18 00:59:39 +01:00
|
|
|
if (whence != SEEK_SET && whence != SEEK_CUR) {
|
2005-06-13 17:14:01 +02:00
|
|
|
struct inode *inode = filp->f_mapping->host;
|
2008-09-23 23:28:35 +02:00
|
|
|
|
2005-06-13 17:14:01 +02:00
|
|
|
int retval = nfs_revalidate_file_size(inode, filp);
|
|
|
|
if (retval < 0)
|
|
|
|
return (loff_t)retval;
|
2011-09-16 01:06:52 +02:00
|
|
|
}
|
2008-09-23 23:28:35 +02:00
|
|
|
|
2012-12-18 00:59:39 +01:00
|
|
|
return generic_file_llseek(filp, offset, whence);
|
2005-06-13 17:14:01 +02:00
|
|
|
}
|
2012-07-30 22:05:25 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_file_llseek);
|
2005-06-13 17:14:01 +02:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
/*
|
|
|
|
* Flush all dirty pages, and check for write errors.
|
|
|
|
*/
|
2015-09-06 01:06:58 +02:00
|
|
|
static int
|
2006-06-23 11:05:12 +02:00
|
|
|
nfs_file_flush(struct file *file, fl_owner_t id)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2013-09-16 16:53:17 +02:00
|
|
|
struct inode *inode = file_inode(file);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2013-09-16 16:53:17 +02:00
|
|
|
dprintk("NFS: flush(%pD2)\n", file);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2010-02-01 20:17:14 +01:00
|
|
|
nfs_inc_stats(inode, NFSIOS_VFSFLUSH);
|
2005-04-17 00:20:36 +02:00
|
|
|
if ((file->f_mode & FMODE_WRITE) == 0)
|
|
|
|
return 0;
|
2007-07-25 20:09:54 +02:00
|
|
|
|
2009-03-19 20:35:50 +01:00
|
|
|
/* Flush writes to the server and return any errors */
|
2019-04-07 19:59:04 +02:00
|
|
|
return nfs_wb_all(inode);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2012-07-16 22:39:15 +02:00
|
|
|
ssize_t
|
2014-04-03 02:14:12 +02:00
|
|
|
nfs_file_read(struct kiocb *iocb, struct iov_iter *to)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2013-09-16 16:53:17 +02:00
|
|
|
struct inode *inode = file_inode(iocb->ki_filp);
|
2005-04-17 00:20:36 +02:00
|
|
|
ssize_t result;
|
|
|
|
|
2015-04-09 19:52:01 +02:00
|
|
|
if (iocb->ki_flags & IOCB_DIRECT)
|
2016-04-07 17:51:58 +02:00
|
|
|
return nfs_file_direct_read(iocb, to);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2014-03-05 03:53:33 +01:00
|
|
|
dprintk("NFS: read(%pD2, %zu@%lu)\n",
|
2013-09-16 16:53:17 +02:00
|
|
|
iocb->ki_filp,
|
2014-04-03 02:14:12 +02:00
|
|
|
iov_iter_count(to), (unsigned long) iocb->ki_pos);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2016-06-03 23:07:19 +02:00
|
|
|
nfs_start_io_read(inode);
|
|
|
|
result = nfs_revalidate_mapping(inode, iocb->ki_filp->f_mapping);
|
2010-02-01 20:17:23 +01:00
|
|
|
if (!result) {
|
2014-04-03 02:14:12 +02:00
|
|
|
result = generic_file_read_iter(iocb, to);
|
2010-02-01 20:17:23 +01:00
|
|
|
if (result > 0)
|
|
|
|
nfs_add_stats(inode, NFSIOS_NORMALREADBYTES, result);
|
|
|
|
}
|
2016-06-03 23:07:19 +02:00
|
|
|
nfs_end_io_read(inode);
|
2005-04-17 00:20:36 +02:00
|
|
|
return result;
|
|
|
|
}
|
2012-07-30 22:05:25 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_file_read);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2012-07-16 22:39:15 +02:00
|
|
|
int
|
2005-04-17 00:20:36 +02:00
|
|
|
nfs_file_mmap(struct file * file, struct vm_area_struct * vma)
|
|
|
|
{
|
2013-09-16 16:53:17 +02:00
|
|
|
struct inode *inode = file_inode(file);
|
2005-04-17 00:20:36 +02:00
|
|
|
int status;
|
|
|
|
|
2013-09-16 16:53:17 +02:00
|
|
|
dprintk("NFS: mmap(%pD2)\n", file);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2009-03-11 19:37:54 +01:00
|
|
|
/* Note: generic_file_mmap() returns ENOSYS on nommu systems
|
|
|
|
* so we call that before revalidating the mapping
|
|
|
|
*/
|
|
|
|
status = generic_file_mmap(file, vma);
|
2007-07-22 23:09:05 +02:00
|
|
|
if (!status) {
|
|
|
|
vma->vm_ops = &nfs_file_vm_ops;
|
2009-03-11 19:37:54 +01:00
|
|
|
status = nfs_revalidate_mapping(inode, file->f_mapping);
|
2007-07-22 23:09:05 +02:00
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
return status;
|
|
|
|
}
|
2012-07-30 22:05:25 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_file_mmap);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Flush any dirty pages for this process, and check for write errors.
|
|
|
|
* The return status from this call provides a reliable indication of
|
|
|
|
* whether any write errors occurred for this process.
|
|
|
|
*/
|
2016-03-02 17:35:54 +01:00
|
|
|
static int
|
2017-09-11 05:15:50 +02:00
|
|
|
nfs_file_fsync_commit(struct file *file, int datasync)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2007-08-10 23:44:32 +02:00
|
|
|
struct nfs_open_context *ctx = nfs_file_open_context(file);
|
2013-09-16 16:53:17 +02:00
|
|
|
struct inode *inode = file_inode(file);
|
2017-09-11 05:15:50 +02:00
|
|
|
int do_resend, status;
|
2010-07-31 20:29:06 +02:00
|
|
|
int ret = 0;
|
|
|
|
|
2013-09-16 16:53:17 +02:00
|
|
|
dprintk("NFS: fsync file(%pD2) datasync %d\n", file, datasync);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2006-03-20 19:44:14 +01:00
|
|
|
nfs_inc_stats(inode, NFSIOS_VFSFSYNC);
|
2012-09-11 22:01:22 +02:00
|
|
|
do_resend = test_and_clear_bit(NFS_CONTEXT_RESEND_WRITES, &ctx->flags);
|
2010-07-31 20:29:06 +02:00
|
|
|
status = nfs_commit_inode(inode, FLUSH_SYNC);
|
2019-04-07 19:59:05 +02:00
|
|
|
if (status == 0)
|
|
|
|
status = file_check_and_advance_wb_err(file);
|
2012-09-11 22:01:22 +02:00
|
|
|
if (status < 0) {
|
2010-07-31 20:29:06 +02:00
|
|
|
ret = status;
|
2012-09-11 22:01:22 +02:00
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
do_resend |= test_bit(NFS_CONTEXT_RESEND_WRITES, &ctx->flags);
|
|
|
|
if (do_resend)
|
|
|
|
ret = -EAGAIN;
|
|
|
|
out:
|
2012-06-20 21:53:42 +02:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2016-03-02 17:35:54 +01:00
|
|
|
int
|
2012-06-20 21:53:42 +02:00
|
|
|
nfs_file_fsync(struct file *file, loff_t start, loff_t end, int datasync)
|
|
|
|
{
|
|
|
|
int ret;
|
2013-01-23 23:07:38 +01:00
|
|
|
struct inode *inode = file_inode(file);
|
2012-06-20 21:53:42 +02:00
|
|
|
|
2013-08-20 00:59:33 +02:00
|
|
|
trace_nfs_fsync_enter(inode);
|
|
|
|
|
2012-09-11 22:01:22 +02:00
|
|
|
do {
|
2019-04-07 19:59:05 +02:00
|
|
|
ret = file_write_and_wait_range(file, start, end);
|
2012-09-11 22:01:22 +02:00
|
|
|
if (ret != 0)
|
|
|
|
break;
|
2017-09-11 05:15:50 +02:00
|
|
|
ret = nfs_file_fsync_commit(file, datasync);
|
2016-03-02 17:35:54 +01:00
|
|
|
if (!ret)
|
|
|
|
ret = pnfs_sync_inode(inode, !!datasync);
|
2012-09-11 22:19:38 +02:00
|
|
|
/*
|
|
|
|
* If nfs_file_fsync_commit detected a server reboot, then
|
|
|
|
* resend all dirty pages that might have been covered by
|
|
|
|
* the NFS_CONTEXT_RESEND_WRITES flag
|
|
|
|
*/
|
|
|
|
start = 0;
|
|
|
|
end = LLONG_MAX;
|
2012-09-11 22:01:22 +02:00
|
|
|
} while (ret == -EAGAIN);
|
|
|
|
|
2013-08-20 00:59:33 +02:00
|
|
|
trace_nfs_fsync_exit(inode, ret);
|
2010-07-31 20:29:06 +02:00
|
|
|
return ret;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2016-03-02 17:35:54 +01:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_file_fsync);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
/*
|
|
|
|
* Decide whether a read/modify/write cycle may be more efficient
|
|
|
|
* then a modify/write/read cycle when writing to a page in the
|
|
|
|
* page cache.
|
|
|
|
*
|
2019-02-14 10:39:03 +01:00
|
|
|
* Some pNFS layout drivers can only read/write at a certain block
|
|
|
|
* granularity like all block devices and therefore we must perform
|
|
|
|
* read/modify/write whenever a page hasn't read yet and the data
|
|
|
|
* to be written there is not aligned to a block boundary and/or
|
|
|
|
* smaller than the block size.
|
|
|
|
*
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
* The modify/write/read cycle may occur if a page is read before
|
|
|
|
* being completely filled by the writer. In this situation, the
|
|
|
|
* page must be completely written to stable storage on the server
|
|
|
|
* before it can be refilled by reading in the page from the server.
|
|
|
|
* This can lead to expensive, small, FILE_SYNC mode writes being
|
|
|
|
* done.
|
|
|
|
*
|
|
|
|
* It may be more efficient to read the page first if the file is
|
|
|
|
* open for reading in addition to writing, the page is not marked
|
|
|
|
* as Uptodate, it is not dirty or waiting to be committed,
|
|
|
|
* indicating that it was previously allocated and then modified,
|
|
|
|
* that there were valid bytes of data in that range of the file,
|
|
|
|
* and that the new data won't completely replace the old data in
|
|
|
|
* that range of the file.
|
|
|
|
*/
|
2019-02-14 10:39:03 +01:00
|
|
|
static bool nfs_full_page_write(struct page *page, loff_t pos, unsigned int len)
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
{
|
|
|
|
unsigned int pglen = nfs_page_length(page);
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
unsigned int offset = pos & (PAGE_SIZE - 1);
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
unsigned int end = offset + len;
|
|
|
|
|
2019-02-14 10:39:03 +01:00
|
|
|
return !pglen || (end >= pglen && !offset);
|
|
|
|
}
|
2014-09-10 17:23:30 +02:00
|
|
|
|
2019-02-14 10:39:03 +01:00
|
|
|
static bool nfs_want_read_modify_write(struct file *file, struct page *page,
|
|
|
|
loff_t pos, unsigned int len)
|
|
|
|
{
|
|
|
|
/*
|
|
|
|
* Up-to-date pages, those with ongoing or full-page write
|
|
|
|
* don't need read/modify/write
|
|
|
|
*/
|
|
|
|
if (PageUptodate(page) || PagePrivate(page) ||
|
|
|
|
nfs_full_page_write(page, pos, len))
|
|
|
|
return false;
|
|
|
|
|
|
|
|
if (pnfs_ld_read_whole_page(file->f_mapping->host))
|
|
|
|
return true;
|
|
|
|
/* Open for reading too? */
|
|
|
|
if (file->f_mode & FMODE_READ)
|
|
|
|
return true;
|
|
|
|
return false;
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
}
|
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
/*
|
2007-10-16 10:25:16 +02:00
|
|
|
* This does the "real" work of the write. We must allocate and lock the
|
|
|
|
* page to be sent back to the generic routine, which then copies the
|
|
|
|
* data from user space.
|
2005-04-17 00:20:36 +02:00
|
|
|
*
|
|
|
|
* If the writer ends up delaying the write, the writer needs to
|
|
|
|
* increment the page use counts until he is done with the page.
|
|
|
|
*/
|
2007-10-16 10:25:16 +02:00
|
|
|
static int nfs_write_begin(struct file *file, struct address_space *mapping,
|
|
|
|
loff_t pos, unsigned len, unsigned flags,
|
|
|
|
struct page **pagep, void **fsdata)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2007-10-16 10:25:16 +02:00
|
|
|
int ret;
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
pgoff_t index = pos >> PAGE_SHIFT;
|
2007-10-16 10:25:16 +02:00
|
|
|
struct page *page;
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
int once_thru = 0;
|
2007-10-16 10:25:16 +02:00
|
|
|
|
2013-12-17 18:20:16 +01:00
|
|
|
dfprintk(PAGECACHE, "NFS: write_begin(%pD2(%lu), %u@%lld)\n",
|
2013-09-16 16:53:17 +02:00
|
|
|
file, mapping->host->i_ino, len, (long long) pos);
|
2008-06-11 23:55:50 +02:00
|
|
|
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
start:
|
fs: symlink write_begin allocation context fix
With the write_begin/write_end aops, page_symlink was broken because it
could no longer pass a GFP_NOFS type mask into the point where the
allocations happened. They are done in write_begin, which would always
assume that the filesystem can be entered from reclaim. This bug could
cause filesystem deadlocks.
The funny thing with having a gfp_t mask there is that it doesn't really
allow the caller to arbitrarily tinker with the context in which it can be
called. It couldn't ever be GFP_ATOMIC, for example, because it needs to
take the page lock. The only thing any callers care about is __GFP_FS
anyway, so turn that into a single flag.
Add a new flag for write_begin, AOP_FLAG_NOFS. Filesystems can now act on
this flag in their write_begin function. Change __grab_cache_page to
accept a nofs argument as well, to honour that flag (while we're there,
change the name to grab_cache_page_write_begin which is more instructive
and does away with random leading underscores).
This is really a more flexible way to go in the end anyway -- if a
filesystem happens to want any extra allocations aside from the pagecache
ones in ints write_begin function, it may now use GFP_KERNEL (rather than
GFP_NOFS) for common case allocations (eg. ocfs2_alloc_write_ctxt, for a
random example).
[kosaki.motohiro@jp.fujitsu.com: fix ubifs]
[kosaki.motohiro@jp.fujitsu.com: fix fuse]
Signed-off-by: Nick Piggin <npiggin@suse.de>
Reviewed-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: <stable@kernel.org> [2.6.28.x]
Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
[ Cleaned up the calling convention: just pass in the AOP flags
untouched to the grab_cache_page_write_begin() function. That
just simplifies everybody, and may even allow future expansion of the
logic. - Linus ]
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2009-01-04 21:00:53 +01:00
|
|
|
page = grab_cache_page_write_begin(mapping, index, flags);
|
2007-10-16 10:25:16 +02:00
|
|
|
if (!page)
|
|
|
|
return -ENOMEM;
|
|
|
|
*pagep = page;
|
|
|
|
|
|
|
|
ret = nfs_flush_incompatible(file, page);
|
|
|
|
if (ret) {
|
|
|
|
unlock_page(page);
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
put_page(page);
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
} else if (!once_thru &&
|
|
|
|
nfs_want_read_modify_write(file, page, pos, len)) {
|
|
|
|
once_thru = 1;
|
|
|
|
ret = nfs_readpage(file, page);
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
put_page(page);
|
NFS: read-modify-write page updating
Hi.
I have a proposal for possibly resolving this issue.
I believe that this situation occurs due to the way that the
Linux NFS client handles writes which modify partial pages.
The Linux NFS client handles partial page modifications by
allocating a page from the page cache, copying the data from
the user level into the page, and then keeping track of the
offset and length of the modified portions of the page. The
page is not marked as up to date because there are portions
of the page which do not contain valid file contents.
When a read call comes in for a portion of the page, the
contents of the page must be read in the from the server.
However, since the page may already contain some modified
data, that modified data must be written to the server
before the file contents can be read back in the from server.
And, since the writing and reading can not be done atomically,
the data must be written and committed to stable storage on
the server for safety purposes. This means either a
FILE_SYNC WRITE or a UNSTABLE WRITE followed by a COMMIT.
This has been discussed at length previously.
This algorithm could be described as modify-write-read. It
is most efficient when the application only updates pages
and does not read them.
My proposed solution is to add a heuristic to decide whether
to do this modify-write-read algorithm or switch to a read-
modify-write algorithm when initially allocating the page
in the write system call path. The heuristic uses the modes
that the file was opened with, the offset in the page to
read from, and the size of the region to read.
If the file was opened for reading in addition to writing
and the page would not be filled completely with data from
the user level, then read in the old contents of the page
and mark it as Uptodate before copying in the new data. If
the page would be completely filled with data from the user
level, then there would be no reason to read in the old
contents because they would just be copied over.
This would optimize for applications which randomly access
and update portions of files. The linkage editor for the
C compiler is an example of such a thing.
I tested the attached patch by using rpmbuild to build the
current Fedora rawhide kernel. The kernel without the
patch generated about 269,500 WRITE requests. The modified
kernel containing the patch generated about 261,000 WRITE
requests. Thus, about 8,500 fewer WRITE requests were
generated. I suspect that many of these additional
WRITE requests were probably FILE_SYNC requests to WRITE
a single page, but I didn't test this theory.
The difference between this patch and the previous one was
to remove the unneeded PageDirty() test. I then retested to
ensure that the resulting system continued to behave as
desired.
Thanx...
ps
Signed-off-by: Peter Staubach <staubach@redhat.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
2009-08-10 14:54:16 +02:00
|
|
|
if (!ret)
|
|
|
|
goto start;
|
2007-10-16 10:25:16 +02:00
|
|
|
}
|
|
|
|
return ret;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2007-10-16 10:25:16 +02:00
|
|
|
static int nfs_write_end(struct file *file, struct address_space *mapping,
|
|
|
|
loff_t pos, unsigned len, unsigned copied,
|
|
|
|
struct page *page, void *fsdata)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
unsigned offset = pos & (PAGE_SIZE - 1);
|
2013-08-14 17:59:16 +02:00
|
|
|
struct nfs_open_context *ctx = nfs_file_open_context(file);
|
2007-10-16 10:25:16 +02:00
|
|
|
int status;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2013-12-17 18:20:16 +01:00
|
|
|
dfprintk(PAGECACHE, "NFS: write_end(%pD2(%lu), %u@%lld)\n",
|
2013-09-16 16:53:17 +02:00
|
|
|
file, mapping->host->i_ino, len, (long long) pos);
|
2008-06-11 23:55:50 +02:00
|
|
|
|
2008-06-11 00:31:00 +02:00
|
|
|
/*
|
|
|
|
* Zero any uninitialised parts of the page, and then mark the page
|
|
|
|
* as up to date if it turns out that we're extending the file.
|
|
|
|
*/
|
|
|
|
if (!PageUptodate(page)) {
|
|
|
|
unsigned pglen = nfs_page_length(page);
|
2016-09-06 03:42:32 +02:00
|
|
|
unsigned end = offset + copied;
|
2008-06-11 00:31:00 +02:00
|
|
|
|
|
|
|
if (pglen == 0) {
|
|
|
|
zero_user_segments(page, 0, offset,
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
end, PAGE_SIZE);
|
2008-06-11 00:31:00 +02:00
|
|
|
SetPageUptodate(page);
|
|
|
|
} else if (end >= pglen) {
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
zero_user_segment(page, end, PAGE_SIZE);
|
2008-06-11 00:31:00 +02:00
|
|
|
if (offset == 0)
|
|
|
|
SetPageUptodate(page);
|
|
|
|
} else
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
zero_user_segment(page, pglen, PAGE_SIZE);
|
2008-06-11 00:31:00 +02:00
|
|
|
}
|
|
|
|
|
2007-10-16 10:25:16 +02:00
|
|
|
status = nfs_updatepage(file, page, offset, copied);
|
|
|
|
|
|
|
|
unlock_page(page);
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
put_page(page);
|
2007-10-16 10:25:16 +02:00
|
|
|
|
2007-12-20 20:55:04 +01:00
|
|
|
if (status < 0)
|
|
|
|
return status;
|
2012-05-24 19:13:24 +02:00
|
|
|
NFS_I(mapping->host)->write_io += copied;
|
2013-08-14 17:59:16 +02:00
|
|
|
|
sunrpc: move NO_CRKEY_TIMEOUT to the auth->au_flags
A generic_cred can be used to look up a unx_cred or a gss_cred, so it's
not really safe to use the the generic_cred->acred->ac_flags to store
the NO_CRKEY_TIMEOUT flag. A lookup for a unx_cred triggered while the
KEY_EXPIRE_SOON flag is already set will cause both NO_CRKEY_TIMEOUT and
KEY_EXPIRE_SOON to be set in the ac_flags, leaving the user associated
with the auth_cred to be in a state where they're perpetually doing 4K
NFS_FILE_SYNC writes.
This can be reproduced as follows:
1. Mount two NFS filesystems, one with sec=krb5 and one with sec=sys.
They do not need to be the same export, nor do they even need to be from
the same NFS server. Also, v3 is fine.
$ sudo mount -o v3,sec=krb5 server1:/export /mnt/krb5
$ sudo mount -o v3,sec=sys server2:/export /mnt/sys
2. As the normal user, before accessing the kerberized mount, kinit with
a short lifetime (but not so short that renewing the ticket would leave
you within the 4-minute window again by the time the original ticket
expires), e.g.
$ kinit -l 10m -r 60m
3. Do some I/O to the kerberized mount and verify that the writes are
wsize, UNSTABLE:
$ dd if=/dev/zero of=/mnt/krb5/file bs=1M count=1
4. Wait until you're within 4 minutes of key expiry, then do some more
I/O to the kerberized mount to ensure that RPC_CRED_KEY_EXPIRE_SOON gets
set. Verify that the writes are 4K, FILE_SYNC:
$ dd if=/dev/zero of=/mnt/krb5/file bs=1M count=1
5. Now do some I/O to the sec=sys mount. This will cause
RPC_CRED_NO_CRKEY_TIMEOUT to be set:
$ dd if=/dev/zero of=/mnt/sys/file bs=1M count=1
6. Writes for that user will now be permanently 4K, FILE_SYNC for that
user, regardless of which mount is being written to, until you reboot
the client. Renewing the kerberos ticket (assuming it hasn't already
expired) will have no effect. Grabbing a new kerberos ticket at this
point will have no effect either.
Move the flag to the auth->au_flags field (which is currently unused)
and rename it slightly to reflect that it's no longer associated with
the auth_cred->ac_flags. Add the rpc_auth to the arg list of
rpcauth_cred_key_to_expire and check the au_flags there too. Finally,
add the inode to the arg list of nfs_ctx_key_to_expire so we can
determine the rpc_auth to pass to rpcauth_cred_key_to_expire.
Signed-off-by: Scott Mayhew <smayhew@redhat.com>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
2016-06-07 21:14:48 +02:00
|
|
|
if (nfs_ctx_key_to_expire(ctx, mapping->host)) {
|
2013-08-14 17:59:16 +02:00
|
|
|
status = nfs_wb_all(mapping->host);
|
|
|
|
if (status < 0)
|
|
|
|
return status;
|
|
|
|
}
|
|
|
|
|
2007-12-20 20:55:04 +01:00
|
|
|
return copied;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2009-04-03 17:42:41 +02:00
|
|
|
/*
|
|
|
|
* Partially or wholly invalidate a page
|
|
|
|
* - Release the private state associated with a page if undergoing complete
|
|
|
|
* page invalidation
|
2009-04-03 17:42:44 +02:00
|
|
|
* - Called if either PG_private or PG_fscache is set on the page
|
2009-04-03 17:42:41 +02:00
|
|
|
* - Caller holds page lock
|
|
|
|
*/
|
2013-05-22 05:17:23 +02:00
|
|
|
static void nfs_invalidate_page(struct page *page, unsigned int offset,
|
|
|
|
unsigned int length)
|
2006-03-20 19:44:04 +01:00
|
|
|
{
|
2013-05-22 05:17:23 +02:00
|
|
|
dfprintk(PAGECACHE, "NFS: invalidate_page(%p, %u, %u)\n",
|
|
|
|
page, offset, length);
|
2008-06-11 23:55:50 +02:00
|
|
|
|
mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were introduced *long* time
ago with promise that one day it will be possible to implement page
cache with bigger chunks than PAGE_SIZE.
This promise never materialized. And unlikely will.
We have many places where PAGE_CACHE_SIZE assumed to be equal to
PAGE_SIZE. And it's constant source of confusion on whether
PAGE_CACHE_* or PAGE_* constant should be used in a particular case,
especially on the border between fs and mm.
Global switching to PAGE_CACHE_SIZE != PAGE_SIZE would cause to much
breakage to be doable.
Let's stop pretending that pages in page cache are special. They are
not.
The changes are pretty straight-forward:
- <foo> << (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- <foo> >> (PAGE_CACHE_SHIFT - PAGE_SHIFT) -> <foo>;
- PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} -> PAGE_{SIZE,SHIFT,MASK,ALIGN};
- page_cache_get() -> get_page();
- page_cache_release() -> put_page();
This patch contains automated changes generated with coccinelle using
script below. For some reason, coccinelle doesn't patch header files.
I've called spatch for them manually.
The only adjustment after coccinelle is revert of changes to
PAGE_CAHCE_ALIGN definition: we are going to drop it later.
There are few places in the code where coccinelle didn't reach. I'll
fix them manually in a separate patch. Comments and documentation also
will be addressed with the separate patch.
virtual patch
@@
expression E;
@@
- E << (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
expression E;
@@
- E >> (PAGE_CACHE_SHIFT - PAGE_SHIFT)
+ E
@@
@@
- PAGE_CACHE_SHIFT
+ PAGE_SHIFT
@@
@@
- PAGE_CACHE_SIZE
+ PAGE_SIZE
@@
@@
- PAGE_CACHE_MASK
+ PAGE_MASK
@@
expression E;
@@
- PAGE_CACHE_ALIGN(E)
+ PAGE_ALIGN(E)
@@
expression E;
@@
- page_cache_get(E)
+ get_page(E)
@@
expression E;
@@
- page_cache_release(E)
+ put_page(E)
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2016-04-01 14:29:47 +02:00
|
|
|
if (offset != 0 || length < PAGE_SIZE)
|
2006-10-09 22:18:38 +02:00
|
|
|
return;
|
2006-05-31 07:13:38 +02:00
|
|
|
/* Cancel any unstarted writes on this page */
|
2012-08-01 01:45:06 +02:00
|
|
|
nfs_wb_page_cancel(page_file_mapping(page)->host, page);
|
2009-04-03 17:42:44 +02:00
|
|
|
|
|
|
|
nfs_fscache_invalidate_page(page, page->mapping->host);
|
2006-03-20 19:44:04 +01:00
|
|
|
}
|
|
|
|
|
2009-04-03 17:42:41 +02:00
|
|
|
/*
|
|
|
|
* Attempt to release the private state associated with a page
|
2009-04-03 17:42:44 +02:00
|
|
|
* - Called if either PG_private or PG_fscache is set on the page
|
2009-04-03 17:42:41 +02:00
|
|
|
* - Caller holds page lock
|
|
|
|
* - Return true (may release page) or false (may not)
|
|
|
|
*/
|
2006-03-20 19:44:04 +01:00
|
|
|
static int nfs_release_page(struct page *page, gfp_t gfp)
|
|
|
|
{
|
2008-06-11 23:55:50 +02:00
|
|
|
dfprintk(PAGECACHE, "NFS: release_page(%p)\n", page);
|
|
|
|
|
2007-01-11 08:15:39 +01:00
|
|
|
/* If PagePrivate() is set, then the page is not freeable */
|
2009-04-03 17:42:44 +02:00
|
|
|
if (PagePrivate(page))
|
|
|
|
return 0;
|
|
|
|
return nfs_fscache_release_page(page, gfp);
|
2007-01-11 08:15:39 +01:00
|
|
|
}
|
|
|
|
|
2013-07-04 00:02:06 +02:00
|
|
|
static void nfs_check_dirty_writeback(struct page *page,
|
|
|
|
bool *dirty, bool *writeback)
|
|
|
|
{
|
|
|
|
struct nfs_inode *nfsi;
|
|
|
|
struct address_space *mapping = page_file_mapping(page);
|
|
|
|
|
|
|
|
if (!mapping || PageSwapCache(page))
|
|
|
|
return;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Check if an unstable page is currently being committed and
|
|
|
|
* if so, have the VM treat it as if the page is under writeback
|
|
|
|
* so it will not block due to pages that will shortly be freeable.
|
|
|
|
*/
|
|
|
|
nfsi = NFS_I(mapping->host);
|
2015-09-30 02:34:05 +02:00
|
|
|
if (atomic_read(&nfsi->commit_info.rpcs_out)) {
|
2013-07-04 00:02:06 +02:00
|
|
|
*writeback = true;
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* If PagePrivate() is set, then the page is not freeable and as the
|
|
|
|
* inode is not being committed, it's not going to be cleaned in the
|
|
|
|
* near future so treat it as dirty
|
|
|
|
*/
|
|
|
|
if (PagePrivate(page))
|
|
|
|
*dirty = true;
|
|
|
|
}
|
|
|
|
|
2009-04-03 17:42:41 +02:00
|
|
|
/*
|
|
|
|
* Attempt to clear the private state associated with a page when an error
|
|
|
|
* occurs that requires the cached contents of an inode to be written back or
|
|
|
|
* destroyed
|
2009-04-03 17:42:44 +02:00
|
|
|
* - Called if either PG_private or fscache is set on the page
|
2009-04-03 17:42:41 +02:00
|
|
|
* - Caller holds page lock
|
|
|
|
* - Return 0 if successful, -error otherwise
|
|
|
|
*/
|
2007-01-11 08:15:39 +01:00
|
|
|
static int nfs_launder_page(struct page *page)
|
|
|
|
{
|
2012-08-01 01:45:06 +02:00
|
|
|
struct inode *inode = page_file_mapping(page)->host;
|
2009-04-03 17:42:44 +02:00
|
|
|
struct nfs_inode *nfsi = NFS_I(inode);
|
2008-06-11 23:55:50 +02:00
|
|
|
|
|
|
|
dfprintk(PAGECACHE, "NFS: launder_page(%ld, %llu)\n",
|
|
|
|
inode->i_ino, (long long)page_offset(page));
|
|
|
|
|
2009-04-03 17:42:44 +02:00
|
|
|
nfs_fscache_wait_on_page_write(nfsi, page);
|
2017-04-26 18:26:22 +02:00
|
|
|
return nfs_wb_page(inode, page);
|
2006-03-20 19:44:04 +01:00
|
|
|
}
|
|
|
|
|
2012-08-01 01:45:12 +02:00
|
|
|
static int nfs_swap_activate(struct swap_info_struct *sis, struct file *file,
|
|
|
|
sector_t *span)
|
|
|
|
{
|
2014-09-10 15:03:55 +02:00
|
|
|
struct rpc_clnt *clnt = NFS_CLIENT(file->f_mapping->host);
|
|
|
|
|
2012-08-01 01:45:12 +02:00
|
|
|
*span = sis->pages;
|
2014-09-10 15:03:55 +02:00
|
|
|
|
2015-06-03 22:14:25 +02:00
|
|
|
return rpc_clnt_swap_activate(clnt);
|
2012-08-01 01:45:12 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
static void nfs_swap_deactivate(struct file *file)
|
|
|
|
{
|
2014-09-10 15:03:55 +02:00
|
|
|
struct rpc_clnt *clnt = NFS_CLIENT(file->f_mapping->host);
|
|
|
|
|
2015-06-03 22:14:25 +02:00
|
|
|
rpc_clnt_swap_deactivate(clnt);
|
2012-08-01 01:45:12 +02:00
|
|
|
}
|
|
|
|
|
2006-06-28 13:26:44 +02:00
|
|
|
const struct address_space_operations nfs_file_aops = {
|
2005-04-17 00:20:36 +02:00
|
|
|
.readpage = nfs_readpage,
|
|
|
|
.readpages = nfs_readpages,
|
2007-07-22 23:09:05 +02:00
|
|
|
.set_page_dirty = __set_page_dirty_nobuffers,
|
2005-04-17 00:20:36 +02:00
|
|
|
.writepage = nfs_writepage,
|
|
|
|
.writepages = nfs_writepages,
|
2007-10-16 10:25:16 +02:00
|
|
|
.write_begin = nfs_write_begin,
|
|
|
|
.write_end = nfs_write_end,
|
2006-03-20 19:44:04 +01:00
|
|
|
.invalidatepage = nfs_invalidate_page,
|
|
|
|
.releasepage = nfs_release_page,
|
2005-04-17 00:20:36 +02:00
|
|
|
.direct_IO = nfs_direct_IO,
|
2016-09-20 07:59:07 +02:00
|
|
|
#ifdef CONFIG_MIGRATION
|
2009-08-10 14:54:13 +02:00
|
|
|
.migratepage = nfs_migrate_page,
|
2016-09-20 07:59:07 +02:00
|
|
|
#endif
|
2007-01-11 08:15:39 +01:00
|
|
|
.launder_page = nfs_launder_page,
|
2013-07-04 00:02:06 +02:00
|
|
|
.is_dirty_writeback = nfs_check_dirty_writeback,
|
2009-09-16 11:50:17 +02:00
|
|
|
.error_remove_page = generic_error_remove_page,
|
2012-08-01 01:45:12 +02:00
|
|
|
.swap_activate = nfs_swap_activate,
|
|
|
|
.swap_deactivate = nfs_swap_deactivate,
|
2005-04-17 00:20:36 +02:00
|
|
|
};
|
|
|
|
|
2009-04-03 17:42:41 +02:00
|
|
|
/*
|
|
|
|
* Notification that a PTE pointing to an NFS page is about to be made
|
|
|
|
* writable, implying that someone is about to modify the page through a
|
|
|
|
* shared-writable mapping
|
|
|
|
*/
|
2018-07-02 17:27:09 +02:00
|
|
|
static vm_fault_t nfs_vm_page_mkwrite(struct vm_fault *vmf)
|
2007-07-22 23:09:05 +02:00
|
|
|
{
|
2009-04-01 00:23:21 +02:00
|
|
|
struct page *page = vmf->page;
|
2017-02-24 23:56:41 +01:00
|
|
|
struct file *filp = vmf->vma->vm_file;
|
2013-09-16 16:53:17 +02:00
|
|
|
struct inode *inode = file_inode(filp);
|
2007-07-22 23:09:05 +02:00
|
|
|
unsigned pagelen;
|
2018-07-02 17:27:09 +02:00
|
|
|
vm_fault_t ret = VM_FAULT_NOPAGE;
|
2007-10-16 10:25:16 +02:00
|
|
|
struct address_space *mapping;
|
2007-07-22 23:09:05 +02:00
|
|
|
|
2013-12-17 18:20:16 +01:00
|
|
|
dfprintk(PAGECACHE, "NFS: vm_page_mkwrite(%pD2(%lu), offset %lld)\n",
|
2013-09-16 16:53:17 +02:00
|
|
|
filp, filp->f_mapping->host->i_ino,
|
2008-06-11 23:55:50 +02:00
|
|
|
(long long)page_offset(page));
|
|
|
|
|
2016-06-23 17:09:04 +02:00
|
|
|
sb_start_pagefault(inode->i_sb);
|
|
|
|
|
2009-04-03 17:42:44 +02:00
|
|
|
/* make sure the cache has finished storing the page */
|
2013-09-16 16:53:17 +02:00
|
|
|
nfs_fscache_wait_on_page_write(NFS_I(inode), page);
|
2009-04-03 17:42:44 +02:00
|
|
|
|
2015-03-03 06:06:35 +01:00
|
|
|
wait_on_bit_action(&NFS_I(inode)->flags, NFS_INO_INVALIDATING,
|
|
|
|
nfs_wait_bit_killable, TASK_KILLABLE);
|
|
|
|
|
2007-07-22 23:09:05 +02:00
|
|
|
lock_page(page);
|
2012-08-01 01:45:06 +02:00
|
|
|
mapping = page_file_mapping(page);
|
2013-09-16 16:53:17 +02:00
|
|
|
if (mapping != inode->i_mapping)
|
2008-01-22 23:13:06 +01:00
|
|
|
goto out_unlock;
|
|
|
|
|
2012-01-18 04:04:26 +01:00
|
|
|
wait_on_page_writeback(page);
|
|
|
|
|
2007-07-22 23:09:05 +02:00
|
|
|
pagelen = nfs_page_length(page);
|
2008-01-22 23:13:06 +01:00
|
|
|
if (pagelen == 0)
|
|
|
|
goto out_unlock;
|
2007-10-16 10:25:16 +02:00
|
|
|
|
2010-10-04 23:59:08 +02:00
|
|
|
ret = VM_FAULT_LOCKED;
|
|
|
|
if (nfs_flush_incompatible(filp, page) == 0 &&
|
|
|
|
nfs_updatepage(filp, page, 0, pagelen) == 0)
|
|
|
|
goto out;
|
2008-01-22 23:13:06 +01:00
|
|
|
|
2010-10-04 23:59:08 +02:00
|
|
|
ret = VM_FAULT_SIGBUS;
|
2008-01-22 23:13:06 +01:00
|
|
|
out_unlock:
|
|
|
|
unlock_page(page);
|
2010-10-04 23:59:08 +02:00
|
|
|
out:
|
2016-06-23 17:09:04 +02:00
|
|
|
sb_end_pagefault(inode->i_sb);
|
2010-10-04 23:59:08 +02:00
|
|
|
return ret;
|
2007-07-22 23:09:05 +02:00
|
|
|
}
|
|
|
|
|
2009-09-27 20:29:37 +02:00
|
|
|
static const struct vm_operations_struct nfs_file_vm_ops = {
|
2007-07-22 23:09:05 +02:00
|
|
|
.fault = filemap_fault,
|
2014-04-08 00:37:19 +02:00
|
|
|
.map_pages = filemap_map_pages,
|
2007-07-22 23:09:05 +02:00
|
|
|
.page_mkwrite = nfs_vm_page_mkwrite,
|
|
|
|
};
|
|
|
|
|
2015-08-17 23:55:18 +02:00
|
|
|
static int nfs_need_check_write(struct file *filp, struct inode *inode)
|
2007-07-25 20:09:54 +02:00
|
|
|
{
|
|
|
|
struct nfs_open_context *ctx;
|
|
|
|
|
2007-08-10 23:44:32 +02:00
|
|
|
ctx = nfs_file_open_context(filp);
|
2019-04-07 19:59:05 +02:00
|
|
|
if (nfs_ctx_key_to_expire(ctx, inode))
|
2007-07-25 20:09:54 +02:00
|
|
|
return 1;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2014-04-03 20:07:25 +02:00
|
|
|
ssize_t nfs_file_write(struct kiocb *iocb, struct iov_iter *from)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2013-09-16 16:53:17 +02:00
|
|
|
struct file *file = iocb->ki_filp;
|
|
|
|
struct inode *inode = file_inode(file);
|
2010-02-01 20:17:41 +01:00
|
|
|
unsigned long written = 0;
|
2005-04-17 00:20:36 +02:00
|
|
|
ssize_t result;
|
|
|
|
|
2013-09-16 16:53:17 +02:00
|
|
|
result = nfs_key_timeout_notify(file, inode);
|
2013-08-14 17:59:16 +02:00
|
|
|
if (result)
|
|
|
|
return result;
|
|
|
|
|
2016-06-23 16:35:48 +02:00
|
|
|
if (iocb->ki_flags & IOCB_DIRECT)
|
2015-04-09 20:11:08 +02:00
|
|
|
return nfs_file_direct_write(iocb, from);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2014-03-05 03:53:33 +01:00
|
|
|
dprintk("NFS: write(%pD2, %zu@%Ld)\n",
|
2016-06-23 21:00:42 +02:00
|
|
|
file, iov_iter_count(from), (long long) iocb->ki_pos);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
if (IS_SWAPFILE(inode))
|
|
|
|
goto out_swapfile;
|
2005-06-22 19:16:30 +02:00
|
|
|
/*
|
|
|
|
* O_APPEND implies that we must revalidate the file length.
|
|
|
|
*/
|
2015-04-09 19:52:01 +02:00
|
|
|
if (iocb->ki_flags & IOCB_APPEND) {
|
2013-09-16 16:53:17 +02:00
|
|
|
result = nfs_revalidate_file_size(inode, file);
|
2005-06-22 19:16:30 +02:00
|
|
|
if (result)
|
|
|
|
goto out;
|
2005-06-22 19:16:30 +02:00
|
|
|
}
|
NFS: Optimize fallocate by refreshing mapping when needed.
posix_fallocate() will allocate space in an NFS file by considering
the last byte of every 4K block. If it is before EOF, it will read
the byte and if it is zero, a zero is written out. If it is after EOF,
the zero is unconditionally written.
For the blocks beyond EOF, if NFS believes its cache is valid, it will
expand these writes to write full pages, and then will merge the pages.
This results if (typically) 1MB writes. If NFS believes its cache is
not valid (particularly if NFS_INO_INVALID_DATA or
NFS_INO_REVAL_PAGECACHE are set - see nfs_write_pageuptodate()), it will
send the individual 1-byte writes. This results in (typically) 256 times
as many RPC requests, and can be substantially slower.
Currently nfs_revalidate_mapping() is only used when reading a file or
mmapping a file, as these are times when the content needs to be
up-to-date. Writes don't generally need the cache to be up-to-date, but
writes beyond EOF can benefit, particularly in the posix_fallocate()
case.
So this patch calls nfs_revalidate_mapping() when writing beyond EOF -
i.e. when there is a gap between the end of the file and the start of
the write. If the cache is thought to be out of date (as happens after
taking a file lock), this will cause a GETATTR, and the two flags
mentioned above will be cleared. With this, posix_fallocate() on a
newly locked file does not generate excessive tiny writes.
Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
2017-07-24 05:18:50 +02:00
|
|
|
if (iocb->ki_pos > i_size_read(inode))
|
|
|
|
nfs_revalidate_mapping(inode, file->f_mapping);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2016-06-03 23:07:19 +02:00
|
|
|
nfs_start_io_write(inode);
|
2016-06-23 21:00:42 +02:00
|
|
|
result = generic_write_checks(iocb, from);
|
|
|
|
if (result > 0) {
|
|
|
|
current->backing_dev_info = inode_to_bdi(inode);
|
|
|
|
result = generic_perform_write(file, from, iocb->ki_pos);
|
|
|
|
current->backing_dev_info = NULL;
|
|
|
|
}
|
2016-06-03 23:07:19 +02:00
|
|
|
nfs_end_io_write(inode);
|
2016-06-23 21:00:42 +02:00
|
|
|
if (result <= 0)
|
2005-04-17 00:20:36 +02:00
|
|
|
goto out;
|
|
|
|
|
2016-09-03 18:05:31 +02:00
|
|
|
written = result;
|
2016-06-23 21:00:42 +02:00
|
|
|
iocb->ki_pos += written;
|
2017-09-07 15:29:23 +02:00
|
|
|
result = generic_write_sync(iocb, written);
|
|
|
|
if (result < 0)
|
|
|
|
goto out;
|
2010-02-01 20:17:41 +01:00
|
|
|
|
2015-08-17 23:55:18 +02:00
|
|
|
/* Return error values */
|
2016-06-23 21:00:42 +02:00
|
|
|
if (nfs_need_check_write(file, inode)) {
|
2019-04-07 19:59:04 +02:00
|
|
|
int err = nfs_wb_all(inode);
|
2006-12-05 06:35:40 +01:00
|
|
|
if (err < 0)
|
|
|
|
result = err;
|
|
|
|
}
|
2016-06-23 21:00:42 +02:00
|
|
|
nfs_add_stats(inode, NFSIOS_NORMALWRITTENBYTES, written);
|
2005-04-17 00:20:36 +02:00
|
|
|
out:
|
|
|
|
return result;
|
|
|
|
|
|
|
|
out_swapfile:
|
|
|
|
printk(KERN_INFO "NFS: attempt to write to active swap file!\n");
|
2016-06-23 21:00:42 +02:00
|
|
|
return -EBUSY;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2012-07-30 22:05:25 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_file_write);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2010-09-23 14:55:58 +02:00
|
|
|
static int
|
|
|
|
do_getlk(struct file *filp, int cmd, struct file_lock *fl, int is_local)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
|
|
|
struct inode *inode = filp->f_mapping->host;
|
|
|
|
int status = 0;
|
2010-11-28 22:04:05 +01:00
|
|
|
unsigned int saved_type = fl->fl_type;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2005-10-18 23:20:16 +02:00
|
|
|
/* Try local locking first */
|
2007-05-11 22:09:32 +02:00
|
|
|
posix_test_lock(filp, fl);
|
|
|
|
if (fl->fl_type != F_UNLCK) {
|
|
|
|
/* found a conflict */
|
2005-10-18 23:20:16 +02:00
|
|
|
goto out;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2010-11-28 22:04:05 +01:00
|
|
|
fl->fl_type = saved_type;
|
2005-10-18 23:20:16 +02:00
|
|
|
|
2012-06-20 21:53:43 +02:00
|
|
|
if (NFS_PROTO(inode)->have_delegation(inode, FMODE_READ))
|
2005-10-18 23:20:16 +02:00
|
|
|
goto out_noconflict;
|
|
|
|
|
2010-09-23 14:55:58 +02:00
|
|
|
if (is_local)
|
2005-10-18 23:20:16 +02:00
|
|
|
goto out_noconflict;
|
|
|
|
|
|
|
|
status = NFS_PROTO(inode)->lock(filp, cmd, fl);
|
|
|
|
out:
|
2005-04-17 00:20:36 +02:00
|
|
|
return status;
|
2005-10-18 23:20:16 +02:00
|
|
|
out_noconflict:
|
|
|
|
fl->fl_type = F_UNLCK;
|
|
|
|
goto out;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
|
|
|
|
2010-09-23 14:55:58 +02:00
|
|
|
static int
|
|
|
|
do_unlk(struct file *filp, int cmd, struct file_lock *fl, int is_local)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
|
|
|
struct inode *inode = filp->f_mapping->host;
|
2013-04-09 03:49:53 +02:00
|
|
|
struct nfs_lock_context *l_ctx;
|
2005-04-17 00:20:36 +02:00
|
|
|
int status;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Flush all pending writes before doing anything
|
|
|
|
* with locks..
|
|
|
|
*/
|
2019-04-07 19:59:04 +02:00
|
|
|
nfs_wb_all(inode);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2013-04-09 03:49:53 +02:00
|
|
|
l_ctx = nfs_get_lock_context(nfs_file_open_context(filp));
|
|
|
|
if (!IS_ERR(l_ctx)) {
|
2016-01-06 16:40:18 +01:00
|
|
|
status = nfs_iocounter_wait(l_ctx);
|
2013-04-09 03:49:53 +02:00
|
|
|
nfs_put_lock_context(l_ctx);
|
2017-04-11 18:50:12 +02:00
|
|
|
/* NOTE: special case
|
|
|
|
* If we're signalled while cleaning up locks on process exit, we
|
|
|
|
* still need to complete the unlock.
|
|
|
|
*/
|
|
|
|
if (status < 0 && !(fl->fl_flags & FL_CLOSE))
|
2013-04-09 03:49:53 +02:00
|
|
|
return status;
|
|
|
|
}
|
|
|
|
|
2010-09-23 14:55:58 +02:00
|
|
|
/*
|
|
|
|
* Use local locking if mounted with "-onolock" or with appropriate
|
|
|
|
* "-olocal_lock="
|
|
|
|
*/
|
|
|
|
if (!is_local)
|
2005-04-17 00:20:36 +02:00
|
|
|
status = NFS_PROTO(inode)->lock(filp, cmd, fl);
|
|
|
|
else
|
2016-09-18 00:17:32 +02:00
|
|
|
status = locks_lock_file_wait(filp, fl);
|
2005-04-17 00:20:36 +02:00
|
|
|
return status;
|
|
|
|
}
|
|
|
|
|
2010-09-23 14:55:58 +02:00
|
|
|
static int
|
|
|
|
do_setlk(struct file *filp, int cmd, struct file_lock *fl, int is_local)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
|
|
|
struct inode *inode = filp->f_mapping->host;
|
|
|
|
int status;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Flush all pending writes before doing anything
|
|
|
|
* with locks..
|
|
|
|
*/
|
2005-12-13 22:13:54 +01:00
|
|
|
status = nfs_sync_mapping(filp->f_mapping);
|
|
|
|
if (status != 0)
|
2005-04-17 00:20:36 +02:00
|
|
|
goto out;
|
|
|
|
|
2010-09-23 14:55:58 +02:00
|
|
|
/*
|
|
|
|
* Use local locking if mounted with "-onolock" or with appropriate
|
|
|
|
* "-olocal_lock="
|
|
|
|
*/
|
|
|
|
if (!is_local)
|
2005-04-17 00:20:36 +02:00
|
|
|
status = NFS_PROTO(inode)->lock(filp, cmd, fl);
|
2008-04-02 02:26:52 +02:00
|
|
|
else
|
2016-09-18 00:17:32 +02:00
|
|
|
status = locks_lock_file_wait(filp, fl);
|
2005-04-17 00:20:36 +02:00
|
|
|
if (status < 0)
|
|
|
|
goto out;
|
2010-10-13 01:30:05 +02:00
|
|
|
|
2005-04-17 00:20:36 +02:00
|
|
|
/*
|
2017-08-18 09:12:52 +02:00
|
|
|
* Invalidate cache to prevent missing any changes. If
|
|
|
|
* the file is mapped, clear the page cache as well so
|
|
|
|
* those mappings will be loaded.
|
2010-10-13 01:30:05 +02:00
|
|
|
*
|
2005-04-17 00:20:36 +02:00
|
|
|
* This makes locking act as a cache coherency point.
|
|
|
|
*/
|
2005-12-13 22:13:54 +01:00
|
|
|
nfs_sync_mapping(filp->f_mapping);
|
2017-08-18 09:12:52 +02:00
|
|
|
if (!NFS_PROTO(inode)->have_delegation(inode, FMODE_READ)) {
|
NFS: invalidate file size when taking a lock.
Prior to commit ca0daa277aca ("NFS: Cache aggressively when file is open
for writing"), NFS would revalidate, or invalidate, the file size when
taking a lock. Since that commit it only invalidates the file content.
If the file size is changed on the server while wait for the lock, the
client will have an incorrect understanding of the file size and could
corrupt data. This particularly happens when writing beyond the
(supposed) end of file and can be easily be demonstrated with
posix_fallocate().
If an application opens an empty file, waits for a write lock, and then
calls posix_fallocate(), glibc will determine that the underlying
filesystem doesn't support fallocate (assuming version 4.1 or earlier)
and will write out a '0' byte at the end of each 4K page in the region
being fallocated that is after the end of the file.
NFS will (usually) detect that these writes are beyond EOF and will
expand them to cover the whole page, and then will merge the pages.
Consequently, NFS will write out large blocks of zeroes beyond where it
thought EOF was. If EOF had moved, the pre-existing part of the file
will be over-written. Locking should have protected against this,
but it doesn't.
This patch restores the use of nfs_zap_caches() which invalidated the
cached attributes. When posix_fallocate() asks for the file size, the
request will go to the server and get a correct answer.
cc: stable@vger.kernel.org (v4.8+)
Fixes: ca0daa277aca ("NFS: Cache aggressively when file is open for writing")
Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
2017-07-24 05:18:50 +02:00
|
|
|
nfs_zap_caches(inode);
|
2017-08-18 09:12:52 +02:00
|
|
|
if (mapping_mapped(filp->f_mapping))
|
|
|
|
nfs_revalidate_mapping(inode, filp->f_mapping);
|
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
out:
|
|
|
|
return status;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Lock a (portion of) a file
|
|
|
|
*/
|
2012-07-16 22:39:15 +02:00
|
|
|
int nfs_lock(struct file *filp, int cmd, struct file_lock *fl)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2008-06-11 23:55:58 +02:00
|
|
|
struct inode *inode = filp->f_mapping->host;
|
2008-05-21 01:34:39 +02:00
|
|
|
int ret = -ENOLCK;
|
2010-09-23 14:55:58 +02:00
|
|
|
int is_local = 0;
|
2005-04-17 00:20:36 +02:00
|
|
|
|
2013-09-16 16:53:17 +02:00
|
|
|
dprintk("NFS: lock(%pD2, t=%x, fl=%x, r=%lld:%lld)\n",
|
|
|
|
filp, fl->fl_type, fl->fl_flags,
|
2005-04-17 00:20:36 +02:00
|
|
|
(long long)fl->fl_start, (long long)fl->fl_end);
|
2008-06-11 23:55:58 +02:00
|
|
|
|
2006-03-20 19:44:14 +01:00
|
|
|
nfs_inc_stats(inode, NFSIOS_VFSLOCK);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/* No mandatory locks over NFS */
|
2007-10-01 23:41:15 +02:00
|
|
|
if (__mandatory_lock(inode) && fl->fl_type != F_UNLCK)
|
2008-05-21 01:34:39 +02:00
|
|
|
goto out_err;
|
|
|
|
|
2010-09-23 14:55:58 +02:00
|
|
|
if (NFS_SERVER(inode)->flags & NFS_MOUNT_LOCAL_FCNTL)
|
|
|
|
is_local = 1;
|
|
|
|
|
2008-05-21 01:34:39 +02:00
|
|
|
if (NFS_PROTO(inode)->lock_check_bounds != NULL) {
|
|
|
|
ret = NFS_PROTO(inode)->lock_check_bounds(fl);
|
|
|
|
if (ret < 0)
|
|
|
|
goto out_err;
|
|
|
|
}
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
if (IS_GETLK(cmd))
|
2010-09-23 14:55:58 +02:00
|
|
|
ret = do_getlk(filp, cmd, fl, is_local);
|
2008-05-21 01:34:39 +02:00
|
|
|
else if (fl->fl_type == F_UNLCK)
|
2010-09-23 14:55:58 +02:00
|
|
|
ret = do_unlk(filp, cmd, fl, is_local);
|
2008-05-21 01:34:39 +02:00
|
|
|
else
|
2010-09-23 14:55:58 +02:00
|
|
|
ret = do_setlk(filp, cmd, fl, is_local);
|
2008-05-21 01:34:39 +02:00
|
|
|
out_err:
|
|
|
|
return ret;
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2012-07-30 22:05:25 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_lock);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Lock a (portion of) a file
|
|
|
|
*/
|
2012-07-16 22:39:15 +02:00
|
|
|
int nfs_flock(struct file *filp, int cmd, struct file_lock *fl)
|
2005-04-17 00:20:36 +02:00
|
|
|
{
|
2010-09-23 14:55:58 +02:00
|
|
|
struct inode *inode = filp->f_mapping->host;
|
|
|
|
int is_local = 0;
|
|
|
|
|
2013-09-16 16:53:17 +02:00
|
|
|
dprintk("NFS: flock(%pD2, t=%x, fl=%x)\n",
|
|
|
|
filp, fl->fl_type, fl->fl_flags);
|
2005-04-17 00:20:36 +02:00
|
|
|
|
|
|
|
if (!(fl->fl_flags & FL_FLOCK))
|
|
|
|
return -ENOLCK;
|
|
|
|
|
2012-07-23 21:46:23 +02:00
|
|
|
/*
|
|
|
|
* The NFSv4 protocol doesn't support LOCK_MAND, which is not part of
|
|
|
|
* any standard. In principle we might be able to support LOCK_MAND
|
|
|
|
* on NFSv2/3 since NLMv3/4 support DOS share modes, but for now the
|
|
|
|
* NFS code is not set up for it.
|
|
|
|
*/
|
|
|
|
if (fl->fl_type & LOCK_MAND)
|
|
|
|
return -EINVAL;
|
|
|
|
|
2010-09-23 14:55:58 +02:00
|
|
|
if (NFS_SERVER(inode)->flags & NFS_MOUNT_LOCAL_FLOCK)
|
|
|
|
is_local = 1;
|
|
|
|
|
2017-11-10 12:27:49 +01:00
|
|
|
/* We're simulating flock() locks using posix locks on the server */
|
|
|
|
if (fl->fl_type == F_UNLCK)
|
2010-09-23 14:55:58 +02:00
|
|
|
return do_unlk(filp, cmd, fl, is_local);
|
|
|
|
return do_setlk(filp, cmd, fl, is_local);
|
2005-04-17 00:20:36 +02:00
|
|
|
}
|
2012-07-30 22:05:25 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_flock);
|
2007-06-08 21:23:34 +02:00
|
|
|
|
2011-11-04 18:31:22 +01:00
|
|
|
const struct file_operations nfs_file_operations = {
|
|
|
|
.llseek = nfs_file_llseek,
|
2014-04-03 02:14:12 +02:00
|
|
|
.read_iter = nfs_file_read,
|
2014-04-03 20:07:25 +02:00
|
|
|
.write_iter = nfs_file_write,
|
2011-11-04 18:31:22 +01:00
|
|
|
.mmap = nfs_file_mmap,
|
|
|
|
.open = nfs_file_open,
|
|
|
|
.flush = nfs_file_flush,
|
|
|
|
.release = nfs_file_release,
|
|
|
|
.fsync = nfs_file_fsync,
|
|
|
|
.lock = nfs_lock,
|
|
|
|
.flock = nfs_flock,
|
2016-09-23 05:35:42 +02:00
|
|
|
.splice_read = generic_file_splice_read,
|
2014-04-05 10:37:17 +02:00
|
|
|
.splice_write = iter_file_splice_write,
|
2011-11-04 18:31:22 +01:00
|
|
|
.check_flags = nfs_check_flags,
|
2014-08-27 12:49:41 +02:00
|
|
|
.setlease = simple_nosetlease,
|
2011-11-04 18:31:22 +01:00
|
|
|
};
|
2012-07-30 22:05:23 +02:00
|
|
|
EXPORT_SYMBOL_GPL(nfs_file_operations);
|