diff options
author | Christoph Hellwig <hch@lst.de> | 2009-06-17 19:27:51 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-06-18 16:03:55 -0400 |
commit | 17f98dcf6010a1cfd25d179fd0ce77d3dc2685c3 (patch) | |
tree | 520d88ba2282b4811f492be6cc4565a2a3eea128 /include/linux/sched.h | |
parent | 7338f29984114066b00da343a22876bb08259a84 (diff) |
pids: clean up find_task_by_pid variants
find_task_by_pid_type_ns is only used to implement find_task_by_vpid and
find_task_by_pid_ns, but both of them pass PIDTYPE_PID as first argument.
So just fold find_task_by_pid_type_ns into find_task_by_pid_ns and use
find_task_by_pid_ns to implement find_task_by_vpid.
While we're at it also remove the exports for find_task_by_pid_ns and
find_task_by_vpid - we don't have any modular callers left as the only
modular caller of he old pre pid namespace find_task_by_pid (gfs2) was
switched to pid_task which operates on a struct pid pointer instead of a
pid_t. Given the confusion about pid_t values vs namespace that's
generally the better option anyway and I think we're better of restricting
modules to do it that way.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Cc: Pavel Emelyanov <xemul@openvz.org>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include/linux/sched.h')
-rw-r--r-- | include/linux/sched.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/include/linux/sched.h b/include/linux/sched.h index d0342101756a..4d0754269884 100644 --- a/include/linux/sched.h +++ b/include/linux/sched.h | |||
@@ -1878,9 +1878,6 @@ extern struct pid_namespace init_pid_ns; | |||
1878 | /* | 1878 | /* |
1879 | * find a task by one of its numerical ids | 1879 | * find a task by one of its numerical ids |
1880 | * | 1880 | * |
1881 | * find_task_by_pid_type_ns(): | ||
1882 | * it is the most generic call - it finds a task by all id, | ||
1883 | * type and namespace specified | ||
1884 | * find_task_by_pid_ns(): | 1881 | * find_task_by_pid_ns(): |
1885 | * finds a task by its pid in the specified namespace | 1882 | * finds a task by its pid in the specified namespace |
1886 | * find_task_by_vpid(): | 1883 | * find_task_by_vpid(): |
@@ -1889,9 +1886,6 @@ extern struct pid_namespace init_pid_ns; | |||
1889 | * see also find_vpid() etc in include/linux/pid.h | 1886 | * see also find_vpid() etc in include/linux/pid.h |
1890 | */ | 1887 | */ |
1891 | 1888 | ||
1892 | extern struct task_struct *find_task_by_pid_type_ns(int type, int pid, | ||
1893 | struct pid_namespace *ns); | ||
1894 | |||
1895 | extern struct task_struct *find_task_by_vpid(pid_t nr); | 1889 | extern struct task_struct *find_task_by_vpid(pid_t nr); |
1896 | extern struct task_struct *find_task_by_pid_ns(pid_t nr, | 1890 | extern struct task_struct *find_task_by_pid_ns(pid_t nr, |
1897 | struct pid_namespace *ns); | 1891 | struct pid_namespace *ns); |