commit 158e1645e07f3e9f7e4962d7a0997f5c3b98311b
author Al Viro <viro@zeniv.linux.org.uk> Wed Jun 27 09:24:13 2012 +0400
committer Al Viro <viro@zeniv.linux.org.uk> Sun Jul 22 23:57:55 2012 +0400
tree 09d695b3a0410afe15f7fd6006bc6c1effee151d
parent 41f9d29f09ca0b22c3631e8a39676e74cda9bcc0

trim task_work: get rid of hlist

layout based on Oleg's suggestion; single-linked list,
task->task_works points to the last element, forward pointer
from said last element points to head.  I'd still prefer
much more regular scheme with two pointers in task_work,
but...

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

include/linux/sched.h

diff --git a/include/linux/sched.h b/include/linux/sched.h
index 4059c0f..b9216eb 100644
--- a/include/linux/sched.h
+++ b/include/linux/sched.h
@@ -1405,7 +1405,7 @@
 	int (*notifier)(void *priv);
 	void *notifier_data;
 	sigset_t *notifier_mask;
-	struct hlist_head task_works;
+	void *task_works;
 
 	struct audit_context *audit_context;
 #ifdef CONFIG_AUDITSYSCALL

include/linux/task_work.h

diff --git a/include/linux/task_work.h b/include/linux/task_work.h
index 627421c..3b3e2c8 100644
--- a/include/linux/task_work.h
+++ b/include/linux/task_work.h
@@ -8,7 +8,7 @@
 typedef void (*task_work_func_t)(struct task_work *);
 
 struct task_work {
-	struct hlist_node hlist;
+	struct task_work *next;
 	task_work_func_t func;
 };
 
@@ -24,7 +24,7 @@
 
 static inline void exit_task_work(struct task_struct *task)
 {
-	if (unlikely(!hlist_empty(&task->task_works)))
+	if (unlikely(task->task_works))
 		task_work_run();
 }
 

include/linux/tracehook.h

diff --git a/include/linux/tracehook.h b/include/linux/tracehook.h
index 6a4d82b..1e98b55 100644
--- a/include/linux/tracehook.h
+++ b/include/linux/tracehook.h
@@ -192,7 +192,7 @@
 	 * hlist_add_head(task->task_works);
 	 */
 	smp_mb__after_clear_bit();
-	if (unlikely(!hlist_empty(&current->task_works)))
+	if (unlikely(current->task_works))
 		task_work_run();
 }