[PATCH 11/12] pid: rewrite task helper functions avoiding task->pid and task->tgid
Oleg Nesterov
oleg at redhat.com
Thu Aug 22 19:08:48 UTC 2013
On 08/20, Richard Guy Briggs wrote:
>
> static inline int is_global_init(struct task_struct *tsk)
> {
> - return tsk->pid == 1;
> + return task_pid_nr(tsk) == 1;
> }
Probably it would be better to simply kill it. Almost every usage is
wrong.
> static inline bool is_idle_task(const struct task_struct *p)
> {
> - return p->pid == 0;
> + return task_pid(p) == &init_struct_pid;
> }
hmm. there should be a simpler check for this...
> static inline int has_group_leader_pid(struct task_struct *p)
> {
> - return p->pid == p->tgid;
> + return task_pid(p) == task_tgid(p);
> }
>
> static inline
> int same_thread_group(struct task_struct *p1, struct task_struct *p2)
> {
> - return p1->tgid == p2->tgid;
> + return task_tgid(p1) == task_tgid(p2);
This is suboptinal. See the attached
include-linux-schedh-dont-use-task-pid-tgid-in-same_thread_group-has_group_leader_pid.patch
from -mm below.
Oleg.
--- a/include/linux/sched.h~include-linux-schedh-dont-use-task-pid-tgid-in-same_thread_group-has_group_leader_pid
+++ a/include/linux/sched.h
@@ -2179,15 +2179,15 @@ static inline bool thread_group_leader(s
* all we care about is that we have a task with the appropriate
* pid, we don't actually care if we have the right task.
*/
-static inline int has_group_leader_pid(struct task_struct *p)
+static inline bool has_group_leader_pid(struct task_struct *p)
{
- return p->pid == p->tgid;
+ return task_pid(p) == p->signal->leader_pid;
}
static inline
-int same_thread_group(struct task_struct *p1, struct task_struct *p2)
+bool same_thread_group(struct task_struct *p1, struct task_struct *p2)
{
- return p1->tgid == p2->tgid;
+ return p1->signal == p2->signal;
}
static inline struct task_struct *next_thread(const struct task_struct *p)
More information about the Linux-audit
mailing list