diff options
author | Pavel Emelyanov <xemul@openvz.org> | 2007-10-19 02:40:14 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-19 14:53:40 -0400 |
commit | b488893a390edfe027bae7a46e9af8083e740668 (patch) | |
tree | c469a7f99ad01005a73011c029eb5e5d15454559 /ipc/msg.c | |
parent | 3eb07c8c8adb6f0572baba844ba2d9e501654316 (diff) |
pid namespaces: changes to show virtual ids to user
This is the largest patch in the set. Make all (I hope) the places where
the pid is shown to or get from user operate on the virtual pids.
The idea is:
- all in-kernel data structures must store either struct pid itself
or the pid's global nr, obtained with pid_nr() call;
- when seeking the task from kernel code with the stored id one
should use find_task_by_pid() call that works with global pids;
- when showing pid's numerical value to the user the virtual one
should be used, but however when one shows task's pid outside this
task's namespace the global one is to be used;
- when getting the pid from userspace one need to consider this as
the virtual one and use appropriate task/pid-searching functions.
[akpm@linux-foundation.org: build fix]
[akpm@linux-foundation.org: nuther build fix]
[akpm@linux-foundation.org: yet nuther build fix]
[akpm@linux-foundation.org: remove unneeded casts]
Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Signed-off-by: Alexey Dobriyan <adobriyan@openvz.org>
Cc: Sukadev Bhattiprolu <sukadev@us.ibm.com>
Cc: Oleg Nesterov <oleg@tv-sign.ru>
Cc: Paul Menage <menage@google.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'ipc/msg.c')
-rw-r--r-- | ipc/msg.c | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -611,7 +611,7 @@ static inline int pipelined_send(struct msg_queue *msq, struct msg_msg *msg) | |||
611 | msr->r_msg = ERR_PTR(-E2BIG); | 611 | msr->r_msg = ERR_PTR(-E2BIG); |
612 | } else { | 612 | } else { |
613 | msr->r_msg = NULL; | 613 | msr->r_msg = NULL; |
614 | msq->q_lrpid = msr->r_tsk->pid; | 614 | msq->q_lrpid = task_pid_vnr(msr->r_tsk); |
615 | msq->q_rtime = get_seconds(); | 615 | msq->q_rtime = get_seconds(); |
616 | wake_up_process(msr->r_tsk); | 616 | wake_up_process(msr->r_tsk); |
617 | smp_mb(); | 617 | smp_mb(); |
@@ -695,7 +695,7 @@ long do_msgsnd(int msqid, long mtype, void __user *mtext, | |||
695 | } | 695 | } |
696 | } | 696 | } |
697 | 697 | ||
698 | msq->q_lspid = current->tgid; | 698 | msq->q_lspid = task_tgid_vnr(current); |
699 | msq->q_stime = get_seconds(); | 699 | msq->q_stime = get_seconds(); |
700 | 700 | ||
701 | if (!pipelined_send(msq, msg)) { | 701 | if (!pipelined_send(msq, msg)) { |
@@ -810,7 +810,7 @@ long do_msgrcv(int msqid, long *pmtype, void __user *mtext, | |||
810 | list_del(&msg->m_list); | 810 | list_del(&msg->m_list); |
811 | msq->q_qnum--; | 811 | msq->q_qnum--; |
812 | msq->q_rtime = get_seconds(); | 812 | msq->q_rtime = get_seconds(); |
813 | msq->q_lrpid = current->tgid; | 813 | msq->q_lrpid = task_tgid_vnr(current); |
814 | msq->q_cbytes -= msg->m_ts; | 814 | msq->q_cbytes -= msg->m_ts; |
815 | atomic_sub(msg->m_ts, &msg_bytes); | 815 | atomic_sub(msg->m_ts, &msg_bytes); |
816 | atomic_dec(&msg_hdrs); | 816 | atomic_dec(&msg_hdrs); |