* Re: [patch] man2/fcntl.2: F_SETPIPE_SZ returns pipe size, not 0. [not found] <CACV4sJ8mYEnozgDOmN7VODFrvw_51EYDgw2OxsPRq4s5fSh2Cg@mail.gmail.com> @ 2014-07-01 9:43 ` Michael Kerrisk (man-pages) 0 siblings, 0 replies; 2+ messages in thread From: Michael Kerrisk (man-pages) @ 2014-07-01 9:43 UTC (permalink / raw) To: Tomi Salminen Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-man-u79uwXL29TY76Z2rM5mHXA On 07/01/2014 11:05 AM, Tomi Salminen wrote: > Hello. > > I was using F_SETPIPE_SZ and noticed that fcntl was returning the > pipe size set to it and not zero on success. I checked the kernel > source, which seemed to return nr_pages * PAGE_SIZE; I made a patch > to update the man-page of fcntl, since it specified only F_GETPIPE_SZ > as returning the pipe size and others zero. > > Attaching patch made on top of commit > f070e9543a5a7cc600229ee6a3c725804c90de91 Thanks for spotting that, Tomi. I applied a more extensive patch, below. Cheers, Michael --- a/man2/fcntl.2 +++ b/man2/fcntl.2 @@ -1257,9 +1257,7 @@ When allocating the buffer for the pipe, the kernel may use a capacity larger than .IR arg , if that is convenient for the implementation. -The -.B F_GETPIPE_SZ -operation returns the actual size used. +The actual capacity that is set is returned as the function result. Attempting to set the pipe capacity smaller than the amount of buffer space currently used to store data produces the error .BR EBUSY . @@ -1291,7 +1289,7 @@ for traditional .B SIGIO behavior. .TP -.B F_GETPIPE_SZ +.BR F_GETPIPE_SZ ", " F_SETPIPE_SZ The pipe capacity. .TP All other commands -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/ -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [patch] man2/fcntl.2: F_SETPIPE_SZ returns pipe size, not 0. @ 2014-07-01 9:43 ` Michael Kerrisk (man-pages) 0 siblings, 0 replies; 2+ messages in thread From: Michael Kerrisk (man-pages) @ 2014-07-01 9:43 UTC (permalink / raw) To: Tomi Salminen; +Cc: mtk.manpages, linux-kernel, linux-man On 07/01/2014 11:05 AM, Tomi Salminen wrote: > Hello. > > I was using F_SETPIPE_SZ and noticed that fcntl was returning the > pipe size set to it and not zero on success. I checked the kernel > source, which seemed to return nr_pages * PAGE_SIZE; I made a patch > to update the man-page of fcntl, since it specified only F_GETPIPE_SZ > as returning the pipe size and others zero. > > Attaching patch made on top of commit > f070e9543a5a7cc600229ee6a3c725804c90de91 Thanks for spotting that, Tomi. I applied a more extensive patch, below. Cheers, Michael --- a/man2/fcntl.2 +++ b/man2/fcntl.2 @@ -1257,9 +1257,7 @@ When allocating the buffer for the pipe, the kernel may use a capacity larger than .IR arg , if that is convenient for the implementation. -The -.B F_GETPIPE_SZ -operation returns the actual size used. +The actual capacity that is set is returned as the function result. Attempting to set the pipe capacity smaller than the amount of buffer space currently used to store data produces the error .BR EBUSY . @@ -1291,7 +1289,7 @@ for traditional .B SIGIO behavior. .TP -.B F_GETPIPE_SZ +.BR F_GETPIPE_SZ ", " F_SETPIPE_SZ The pipe capacity. .TP All other commands -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/ ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-07-05 6:14 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CACV4sJ8mYEnozgDOmN7VODFrvw_51EYDgw2OxsPRq4s5fSh2Cg@mail.gmail.com>
[not found] ` <CACV4sJ8mYEnozgDOmN7VODFrvw_51EYDgw2OxsPRq4s5fSh2Cg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-07-01 9:43 ` [patch] man2/fcntl.2: F_SETPIPE_SZ returns pipe size, not 0 Michael Kerrisk (man-pages)
2014-07-01 9:43 ` Michael Kerrisk (man-pages)
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.