[PATCH v1 0/7] pid: Introduce helper task_is_in_root_ns() · Leo Yan <hidden> · 2021-12-05
[PATCH v1 1/7] pid: Introduce helper task_is_in_root_ns() · Leo Yan <hidden> · 2021-12-05
Re: [PATCH v1 1/7] pid: Introduce helper task_is_in_root_ns() · Leon Romanovsky <leon@kernel.org> · 2021-12-06
Re: [PATCH v1 1/7] pid: Introduce helper task_is_in_root_ns() · Leo Yan <hidden> · 2021-12-06
Re: [PATCH v1 1/7] pid: Introduce helper task_is_in_root_ns() · Leon Romanovsky <leon@kernel.org> · 2021-12-06
[PATCH v1 2/7] coresight: etm3x: Use task_is_in_root_ns() to check PID namespace · Leo Yan <hidden> · 2021-12-05
[PATCH v1 3/7] coresight: etm4x: Use task_is_in_root_ns() to check PID namespace · Leo Yan <hidden> · 2021-12-05
[PATCH v1 4/7] connector/cn_proc: Use task_is_in_root_ns() to check PID namespace · Leo Yan <hidden> · 2021-12-05
[PATCH v1 5/7] coda: Use task_is_in_root_ns() · Leo Yan <hidden> · 2021-12-05
[PATCH v1 6/7] audit: Use task_is_in_root_ns() · Leo Yan <hidden> · 2021-12-05
[PATCH v1 7/7] taskstats: Use task_is_in_root_ns() · Leo Yan <hidden> · 2021-12-05

Re: [PATCH v1 1/7] pid: Introduce helper task_is_in_root_ns()

From: Leo Yan <hidden>
Date: 2021-12-06 07:04:09
Also in: linux-arm-kernel, lkml

Hi Leon,

On Mon, Dec 06, 2021 at 08:49:01AM +0200, Leon Romanovsky wrote:

On Sun, Dec 05, 2021 at 10:50:59PM +0800, Leo Yan wrote:

[...]

quoted

+static inline bool task_is_in_root_ns(struct task_struct *tsk)

It is bad that this name doesn't reflect PID nature of this namespace.
Won't it better to name it task_is_in_init_pid_ns()?

Yes, task_is_in_init_pid_ns() is more clear.

Will respin for this.  Thank you for suggestion!

Leo