[Devel] Re: [v12][PATCH 4/9] Add target_pids parameter to alloc_pid()
Sukadev Bhattiprolu
sukadev at linux.vnet.ibm.com
Wed Nov 11 14:39:43 PST 2009
Cc: LKML
Sukadev Bhattiprolu [sukadev at linux.vnet.ibm.com] wrote:
|
| Subject: [v12][PATCH 4/9] Add target_pids parameter to alloc_pid()
|
| This parameter is currently NULL, but will be used in a follow-on patch.
|
| Signed-off-by: Sukadev Bhattiprolu <sukadev at linux.vnet.ibm.com>
| Acked-by: Serge Hallyn <serue at us.ibm.com>
| Reviewed-by: Oren Laadan <orenl at cs.columbia.edu>
| ---
| include/linux/pid.h | 2 +-
| kernel/fork.c | 3 ++-
| kernel/pid.c | 9 +++++++--
| 3 files changed, 10 insertions(+), 4 deletions(-)
|
| diff --git a/include/linux/pid.h b/include/linux/pid.h
| index 49f1c2f..914185d 100644
| --- a/include/linux/pid.h
| +++ b/include/linux/pid.h
| @@ -119,7 +119,7 @@ extern struct pid *find_get_pid(int nr);
| extern struct pid *find_ge_pid(int nr, struct pid_namespace *);
| int next_pidmap(struct pid_namespace *pid_ns, int last);
|
| -extern struct pid *alloc_pid(struct pid_namespace *ns);
| +extern struct pid *alloc_pid(struct pid_namespace *ns, pid_t *target_pids);
| extern void free_pid(struct pid *pid);
|
| /*
| diff --git a/kernel/fork.c b/kernel/fork.c
| index 93626b2..3f1dddf 100644
| --- a/kernel/fork.c
| +++ b/kernel/fork.c
| @@ -980,6 +980,7 @@ static struct task_struct *copy_process(unsigned long clone_flags,
| int retval;
| struct task_struct *p;
| int cgroup_callbacks_done = 0;
| + pid_t *target_pids = NULL;
|
| if ((clone_flags & (CLONE_NEWNS|CLONE_FS)) == (CLONE_NEWNS|CLONE_FS))
| return ERR_PTR(-EINVAL);
| @@ -1156,7 +1157,7 @@ static struct task_struct *copy_process(unsigned long clone_flags,
| goto bad_fork_cleanup_io;
|
| if (pid != &init_struct_pid) {
| - pid = alloc_pid(p->nsproxy->pid_ns);
| + pid = alloc_pid(p->nsproxy->pid_ns, target_pids);
| if (IS_ERR(pid)) {
| retval = PTR_ERR(pid);
| goto bad_fork_cleanup_io;
| diff --git a/kernel/pid.c b/kernel/pid.c
| index afef766..3ee52cd 100644
| --- a/kernel/pid.c
| +++ b/kernel/pid.c
| @@ -281,13 +281,14 @@ void free_pid(struct pid *pid)
| call_rcu(&pid->rcu, delayed_put_pid);
| }
|
| -struct pid *alloc_pid(struct pid_namespace *ns)
| +struct pid *alloc_pid(struct pid_namespace *ns, pid_t *target_pids)
| {
| struct pid *pid;
| enum pid_type type;
| int i, nr;
| struct pid_namespace *tmp;
| struct upid *upid;
| + pid_t tpid;
|
| pid = kmem_cache_alloc(ns->pid_cachep, GFP_KERNEL);
| if (!pid) {
| @@ -297,7 +298,11 @@ struct pid *alloc_pid(struct pid_namespace *ns)
|
| tmp = ns;
| for (i = ns->level; i >= 0; i--) {
| - nr = alloc_pidmap(tmp);
| + tpid = 0;
| + if (target_pids)
| + tpid = target_pids[i];
| +
| + nr = set_pidmap(tmp, tpid);
| if (nr < 0)
| goto out_free;
|
| --
| 1.6.0.4
|
| _______________________________________________
| Containers mailing list
| Containers at lists.linux-foundation.org
| https://lists.linux-foundation.org/mailman/listinfo/containers
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
More information about the Devel
mailing list