Messages in this thread Patch in this message | | | Date | Sun, 8 Sep 2013 20:01:37 +0200 | From | Oleg Nesterov <> | Subject | free_pid() && PIDNS_HASH_ADDING |
| |
On 09/08, Oleg Nesterov wrote: > > Off topic. What if the first alloc_pid() succeeds and then later > copy_process() fails. In this case free_pid() is called but > PIDNS_HASH_ADDING was not cleared, we miss kern_unmount(), no?
Perhaps something like below?
Oleg.
--- x/kernel/pid.c +++ x/kernel/pid.c @@ -272,6 +272,8 @@ void free_pid(struct pid *pid) */ wake_up_process(ns->child_reaper); break; + case PIDNS_HASH_ADDING: + WARN_ON(ns->child_reaper); case 0: schedule_work(&ns->proc_work); break;
| |