Subject: Re: [PATCH v3 11/70] ns: add active reference count

From: Thomas Gleixner <[email-masked]>
Date: Mon, 27 Oct 2025 17:36:27 +0100

On Fri, Oct 24 2025 at 12:52, Christian Brauner wrote:
> diff --git a/kernel/time/namespace.c b/kernel/time/namespace.c
> index ee05cad288da..2e7c110bd13f 100644
> --- a/kernel/time/namespace.c
> +++ b/kernel/time/namespace.c
> @@ -106,6 +106,7 @@ static struct time_namespace *clone_time_ns(struct user_namespace *user_ns,
>  	ns->offsets = old_ns->offsets;
>  	ns->frozen_offsets = false;
>  	ns_tree_add(ns);
> +	ns_ref_active_get_owner(ns);

It seems all places where ns_ref_active_get_owner() is added it is
preceeded by a variant of ns_tree_add(). So why don't you stilck that
refcount thing into ns_tree_add()? I'm probably missing something here.

Thanks,

        tglx