[PATCH] clone.2: tfix

[PATCH] clone.2: tfix

[Jakub Wilk]

Remove duplicated word.

Signed-off-by: Jakub Wilk <jwilk@jwilk.net>
---
 man2/clone.2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man2/clone.2 b/man2/clone.2
index 02422b82f..db1ab586a 100644
--- a/man2/clone.2
+++ b/man2/clone.2
@@ -78,7 +78,7 @@ context are shared between the calling process and the child process.
 For example, using these system calls, the caller can control whether
 or not the two processes share the virtual address space,
 the table of file descriptors, and the table of signal handlers.
-These system system calls also allow the new child process to placed
+These system calls also allow the new child process to placed
 in separate
 .BR namespaces (7).
 .PP
-- 
2.24.0

Re: [PATCH] clone.2: tfix

[Michael Kerrisk (man-pages)]

On 11/9/19 9:50 AM, Jakub Wilk wrote:
> Remove duplicated word.

Um -- wow. That was quick! Patch applied.

Thanks,

Michael

> Signed-off-by: Jakub Wilk <jwilk@jwilk.net>
> ---
>  man2/clone.2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man2/clone.2 b/man2/clone.2
> index 02422b82f..db1ab586a 100644
> --- a/man2/clone.2
> +++ b/man2/clone.2
> @@ -78,7 +78,7 @@ context are shared between the calling process and the child process.
>  For example, using these system calls, the caller can control whether
>  or not the two processes share the virtual address space,
>  the table of file descriptors, and the table of signal handlers.
> -These system system calls also allow the new child process to placed
> +These system calls also allow the new child process to placed
>  in separate
>  .BR namespaces (7).
>  .PP
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

Re: [PATCH] clone.2: tfix

[Michael Kerrisk (man-pages)]

Hi Alex,

On 5/10/21 9:39 AM, Alejandro Colomar wrote:
> Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>

Applied.

Thanks,

Michael


> ---
>  man2/clone.2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man2/clone.2 b/man2/clone.2
> index 838fbd669..04b4ec38c 100644
> --- a/man2/clone.2
> +++ b/man2/clone.2
> @@ -969,7 +969,7 @@ mask must also include
>  .B CLONE_VM
>  if
>  .B CLONE_SIGHAND
> -is specified
> +is specified.
>  .TP
>  .BR CLONE_STOPPED " (since Linux 2.6.0)"
>  .\" Precisely: Linux 2.6.0-test2
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

[PATCH] clone.2: tfix

[Alejandro Colomar]

Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>
---
 man2/clone.2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man2/clone.2 b/man2/clone.2
index 838fbd669..04b4ec38c 100644
--- a/man2/clone.2
+++ b/man2/clone.2
@@ -969,7 +969,7 @@ mask must also include
 .B CLONE_VM
 if
 .B CLONE_SIGHAND
-is specified
+is specified.
 .TP
 .BR CLONE_STOPPED " (since Linux 2.6.0)"
 .\" Precisely: Linux 2.6.0-test2
-- 
2.31.1

Re: [PATCH] clone.2: tfix

[Alejandro Colomar (man-pages)]

Hello Johannes,

On 4/19/21 10:01 PM, Johannes Berg wrote:
> Despite my mention of this spawning a hilarious discussion
> on IRC, this alignment restriction should be 128-bit, not
> 126-bit.
> 
> Signed-off-by: Johannes Berg <johannes@sipsolutions.net>

Nice typo :)
Patch applied.

Thanks,

Alex

> ---
>   man2/clone.2 | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man2/clone.2 b/man2/clone.2
> index 818e584ceee1..838fbd669c57 100644
> --- a/man2/clone.2
> +++ b/man2/clone.2
> @@ -1418,7 +1418,7 @@ was an invalid.
>   .TP
>   .BR EINVAL " (AArch64 only, Linux 4.6 and earlier)"
>   .I stack
> -was not aligned to a 126-bit boundary.
> +was not aligned to a 128-bit boundary.
>   .TP
>   .B ENOMEM
>   Cannot allocate sufficient memory to allocate a task structure for the
> 


-- 
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/

[PATCH] clone.2: tfix

[Johannes Berg]

