From: Alejandro Colomar <alx@kernel.org>
To: Matthieu Buffet <matthieu@buffet.re>
Cc: linux-man@vger.kernel.org
Subject: Re: [PATCH v3 1/2] man/man7/pid_namespaces.7: Fix requirements on namespace+process trees
Date: Wed, 13 May 2026 13:40:39 +0200 [thread overview]
Message-ID: <agRi-dosgYg5EK8h@devuan> (raw)
In-Reply-To: <20260513083339.27911-1-matthieu@buffet.re>
[-- Attachment #1: Type: text/plain, Size: 1700 bytes --]
Hi Matthieu,
[CC += linux-man@]
(You forgot to CC the list.)
On 2026-05-13T10:33:38+0200, Matthieu Buffet wrote:
> Creating processes in non-direct-child PID namespaces has been possible
> since the addition of setns() support for PID namespaces in commit
> 57e8391d3276 (2012-11-19; "pidns: Add setns support"). The tree check in
> pidns_install() in kernel/pid_namespace.c has always allowed
> non-direct-child PID namespaces, but was written inline instead of the
> more readable current pidns_is_ancestor() helper (possibly explaining
> the confusion).
>
> Fixes: 6e377abf9 (2014-09-13; "pid_namespaces.7: Parent process relationships mirror parent PID namespace relationships")
> Signed-off-by: Matthieu Buffet <matthieu@buffet.re>
Thanks! I've applied the patch.
Cheers,
Alex
> ---
> man/man7/pid_namespaces.7 | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/man/man7/pid_namespaces.7 b/man/man7/pid_namespaces.7
> index 1068d5be5..b19afd505 100644
> --- a/man/man7/pid_namespaces.7
> +++ b/man/man7/pid_namespaces.7
> @@ -214,11 +214,12 @@ and cannot be changed thereafter.
> Among other things,
> this means that
> the parental relationship between processes
> -mirrors
> +loosely mirrors
> the parental relationship between PID namespaces:
> the parent of a process
> is either in the same namespace
> -or resides in the immediate parent PID namespace.
> +or resides in an ancestor PID namespace
> +(immediate parent or not).
> .P
> A process may call
> .BR unshare (2)
>
> base-commit: 9afc5a7cfa9ecd91db055abb875e3b83d086f6b5
> --
> 2.47.3
>
--
<https://www.alejandro-colomar.es>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next parent reply other threads:[~2026-05-13 11:40 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <75614ec3-0def-4cdd-b45c-17d21cf8357b@buffet.re>
[not found] ` <20260513083339.27911-1-matthieu@buffet.re>
2026-05-13 11:40 ` Alejandro Colomar [this message]
[not found] ` <20260513083339.27911-2-matthieu@buffet.re>
2026-05-13 11:41 ` [PATCH v3 2/2] man/man7/pid_namespaces.7: Add setns restriction and reasoning Alejandro Colomar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=agRi-dosgYg5EK8h@devuan \
--to=alx@kernel.org \
--cc=linux-man@vger.kernel.org \
--cc=matthieu@buffet.re \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox