From c44939ecb6e05aeaaf12d4e1bb046719c97e457e Mon Sep 17 00:00:00 2001 From: "akpm@linux-foundation.org" Date: Sun, 15 Jul 2007 23:38:25 -0700 Subject: NeilBrown The do_loop_readv_writev implementation of readv breaks out of the loop as soon as a single read request didn't fill it's buffer: if (nr != len) break; The generic_file_aio_read version doesn't. So if it hits EOF before the end of the list of buffers, it will try again on the next buffer. If the file was extended in the mean time, this will produce a bad result. Signed-off-by: Neil Brown Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- mm/filemap.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'mm') diff --git a/mm/filemap.c b/mm/filemap.c index e006c57bda5..adbac104f34 100644 --- a/mm/filemap.c +++ b/mm/filemap.c @@ -1219,6 +1219,8 @@ generic_file_aio_read(struct kiocb *iocb, const struct iovec *iov, retval = retval ?: desc.error; break; } + if (desc.count > 0) + break; } } out: -- cgit v1.2.3