[PATCH] sigprocmask.2: do not specify an exact value of rt_sigprocmask's 4th argument

[PATCH] sigprocmask.2: do not specify an exact value of rt_sigprocmask's 4th argument

[Dmitry V. Levin]

As sizeof(kernel_sigset_t) is not the same for all architectures,
it would be better not to mention any numbers as its value.

Signed-off-by: Dmitry V. Levin <ldv-u2l5PoMzF/Vg9hUCZPvPmw@public.gmane.org>
---
 man2/sigprocmask.2 | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/man2/sigprocmask.2 b/man2/sigprocmask.2
index 8388340..6379a4d 100644
--- a/man2/sigprocmask.2
+++ b/man2/sigprocmask.2
@@ -202,10 +202,11 @@ which specifies the size in bytes of the signal sets in
 .IR set
 and
 .IR oldset .
-This argument is currently required to have the value 8
-(
-.IR sizeof(kernel_sigset_t)
-).
+This argument is currently required to have a fixed value
+(equal to
+.IR sizeof(kernel_sigset_t) ).
+.\" sizeof(kernel_sigset_t) == _NSIG / 8,
+.\" which equals to 8 on most architectures, but e.g. on MIPS it's 16.
 
 The glibc
 .BR sigprocmask ()
-- 
ldv

--
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] sigprocmask.2: do not specify an exact value of rt_sigprocmask's 4th argument

[walter harms]

Am 19.02.2017 22:13, schrieb Dmitry V. Levin:
> As sizeof(kernel_sigset_t) is not the same for all architectures,
> it would be better not to mention any numbers as its value.
> 
> Signed-off-by: Dmitry V. Levin <ldv-u2l5PoMzF/Vg9hUCZPvPmw@public.gmane.org>
> ---
>  man2/sigprocmask.2 | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/man2/sigprocmask.2 b/man2/sigprocmask.2
> index 8388340..6379a4d 100644
> --- a/man2/sigprocmask.2
> +++ b/man2/sigprocmask.2
> @@ -202,10 +202,11 @@ which specifies the size in bytes of the signal sets in
>  .IR set
>  and
>  .IR oldset .
> -This argument is currently required to have the value 8
> -(
> -.IR sizeof(kernel_sigset_t)
> -).
> +This argument is currently required to have a fixed value
> +(equal to
> +.IR sizeof(kernel_sigset_t) ).
> +.\" sizeof(kernel_sigset_t) == _NSIG / 8,
> +.\" which equals to 8 on most architectures, but e.g. on MIPS it's 16.
>  
>  The glibc
>  .BR sigprocmask ()


why not add this information like:
Note: sizeof(kernel_sigset_t) is not the same for all architectures

re,
 wh
--
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 v2] sigprocmask.2: do not specify an exact value of rt_sigprocmask's 4th argument

[Dmitry V. Levin]

As sizeof(kernel_sigset_t) is not the same for all architectures,
it would be better not to mention any numbers as its value.

Signed-off-by: Dmitry V. Levin <ldv-u2l5PoMzF/Vg9hUCZPvPmw@public.gmane.org>
---
v2: mention that sizeof(kernel_sigset_t) is architecture specific.

 man2/sigprocmask.2 | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/man2/sigprocmask.2 b/man2/sigprocmask.2
index 8388340..499fd0c 100644
--- a/man2/sigprocmask.2
+++ b/man2/sigprocmask.2
@@ -202,10 +202,11 @@ which specifies the size in bytes of the signal sets in
 .IR set
 and
 .IR oldset .
-This argument is currently required to have the value 8
-(
-.IR sizeof(kernel_sigset_t)
-).
+This argument is currently required to have a fixed architecture specific value
+(equal to
+.IR sizeof(kernel_sigset_t) ).
+.\" sizeof(kernel_sigset_t) == _NSIG / 8,
+.\" which equals to 8 on most architectures, but e.g. on MIPS it's 16.
 
 The glibc
 .BR sigprocmask ()
-- 
ldv

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

[PING] [PATCH v2] sigprocmask.2: do not specify an exact value of rt_sigprocmask's 4th argument

[Dmitry V. Levin]

As sizeof(kernel_sigset_t) is not the same for all architectures,
it would be better not to mention any numbers as its value.

Signed-off-by: Dmitry V. Levin <ldv-u2l5PoMzF/Vg9hUCZPvPmw@public.gmane.org>
---
v2: mention that sizeof(kernel_sigset_t) is architecture specific.

 man2/sigprocmask.2 | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/man2/sigprocmask.2 b/man2/sigprocmask.2
index 8388340..499fd0c 100644
--- a/man2/sigprocmask.2
+++ b/man2/sigprocmask.2
@@ -202,10 +202,11 @@ which specifies the size in bytes of the signal sets in
 .IR set
 and
 .IR oldset .
-This argument is currently required to have the value 8
-(
-.IR sizeof(kernel_sigset_t)
-).
+This argument is currently required to have a fixed architecture specific value
+(equal to
+.IR sizeof(kernel_sigset_t) ).
+.\" sizeof(kernel_sigset_t) == _NSIG / 8,
+.\" which equals to 8 on most architectures, but e.g. on MIPS it's 16.
 
 The glibc
 .BR sigprocmask ()
-- 
ldv
--
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 v2] sigprocmask.2: do not specify an exact value of rt_sigprocmask's 4th argument

[Michael Kerrisk (man-pages)]

Hello Dmitry

On 02/20/2017 04:43 PM, Dmitry V. Levin wrote:
> As sizeof(kernel_sigset_t) is not the same for all architectures,
> it would be better not to mention any numbers as its value.

Thanks! Applied.

Cheers,

Michael


> Signed-off-by: Dmitry V. Levin <ldv-u2l5PoMzF/Vg9hUCZPvPmw@public.gmane.org>
> ---
> v2: mention that sizeof(kernel_sigset_t) is architecture specific.
> 
>  man2/sigprocmask.2 | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/man2/sigprocmask.2 b/man2/sigprocmask.2
> index 8388340..499fd0c 100644
> --- a/man2/sigprocmask.2
> +++ b/man2/sigprocmask.2
> @@ -202,10 +202,11 @@ which specifies the size in bytes of the signal sets in
>  .IR set
>  and
>  .IR oldset .
> -This argument is currently required to have the value 8
> -(
> -.IR sizeof(kernel_sigset_t)
> -).
> +This argument is currently required to have a fixed architecture specific value
> +(equal to
> +.IR sizeof(kernel_sigset_t) ).
> +.\" sizeof(kernel_sigset_t) == _NSIG / 8,
> +.\" which equals to 8 on most architectures, but e.g. on MIPS it's 16.
>  
>  The glibc
>  .BR sigprocmask ()
> 


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