Re: [PATCH 10/10] Bsdacct: account dying tasks in all relevant namespaces

Andrew Morton
2008-05-16T02:44:59Z

On Thu, 15 May 2008 15:11:55 +0400 Pavel Emelyanov <xemul@openvz.org> wrote:

> +/**
> + * acct_process - now just a wrapper around do_acct_process

comment seems wrong.

> + * handles process accounting for an exiting task
> + */
> +void acct_process(void)
> +{
> + struct pid_namespace *ns;
> +
> + for (ns = task_active_pid_ns(current); ns != NULL; ns = ns->parent)
> + acct_process_in_ns(ns);
> +}

Is it really safe to walk this list locklessly?