From f77d181a428c180bf88fb832ead63b094efd6485 Mon Sep 17 00:00:00 2001 From: Jiri Pirko Date: Tue, 14 Apr 2009 20:17:16 +0200 Subject: [PATCH] --- yaml --- r: 145961 b: refs/heads/master c: 05725f7eb4b8acb147c5fc7b91397b1f6bcab00d h: refs/heads/master i: 145959: 6b8cc973c0235b194be851d803b691bdd70b9b91 v: v3 --- [refs] | 2 +- trunk/include/linux/sched.h | 8 +++++--- trunk/ipc/sem.c | 4 ++-- trunk/security/integrity/ima/ima_fs.c | 4 ++-- trunk/security/smack/smackfs.c | 8 ++++---- 5 files changed, 14 insertions(+), 12 deletions(-) diff --git a/[refs] b/[refs] index a79e13cdfe65..e6967c746e85 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 72c6a9870f901045f2464c3dc6ee8914bfdc07aa +refs/heads/master: 05725f7eb4b8acb147c5fc7b91397b1f6bcab00d diff --git a/trunk/include/linux/sched.h b/trunk/include/linux/sched.h index b4c38bc8049c..886df41e7452 100644 --- a/trunk/include/linux/sched.h +++ b/trunk/include/linux/sched.h @@ -77,6 +77,7 @@ struct sched_param { #include #include #include +#include #include #include @@ -2010,7 +2011,8 @@ static inline unsigned long wait_task_inactive(struct task_struct *p, } #endif -#define next_task(p) list_entry(rcu_dereference((p)->tasks.next), struct task_struct, tasks) +#define next_task(p) \ + list_entry_rcu((p)->tasks.next, struct task_struct, tasks) #define for_each_process(p) \ for (p = &init_task ; (p = next_task(p)) != &init_task ; ) @@ -2049,8 +2051,8 @@ int same_thread_group(struct task_struct *p1, struct task_struct *p2) static inline struct task_struct *next_thread(const struct task_struct *p) { - return list_entry(rcu_dereference(p->thread_group.next), - struct task_struct, thread_group); + return list_entry_rcu(p->thread_group.next, + struct task_struct, thread_group); } static inline int thread_group_empty(struct task_struct *p) diff --git a/trunk/ipc/sem.c b/trunk/ipc/sem.c index 16a2189e96f9..87c2b641fd7b 100644 --- a/trunk/ipc/sem.c +++ b/trunk/ipc/sem.c @@ -1290,8 +1290,8 @@ void exit_sem(struct task_struct *tsk) int i; rcu_read_lock(); - un = list_entry(rcu_dereference(ulp->list_proc.next), - struct sem_undo, list_proc); + un = list_entry_rcu(ulp->list_proc.next, + struct sem_undo, list_proc); if (&un->list_proc == &ulp->list_proc) semid = -1; else diff --git a/trunk/security/integrity/ima/ima_fs.c b/trunk/security/integrity/ima/ima_fs.c index ffbe259700b1..510186f0b72e 100644 --- a/trunk/security/integrity/ima/ima_fs.c +++ b/trunk/security/integrity/ima/ima_fs.c @@ -84,8 +84,8 @@ static void *ima_measurements_next(struct seq_file *m, void *v, loff_t *pos) * against concurrent list-extension */ rcu_read_lock(); - qe = list_entry(rcu_dereference(qe->later.next), - struct ima_queue_entry, later); + qe = list_entry_rcu(qe->later.next, + struct ima_queue_entry, later); rcu_read_unlock(); (*pos)++; diff --git a/trunk/security/smack/smackfs.c b/trunk/security/smack/smackfs.c index e03a7e19c73b..11d2cb19d7a6 100644 --- a/trunk/security/smack/smackfs.c +++ b/trunk/security/smack/smackfs.c @@ -734,8 +734,8 @@ static void smk_netlbladdr_insert(struct smk_netlbladdr *new) return; } - m = list_entry(rcu_dereference(smk_netlbladdr_list.next), - struct smk_netlbladdr, list); + m = list_entry_rcu(smk_netlbladdr_list.next, + struct smk_netlbladdr, list); /* the comparison '>' is a bit hacky, but works */ if (new->smk_mask.s_addr > m->smk_mask.s_addr) { @@ -748,8 +748,8 @@ static void smk_netlbladdr_insert(struct smk_netlbladdr *new) list_add_rcu(&new->list, &m->list); return; } - m_next = list_entry(rcu_dereference(m->list.next), - struct smk_netlbladdr, list); + m_next = list_entry_rcu(m->list.next, + struct smk_netlbladdr, list); if (new->smk_mask.s_addr > m_next->smk_mask.s_addr) { list_add_rcu(&new->list, &m->list); return;