On Wed, Feb 08, 2023 at 04:09:51PM +0000, David Howells wrote: > @@ -2688,7 +2689,7 @@ ssize_t filemap_read(struct kiocb *iocb, struct iov_iter *iter, > if (unlikely(iocb->ki_pos >= i_size_read(inode))) > break; > > - error = filemap_get_pages(iocb, iter, &fbatch); > + error = filemap_get_pages(iocb, iov_iter_count(iter), &fbatch); What's the point in iov_iter_count() anyway? It's more characters than iter->count, so why not use it directly?