Thread (92 messages) 92 messages, 7 authors, 2021-08-11

Re: [PATCH 02/32] readv.2: Minor tweaks to Will's patch

From: Michael Kerrisk (man-pages) <hidden>
Date: 2021-08-08 02:42:59
Subsystem: the rest · Maintainer: Linus Torvalds

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:
quoted
Cc: Will Manley <redacted>
Signed-off-by: Alejandro Colomar <redacted>
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 [off-list ref]
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 [off-list ref]
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@www.fastmail.com\:/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@www.fastmail.com/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/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help