[PATCH 0/2] pipe.2 and pipe.7 fixes

[PATCH 0/2] pipe.2 and pipe.7 fixes

[Elie De Brauwer]

Hello Michael,

Attached are two patches related to the pipe man pages.
 - pipe.2 mentions the use of PIPE_BUF but nowhere is mentioned that it 
should be included via limits.h
 - pipe.7 only talks about static pipe capacity, however proc.5 and fcntl.2
already mention F_SETPIPE_SZ, F_GETPIPE_SZ and /proc/sys/fs/pipe-max-size 
hence I added a reference to fcntl.2.

my 2 cents
E.

Elie De Brauwer (2):
  pipe.2: PIPE_BUF is defined in limits.h
  pipe.7: Add reference that the pipe capacity can be changed.

 man2/pipe.2 |  4 ++++
 man7/pipe.7 | 10 ++++++++++
 2 files changed, 14 insertions(+)

-- 
2.0.0

--
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

[PATCH 1/2] pipe.2: PIPE_BUF is defined in limits.h

[Elie De Brauwer]

To make use of PIPE_BUF in an application one should include limits.h.
Adding a reference to this inclusion.

Signed-off-by: Elie De Brauwer <eliedebrauwer-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 man2/pipe.2 | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/man2/pipe.2 b/man2/pipe.2
index a5ed2ea..7f1a1d0 100644
--- a/man2/pipe.2
+++ b/man2/pipe.2
@@ -98,6 +98,10 @@ Writes of greater than
 bytes (see
 .BR pipe (7))
 will be split into multiple packets.
+The constant
+.BR PIPE_BUF
+is defined in
+.IR <limits.h> .
 .IP *
 If a
 .BR read (2)
-- 
2.0.0

--
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

[PATCH 2/2] pipe.7: Add reference that the pipe capacity can be changed.

[Elie De Brauwer]

In fcntl(2) F_SETPIPE_SZ, F_GETPIPE_SZ and /proc/sys/fs/pipe-max-size are
defined, however pipe(7) still defines the pipe capacity as being a static
entity. Adding a reference to fcntl(2).

Signed-off-by: Elie De Brauwer <eliedebrauwer-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 man7/pipe.7 | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/man7/pipe.7 b/man7/pipe.7
index 78d1cea..64a6761 100644
--- a/man7/pipe.7
+++ b/man7/pipe.7
@@ -133,6 +133,16 @@ so that a writing process does not remain blocked.
 In Linux versions before 2.6.11, the capacity of a pipe was the same as
 the system page size (e.g., 4096 bytes on i386).
 Since Linux 2.6.11, the pipe capacity is 65536 bytes.
+Since Linux 2.6.35, the pipe capacity can be queried and set using the
+.BR fcntl (2)
+.BR F_GETPIPE_SZ
+and
+.BR F_SETPIPE_SZ
+operations.
+See
+.BR fcntl (2)
+for more information.
+
 .SS PIPE_BUF
 POSIX.1-2001 says that
 .BR write (2)s
-- 
2.0.0

--
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

Re: [PATCH 1/2] pipe.2: PIPE_BUF is defined in limits.h

[Michael Kerrisk (man-pages)]

On 07/06/2014 03:44 PM, Elie De Brauwer wrote:
> To make use of PIPE_BUF in an application one should include limits.h.
> Adding a reference to this inclusion.

Thanks, Elie. Applied.

Cheers,

Michael


> Signed-off-by: Elie De Brauwer <eliedebrauwer-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  man2/pipe.2 | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/man2/pipe.2 b/man2/pipe.2
> index a5ed2ea..7f1a1d0 100644
> --- a/man2/pipe.2
> +++ b/man2/pipe.2
> @@ -98,6 +98,10 @@ Writes of greater than
>  bytes (see
>  .BR pipe (7))
>  will be split into multiple packets.
> +The constant
> +.BR PIPE_BUF
> +is defined in
> +.IR <limits.h> .
>  .IP *
>  If a
>  .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/
--
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

Re: [PATCH 2/2] pipe.7: Add reference that the pipe capacity can be changed.

[Michael Kerrisk (man-pages)]

On 07/06/2014 03:44 PM, Elie De Brauwer wrote:
> In fcntl(2) F_SETPIPE_SZ, F_GETPIPE_SZ and /proc/sys/fs/pipe-max-size are
> defined, however pipe(7) still defines the pipe capacity as being a static
> entity. Adding a reference to fcntl(2).

Hi Elie,

Thanks for that! Applied.

Cheers,

Michael

> Signed-off-by: Elie De Brauwer <eliedebrauwer-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  man7/pipe.7 | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/man7/pipe.7 b/man7/pipe.7
> index 78d1cea..64a6761 100644
> --- a/man7/pipe.7
> +++ b/man7/pipe.7
> @@ -133,6 +133,16 @@ so that a writing process does not remain blocked.
>  In Linux versions before 2.6.11, the capacity of a pipe was the same as
>  the system page size (e.g., 4096 bytes on i386).
>  Since Linux 2.6.11, the pipe capacity is 65536 bytes.
> +Since Linux 2.6.35, the pipe capacity can be queried and set using the
> +.BR fcntl (2)
> +.BR F_GETPIPE_SZ
> +and
> +.BR F_SETPIPE_SZ
> +operations.
> +See
> +.BR fcntl (2)
> +for more information.
> +
>  .SS PIPE_BUF
>  POSIX.1-2001 says that
>  .BR write (2)s
> 


-- 
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

Re: [PATCH 2/2] pipe.7: Add reference that the pipe capacity can be changed.

[walter harms]

Am 06.07.2014 15:44, schrieb Elie De Brauwer:
> In fcntl(2) F_SETPIPE_SZ, F_GETPIPE_SZ and /proc/sys/fs/pipe-max-size are
> defined, however pipe(7) still defines the pipe capacity as being a static
> entity. Adding a reference to fcntl(2).
> 
> Signed-off-by: Elie De Brauwer <eliedebrauwer-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  man7/pipe.7 | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/man7/pipe.7 b/man7/pipe.7
> index 78d1cea..64a6761 100644
> --- a/man7/pipe.7
> +++ b/man7/pipe.7
> @@ -133,6 +133,16 @@ so that a writing process does not remain blocked.
>  In Linux versions before 2.6.11, the capacity of a pipe was the same as
>  the system page size (e.g., 4096 bytes on i386).
>  Since Linux 2.6.11, the pipe capacity is 65536 bytes.
> +Since Linux 2.6.35, the pipe capacity can be queried and set using the
> +.BR fcntl (2)
> +.BR F_GETPIPE_SZ
> +and
> +.BR F_SETPIPE_SZ
> +operations.
> +See
> +.BR fcntl (2)
> +for more information.
> +

Could to add a line like:
"the default pipe capacity is still 65536 bytes." ? (or what ever)

re,
 wh

>  .SS PIPE_BUF
>  POSIX.1-2001 says that
>  .BR write (2)s
--
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

Re: [PATCH 2/2] pipe.7: Add reference that the pipe capacity can be changed.

[Michael Kerrisk (man-pages)]

On 07/07/2014 01:55 PM, walter harms wrote:
> 
> 
> Am 06.07.2014 15:44, schrieb Elie De Brauwer:
>> In fcntl(2) F_SETPIPE_SZ, F_GETPIPE_SZ and /proc/sys/fs/pipe-max-size are
>> defined, however pipe(7) still defines the pipe capacity as being a static
>> entity. Adding a reference to fcntl(2).
>>
>> Signed-off-by: Elie De Brauwer <eliedebrauwer-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>> ---
>>  man7/pipe.7 | 10 ++++++++++
>>  1 file changed, 10 insertions(+)
>>
>> diff --git a/man7/pipe.7 b/man7/pipe.7
>> index 78d1cea..64a6761 100644
>> --- a/man7/pipe.7
>> +++ b/man7/pipe.7
>> @@ -133,6 +133,16 @@ so that a writing process does not remain blocked.
>>  In Linux versions before 2.6.11, the capacity of a pipe was the same as
>>  the system page size (e.g., 4096 bytes on i386).
>>  Since Linux 2.6.11, the pipe capacity is 65536 bytes.
>> +Since Linux 2.6.35, the pipe capacity can be queried and set using the
>> +.BR fcntl (2)
>> +.BR F_GETPIPE_SZ
>> +and
>> +.BR F_SETPIPE_SZ
>> +operations.
>> +See
>> +.BR fcntl (2)
>> +for more information.
>> +
> 
> Could to add a line like:
> "the default pipe capacity is still 65536 bytes." ? (or what ever)

Yup, seems worthwhile. Added. Thanks, Walter.

Cheers,

Michael
--
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