On 08/06, Andrew Morton wrote: > > On Fri, 3 Aug 2007 01:20:09 +0400 Oleg Nesterov <oleg@xxxxxxxxxx> wrote: > > > 2. We are playing games with ->nsproxy->pid_ns. This code is bogus today, and > > it has to be changed anyway when we really support pid namespaces, just > > remove it. > > This patch broke > > pid-namespaces-define-and-use-task_active_pid_ns-wrapper.patch. This hunk: > > *************** > *** 908,915 **** > if (unlikely(!tsk->pid)) > panic("Attempted to kill the idle task!"); > if (unlikely(tsk == child_reaper(tsk))) { > - if (tsk->nsproxy->pid_ns != &init_pid_ns) > - tsk->nsproxy->pid_ns->child_reaper = init_pid_ns.child_reaper; > else > panic("Attempted to kill init!"); > } > --- 908,916 ---- > if (unlikely(!tsk->pid)) > panic("Attempted to kill the idle task!"); > if (unlikely(tsk == child_reaper(tsk))) { > + if (task_active_pid_ns(tsk) != &init_pid_ns) > + task_active_pid_ns(tsk)->child_reaper = > + init_pid_ns.child_reaper; > else > panic("Attempted to kill init!"); > } > > has no place to live any more, so I just removed it. Ah, thanks. I should have done this patch against -mm tree. I hope it is OK to drop this chunk of pid-namespaces-define-and-use-task_active_pid_ns-wrapper.patch Because it can't work right now anyway, and Sukadev+Pavel already have new patches on top this one which make namespace switch actually work. Oleg. _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers