Hi Alex and Will
Will, thanks for taking time to write up the patch!
On 8/8/21 4:30 AM, Michael Kerrisk (man-pages) wrote:
> Hello Alex,
>
> On 7/28/21 10:19 PM, Alejandro Colomar wrote:
>> Cc: Will Manley <will@xxxxxxxxxxxxxxxxx>
>> Signed-off-by: Alejandro Colomar <alx.manpages@xxxxxxxxx>
>
> Patch applied, but see my follow-up mail.
Alex, see the follow-up patch that I applied, below.
Cheers,
Michael
commit 76dec7bbd4e8761249a339db5f13f7e63a37aac3 (HEAD -> master)
Author: Michael Kerrisk <mtk.manpages@xxxxxxxxx>
Date: Sun Aug 8 04:34:37 2021 +0200
readv.2: Minor fixes (part 2) to Will Manley's patch
Mainly: I generally don't want us to be including URLs to mailing
list discussions in a manual page. Either, the issue in the
discussion is worth writing up in the manual page (so that
the reader doesn't have to look elsewhere), or the details
are less important, in which case it is sufficient to note the
existence of the bug. I think this is an example of the latter.
Signed-off-by: Michael Kerrisk <mtk.manpages@xxxxxxxxx>
diff --git a/man2/readv.2 b/man2/readv.2
index 0debc2d57..bd0a6cd2f 100644
--- a/man2/readv.2
+++ b/man2/readv.2
@@ -428,12 +428,17 @@ nwritten = writev(STDOUT_FILENO, iov, 2);
.EE
.in
.SH BUGS
-Linux v5.9 and v5.10 have a bug where
+Linux 5.9 and 5.10 have a bug where
.BR preadv2()
with the
.BR RWF_NOWAIT
-flag may return 0 even when not at end of file. See
-https://lore.kernel.org\:/linux-fsdevel\:/fea8b16d-5a69-40f9-b123-e84dcd6e8f2e@xxxxxxxxxxxxxxxx\:/T\:/#u
+flag may return 0 even when not at end of file.
+.\" See
+.\" <https://lore.kernel.org/linux-fsdevel/fea8b16d-5a69-40f9-b123-e84dcd6e8f2e@xxxxxxxxxxxxxxxx/T/#u>
+.\" The bug was introduced in
+.\" efa8480a831 fs: RWF_NOWAIT should imply IOCB_NOIO
+.\"and fixed in
+.\" 06c0444290 mm/filemap.c: generic_file_buffered_read() now uses find_get_pages_contig
.SH SEE ALSO
.BR pread (2),
.BR read (2),
--
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
