From: Jeff Moyer <jmoyer@redhat.com>
To: Christoph Hellwig <hch@lst.de>
Cc: viro@zeniv.linux.org.uk, axboe@fb.com, milosz@adfin.com,
linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org,
linux-api@vger.kernel.org
Subject: Re: [PATCH 1/7] vfs: pass a flags argument to vfs_readv/vfs_writev
Date: Fri, 26 Feb 2016 16:52:40 -0500 [thread overview]
Message-ID: <x49y4a7kul3.fsf@segfault.boston.devel.redhat.com> (raw)
In-Reply-To: <1456160876-14560-2-git-send-email-hch@lst.de> (Christoph Hellwig's message of "Mon, 22 Feb 2016 18:07:50 +0100")
Christoph Hellwig <hch@lst.de> writes:
> This way we can set kiocb flags also from the sync read/write path.
>
> Signed-off-by: Milosz Tanski <milosz@adfin.com>
> [hch: rebased on top of my kiocb changes]
> Signed-off-by: Christoph Hellwig <hch@lst.de>
Looks sane to me.
Acked-by: Jeff Moyer <jmoyer@redhat.com>
> ---
> fs/nfsd/vfs.c | 4 ++--
> fs/read_write.c | 44 ++++++++++++++++++++++++++------------------
> fs/splice.c | 2 +-
> include/linux/fs.h | 4 ++--
> 4 files changed, 31 insertions(+), 23 deletions(-)
>
> diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
> index 5d2a57e..d40010e 100644
> --- a/fs/nfsd/vfs.c
> +++ b/fs/nfsd/vfs.c
> @@ -870,7 +870,7 @@ __be32 nfsd_readv(struct file *file, loff_t offset, struct kvec *vec, int vlen,
>
> oldfs = get_fs();
> set_fs(KERNEL_DS);
> - host_err = vfs_readv(file, (struct iovec __user *)vec, vlen, &offset);
> + host_err = vfs_readv(file, (struct iovec __user *)vec, vlen, &offset, 0);
> set_fs(oldfs);
> return nfsd_finish_read(file, count, host_err);
> }
> @@ -957,7 +957,7 @@ nfsd_vfs_write(struct svc_rqst *rqstp, struct svc_fh *fhp, struct file *file,
>
> /* Write the data. */
> oldfs = get_fs(); set_fs(KERNEL_DS);
> - host_err = vfs_writev(file, (struct iovec __user *)vec, vlen, &pos);
> + host_err = vfs_writev(file, (struct iovec __user *)vec, vlen, &pos, 0);
> set_fs(oldfs);
> if (host_err < 0)
> goto out_nfserr;
> diff --git a/fs/read_write.c b/fs/read_write.c
> index 324ec27..7d453c3 100644
> --- a/fs/read_write.c
> +++ b/fs/read_write.c
> @@ -692,11 +692,14 @@ unsigned long iov_shorten(struct iovec *iov, unsigned long nr_segs, size_t to)
> EXPORT_SYMBOL(iov_shorten);
>
> static ssize_t do_iter_readv_writev(struct file *filp, struct iov_iter *iter,
> - loff_t *ppos, iter_fn_t fn)
> + loff_t *ppos, iter_fn_t fn, int flags)
> {
> struct kiocb kiocb;
> ssize_t ret;
>
> + if (flags)
> + return -EOPNOTSUPP;
> +
> init_sync_kiocb(&kiocb, filp);
> kiocb.ki_pos = *ppos;
>
> @@ -708,10 +711,13 @@ static ssize_t do_iter_readv_writev(struct file *filp, struct iov_iter *iter,
>
> /* Do it by hand, with file-ops */
> static ssize_t do_loop_readv_writev(struct file *filp, struct iov_iter *iter,
> - loff_t *ppos, io_fn_t fn)
> + loff_t *ppos, io_fn_t fn, int flags)
> {
> ssize_t ret = 0;
>
> + if (flags)
> + return -EOPNOTSUPP;
> +
> while (iov_iter_count(iter)) {
> struct iovec iovec = iov_iter_iovec(iter);
> ssize_t nr;
> @@ -812,7 +818,8 @@ out:
>
> static ssize_t do_readv_writev(int type, struct file *file,
> const struct iovec __user * uvector,
> - unsigned long nr_segs, loff_t *pos)
> + unsigned long nr_segs, loff_t *pos,
> + int flags)
> {
> size_t tot_len;
> struct iovec iovstack[UIO_FASTIOV];
> @@ -844,9 +851,9 @@ static ssize_t do_readv_writev(int type, struct file *file,
> }
>
> if (iter_fn)
> - ret = do_iter_readv_writev(file, &iter, pos, iter_fn);
> + ret = do_iter_readv_writev(file, &iter, pos, iter_fn, flags);
> else
> - ret = do_loop_readv_writev(file, &iter, pos, fn);
> + ret = do_loop_readv_writev(file, &iter, pos, fn, flags);
>
> if (type != READ)
> file_end_write(file);
> @@ -863,27 +870,27 @@ out:
> }
>
> ssize_t vfs_readv(struct file *file, const struct iovec __user *vec,
> - unsigned long vlen, loff_t *pos)
> + unsigned long vlen, loff_t *pos, int flags)
> {
> if (!(file->f_mode & FMODE_READ))
> return -EBADF;
> if (!(file->f_mode & FMODE_CAN_READ))
> return -EINVAL;
>
> - return do_readv_writev(READ, file, vec, vlen, pos);
> + return do_readv_writev(READ, file, vec, vlen, pos, flags);
> }
>
> EXPORT_SYMBOL(vfs_readv);
>
> ssize_t vfs_writev(struct file *file, const struct iovec __user *vec,
> - unsigned long vlen, loff_t *pos)
> + unsigned long vlen, loff_t *pos, int flags)
> {
> if (!(file->f_mode & FMODE_WRITE))
> return -EBADF;
> if (!(file->f_mode & FMODE_CAN_WRITE))
> return -EINVAL;
>
> - return do_readv_writev(WRITE, file, vec, vlen, pos);
> + return do_readv_writev(WRITE, file, vec, vlen, pos, flags);
> }
>
> EXPORT_SYMBOL(vfs_writev);
> @@ -896,7 +903,7 @@ SYSCALL_DEFINE3(readv, unsigned long, fd, const struct iovec __user *, vec,
>
> if (f.file) {
> loff_t pos = file_pos_read(f.file);
> - ret = vfs_readv(f.file, vec, vlen, &pos);
> + ret = vfs_readv(f.file, vec, vlen, &pos, 0);
> if (ret >= 0)
> file_pos_write(f.file, pos);
> fdput_pos(f);
> @@ -916,7 +923,7 @@ SYSCALL_DEFINE3(writev, unsigned long, fd, const struct iovec __user *, vec,
>
> if (f.file) {
> loff_t pos = file_pos_read(f.file);
> - ret = vfs_writev(f.file, vec, vlen, &pos);
> + ret = vfs_writev(f.file, vec, vlen, &pos, 0);
> if (ret >= 0)
> file_pos_write(f.file, pos);
> fdput_pos(f);
> @@ -948,7 +955,7 @@ SYSCALL_DEFINE5(preadv, unsigned long, fd, const struct iovec __user *, vec,
> if (f.file) {
> ret = -ESPIPE;
> if (f.file->f_mode & FMODE_PREAD)
> - ret = vfs_readv(f.file, vec, vlen, &pos);
> + ret = vfs_readv(f.file, vec, vlen, &pos, 0);
> fdput(f);
> }
>
> @@ -972,7 +979,7 @@ SYSCALL_DEFINE5(pwritev, unsigned long, fd, const struct iovec __user *, vec,
> if (f.file) {
> ret = -ESPIPE;
> if (f.file->f_mode & FMODE_PWRITE)
> - ret = vfs_writev(f.file, vec, vlen, &pos);
> + ret = vfs_writev(f.file, vec, vlen, &pos, 0);
> fdput(f);
> }
>
> @@ -986,7 +993,8 @@ SYSCALL_DEFINE5(pwritev, unsigned long, fd, const struct iovec __user *, vec,
>
> static ssize_t compat_do_readv_writev(int type, struct file *file,
> const struct compat_iovec __user *uvector,
> - unsigned long nr_segs, loff_t *pos)
> + unsigned long nr_segs, loff_t *pos,
> + int flags)
> {
> compat_ssize_t tot_len;
> struct iovec iovstack[UIO_FASTIOV];
> @@ -1018,9 +1026,9 @@ static ssize_t compat_do_readv_writev(int type, struct file *file,
> }
>
> if (iter_fn)
> - ret = do_iter_readv_writev(file, &iter, pos, iter_fn);
> + ret = do_iter_readv_writev(file, &iter, pos, iter_fn, flags);
> else
> - ret = do_loop_readv_writev(file, &iter, pos, fn);
> + ret = do_loop_readv_writev(file, &iter, pos, fn, flags);
>
> if (type != READ)
> file_end_write(file);
> @@ -1049,7 +1057,7 @@ static size_t compat_readv(struct file *file,
> if (!(file->f_mode & FMODE_CAN_READ))
> goto out;
>
> - ret = compat_do_readv_writev(READ, file, vec, vlen, pos);
> + ret = compat_do_readv_writev(READ, file, vec, vlen, pos, 0);
>
> out:
> if (ret > 0)
> @@ -1126,7 +1134,7 @@ static size_t compat_writev(struct file *file,
> if (!(file->f_mode & FMODE_CAN_WRITE))
> goto out;
>
> - ret = compat_do_readv_writev(WRITE, file, vec, vlen, pos);
> + ret = compat_do_readv_writev(WRITE, file, vec, vlen, pos, 0);
>
> out:
> if (ret > 0)
> diff --git a/fs/splice.c b/fs/splice.c
> index 82bc0d6..3dc1426 100644
> --- a/fs/splice.c
> +++ b/fs/splice.c
> @@ -577,7 +577,7 @@ static ssize_t kernel_readv(struct file *file, const struct iovec *vec,
> old_fs = get_fs();
> set_fs(get_ds());
> /* The cast to a user pointer is valid due to the set_fs() */
> - res = vfs_readv(file, (const struct iovec __user *)vec, vlen, &pos);
> + res = vfs_readv(file, (const struct iovec __user *)vec, vlen, &pos, 0);
> set_fs(old_fs);
>
> return res;
> diff --git a/include/linux/fs.h b/include/linux/fs.h
> index ae68100..875277a 100644
> --- a/include/linux/fs.h
> +++ b/include/linux/fs.h
> @@ -1709,9 +1709,9 @@ extern ssize_t __vfs_write(struct file *, const char __user *, size_t, loff_t *)
> extern ssize_t vfs_read(struct file *, char __user *, size_t, loff_t *);
> extern ssize_t vfs_write(struct file *, const char __user *, size_t, loff_t *);
> extern ssize_t vfs_readv(struct file *, const struct iovec __user *,
> - unsigned long, loff_t *);
> + unsigned long, loff_t *, int);
> extern ssize_t vfs_writev(struct file *, const struct iovec __user *,
> - unsigned long, loff_t *);
> + unsigned long, loff_t *, int);
> extern ssize_t vfs_copy_file_range(struct file *, loff_t , struct file *,
> loff_t, size_t, unsigned int);
> extern int vfs_clone_file_range(struct file *file_in, loff_t pos_in,
next prev parent reply other threads:[~2016-02-26 21:52 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-22 17:07 selective block polling and preadv2/pwritev2 revisited V2 Christoph Hellwig
2016-02-22 17:07 ` [PATCH 1/7] vfs: pass a flags argument to vfs_readv/vfs_writev Christoph Hellwig
2016-02-26 21:52 ` Jeff Moyer [this message]
2016-02-22 17:07 ` [PATCH 2/7] vfs: vfs: Define new syscalls preadv2,pwritev2 Christoph Hellwig
2016-02-26 21:51 ` Jeff Moyer
2016-02-27 8:58 ` Christoph Hellwig
2016-02-22 17:07 ` [PATCH 3/7] x86: wire up preadv2 and pwritev2 Christoph Hellwig
2016-02-22 17:07 ` [PATCH 4/7] vfs: add the RWF_HIPRI flag for preadv2/pwritev2 Christoph Hellwig
2016-02-26 21:56 ` Jeff Moyer
2016-02-27 8:58 ` Christoph Hellwig
2016-02-22 17:07 ` [PATCH 5/7] direct-io: only use block polling if explicitly requested Christoph Hellwig
2016-02-26 21:58 ` Jeff Moyer
2016-02-22 17:07 ` [PATCH 6/7] blk-mq: enable polling support by default Christoph Hellwig
2016-02-26 20:44 ` Jeff Moyer
2016-02-27 8:56 ` Christoph Hellwig
2016-02-29 14:27 ` Jeff Moyer
2016-02-22 17:07 ` [PATCH 7/7] block, directio: set a REQ_POLL flag when submitting polled bios Christoph Hellwig
2016-02-26 21:10 ` Jeff Moyer
2016-02-27 8:57 ` Christoph Hellwig
2016-02-29 14:28 ` Jeff Moyer
2016-02-26 15:06 ` selective block polling and preadv2/pwritev2 revisited V2 Stephen Bates
2016-02-26 21:18 ` Jeff Moyer
2016-02-27 8:57 ` Christoph Hellwig
2016-02-29 1:30 ` Damien Le Moal
2016-02-29 14:59 ` Jeff Moyer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=x49y4a7kul3.fsf@segfault.boston.devel.redhat.com \
--to=jmoyer@redhat.com \
--cc=axboe@fb.com \
--cc=hch@lst.de \
--cc=linux-api@vger.kernel.org \
--cc=linux-block@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=milosz@adfin.com \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).