From: Jeff Moyer <jmoyer@redhat.com>
To: Milosz Tanski <milosz@adfin.com>
Cc: linux-kernel@vger.kernel.org,
Christoph Hellwig <hch@infradead.org>,
linux-fsdevel@vger.kernel.org, linux-aio@kvack.org,
Mel Gorman <mgorman@suse.de>,
Volker Lendecke <Volker.Lendecke@sernet.de>,
Tejun Heo <tj@kernel.org>, "Theodore Ts'o" <tytso@mit.edu>,
Al Viro <viro@zeniv.linux.org.uk>,
linux-api@vger.kernel.org,
Michael Kerrisk <mtk.manpages@gmail.com>
Subject: Re: [PATCH 4/4] vfs: RWF_NONBLOCK flag for preadv2
Date: Fri, 24 Oct 2014 11:55:23 -0400 [thread overview]
Message-ID: <x49k33po42c.fsf@segfault.boston.devel.redhat.com> (raw)
In-Reply-To: <ba8d704a12c7ceffce610fc438a0af11a3422002.1413923420.git.milosz@adfin.com> (Milosz Tanski's message of "Tue, 21 Oct 2014 16:46:59 -0400")
Milosz Tanski <milosz@adfin.com> writes:
> Filesystems that generic_file_read_iter will not be allowed to perform
> non-blocking reads. This only will read data if it's in the page cache and if
> there is no page error (causing a re-read).
>
> Christoph Hellwig wrote the filesystem specify code (cifs, ofs, shm, xfs).
>
> Signed-off-by: Milosz Tanski <milosz@adfin.com>
Add in the 2 hunks from the other patches, as suggested by Christoph,
and you can add my:
Reviewed-by: Jeff Moyer <jmoyer@redhat.com>
--
To unsubscribe, send a message with 'unsubscribe linux-aio' in
the body to majordomo@kvack.org. For more info on Linux AIO,
see: http://www.kvack.org/aio/
Don't email: <a href=mailto:"aart@kvack.org">aart@kvack.org</a>
next prev parent reply other threads:[~2014-10-24 15:55 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-21 20:46 [PATCH 0/4] vfs: Non-blockling buffered fs read (page cache only) Milosz Tanski
2014-10-21 20:46 ` [PATCH 1/4] vfs: Prepare for adding a new preadv/pwritev with user flags Milosz Tanski
2014-10-24 9:35 ` Christoph Hellwig
2014-10-24 15:12 ` Jeff Moyer
2014-10-24 16:59 ` Milosz Tanski
2014-10-21 20:46 ` [PATCH 2/4] vfs: Define new syscalls preadv2,pwritev2 Milosz Tanski
2014-10-24 9:36 ` Christoph Hellwig
2014-10-24 17:08 ` Milosz Tanski
2014-10-24 15:46 ` Jeff Moyer
2014-11-05 20:32 ` Milosz Tanski
2014-10-21 20:46 ` [PATCH 3/4] vfs: Export new vector IO syscalls (with flags) to userland Milosz Tanski
[not found] ` <e43b43c1e7b14b454b6425a650cd45cbb75fd325.1413923420.git.milosz-B5zB6C1i6pkAvxtiuMwx3w@public.gmane.org>
2014-10-24 9:38 ` Christoph Hellwig
2014-10-21 20:46 ` [PATCH 4/4] vfs: RWF_NONBLOCK flag for preadv2 Milosz Tanski
[not found] ` <ba8d704a12c7ceffce610fc438a0af11a3422002.1413923420.git.milosz-B5zB6C1i6pkAvxtiuMwx3w@public.gmane.org>
2014-10-24 9:40 ` Christoph Hellwig
2014-10-24 15:55 ` Jeff Moyer [this message]
2014-10-24 17:16 ` Milosz Tanski
2014-10-27 9:58 ` Christoph Hellwig
2014-10-27 19:03 ` Steve French
2014-10-27 19:09 ` Milosz Tanski
2014-10-21 20:59 ` [PATCH] Add preadv2/pwritev2 documentation Milosz Tanski
2014-10-24 9:41 ` [PATCH 0/4] vfs: Non-blockling buffered fs read (page cache only) Christoph Hellwig
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=x49k33po42c.fsf@segfault.boston.devel.redhat.com \
--to=jmoyer@redhat.com \
--cc=Volker.Lendecke@sernet.de \
--cc=hch@infradead.org \
--cc=linux-aio@kvack.org \
--cc=linux-api@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mgorman@suse.de \
--cc=milosz@adfin.com \
--cc=mtk.manpages@gmail.com \
--cc=tj@kernel.org \
--cc=tytso@mit.edu \
--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).