Despite my mention of this spawning a hilarious discussion
on IRC, this alignment restriction should be 128-bit, not
126-bit.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
---
 man2/clone.2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man2/clone.2 b/man2/clone.2
index 818e584ceee1..838fbd669c57 100644
--- a/man2/clone.2
+++ b/man2/clone.2
@@ -1418,7 +1418,7 @@ was an invalid.
 .TP
 .BR EINVAL " (AArch64 only, Linux 4.6 and earlier)"
 .I stack
-was not aligned to a 126-bit boundary.
+was not aligned to a 128-bit boundary.
 .TP
 .B ENOMEM
 Cannot allocate sufficient memory to allocate a task structure for the
-- 
2.30.2

Re: [patch] clone.2: tfix

[Michael Kerrisk (man-pages)]

Hello Johannes,

On 1/18/21 10:12 PM, Johannes Wellhöfer wrote:
> For the alternate signal stack to be cleared, CLONE_VM should and
> CLONE_VFORK should not be specified.
> 
> Signed-off-by: Johannes Wellhöfer <johannes.wellhofer@gmail.com>

Thanks! Patch applied.

Cheers,

Michael

> ---
>  man2/clone.2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man2/clone.2 b/man2/clone.2
> index fecec90c8..11eb6c622 100644
> --- a/man2/clone.2
> +++ b/man2/clone.2
> @@ -1187,7 +1187,7 @@ processes do not affect the other, as with
>  If the
>  .BR CLONE_VM
>  flag is specified and the
> -.BR CLONE_VM
> +.BR CLONE_VFORK
>  flag is not specified,
>  then any alternate signal stack that was established by
>  .BR sigaltstack (2)
> --
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

[patch] clone.2: tfix

[Johannes Wellhöfer]

For the alternate signal stack to be cleared, CLONE_VM should and
CLONE_VFORK should not be specified.

Signed-off-by: Johannes Wellhöfer <johannes.wellhofer@gmail.com>
---
 man2/clone.2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man2/clone.2 b/man2/clone.2
index fecec90c8..11eb6c622 100644
--- a/man2/clone.2
+++ b/man2/clone.2
@@ -1187,7 +1187,7 @@ processes do not affect the other, as with
 If the
 .BR CLONE_VM
 flag is specified and the
-.BR CLONE_VM
+.BR CLONE_VFORK
 flag is not specified,
 then any alternate signal stack that was established by
 .BR sigaltstack (2)
--

Re: [PATCH] clone.2: tfix

[Michael Kerrisk (man-pages)]

On 3/5/20 10:35 PM, Krzysztof Małysa wrote:
> Fix clone3() syscall description for CLONE_PARENT_SETTID: kernel uses
> cl_args.parent_tid instead of the specified cl_args.child_tid.

Thanks, Krzysztof, Patch applied.

Cheers,

Michael

> Signed-off-by: Krzysztof Małysa <varqox@gmail.com>
> ---
>  man2/clone.2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man2/clone.2 b/man2/clone.2
> index f9920ee66..39cec4c86 100644
> --- a/man2/clone.2
> +++ b/man2/clone.2
> @@ -765,7 +765,7 @@ Store the child thread ID at the location pointed to by
>  .I parent_tid
>  .RB ( clone ())
>  or
> -.I cl_args.child_tid
> +.I cl_args.parent_tid
>  .RB ( clone3 ())
>  in the parent's memory.
>  (In Linux 2.5.32-2.5.48 there was a flag
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/

[PATCH] clone.2: tfix

[Krzysztof Małysa]

Fix clone3() syscall description for CLONE_PARENT_SETTID: kernel uses
cl_args.parent_tid instead of the specified cl_args.child_tid.

Signed-off-by: Krzysztof Małysa <varqox@gmail.com>
---
 man2/clone.2 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man2/clone.2 b/man2/clone.2
index f9920ee66..39cec4c86 100644
--- a/man2/clone.2
+++ b/man2/clone.2
@@ -765,7 +765,7 @@ Store the child thread ID at the location pointed to by
 .I parent_tid
 .RB ( clone ())
 or
-.I cl_args.child_tid
+.I cl_args.parent_tid
 .RB ( clone3 ())
 in the parent's memory.
 (In Linux 2.5.32-2.5.48 there was a flag
-- 
2.25.1