Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Jonathan Corbet <corbet@lwn.net>
To:     Chen Linxuan <chenlinxuan@uniontech.com>
Cc:     Chen Linxuan <chenlinxuan@uniontech.com>,
        Yuan Haisheng <heysion@deepin.com>,
        linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
        linux-doc@vger.kernel.org
Subject: Re: [PATCH] Documentation: update the description of TracerPid in
 procfs.rst
In-Reply-To: <20221102081517.19770-1-chenlinxuan@uniontech.com>
References: <20221102081517.19770-1-chenlinxuan@uniontech.com>
Date:   Tue, 15 Nov 2022 15:09:00 -0700
Message-ID: <87v8nf98g3.fsf@meer.lwn.net>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Chen Linxuan <chenlinxuan@uniontech.com> writes:

> When the tracer of process is outside of current pid namespace, field
> `TracerPid` in /proc/<pid>/status will be 0, too, just like this process
> not have been traced.
>
> This is because that function `task_pid_nr_ns` used to get the pid of
> tracer will return 0 in this situation.
>
> Co-authored-by: Yuan Haisheng <heysion@deepin.com>
> Signed-off-by: Chen Linxuan <chenlinxuan@uniontech.com>
> ---
>  Documentation/filesystems/proc.rst | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/filesystems/proc.rst b/Documentation/filesystems/proc.rst
> index 898c99eae8e4..e98e0277f05e 100644
> --- a/Documentation/filesystems/proc.rst
> +++ b/Documentation/filesystems/proc.rst
> @@ -245,7 +245,8 @@ It's slow but very precise.
>   Ngid                        NUMA group ID (0 if none)
>   Pid                         process id
>   PPid                        process id of the parent process
> - TracerPid                   PID of process tracing this process (0 if not)
> + TracerPid                   PID of process tracing this process (0 if not, or
> +                             the tracer is outside of the current pid namespace)

Applied, thanks.

jon