The patch titled
Task watchers: register process events task watcher
has been removed from the -mm tree. Its filename is
task-watchers-register-process-events-task-watcher.patch
This patch was dropped because it was nacked by the maintainer
------------------------------------------------------
Subject: Task watchers: register process events task watcher
From: Matt Helsley <matthltc@xxxxxxxxxx>
This patch makes process events utilize task watchers instead of calling from
fork, exec, exit, and [re][ug]id changes directly.
Signed-off-by: Matt Helsley <matthltc@xxxxxxxxxx>
Cc: Guillaume Thouvenin <guillaume.thouvenin@xxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---
drivers/connector/cn_proc.c | 65 ++++++++++++++++++++++++++++------
fs/exec.c | 2 -
include/linux/cn_proc.h | 22 -----------
kernel/exit.c | 2 -
kernel/fork.c | 7 +--
kernel/sys.c | 8 ----
6 files changed, 57 insertions(+), 49 deletions(-)
diff -puN drivers/connector/cn_proc.c~task-watchers-register-process-events-task-watcher drivers/connector/cn_proc.c
--- a/drivers/connector/cn_proc.c~task-watchers-register-process-events-task-watcher
+++ a/drivers/connector/cn_proc.c
@@ -27,6 +27,7 @@
#include <linux/ktime.h>
#include <linux/init.h>
#include <linux/connector.h>
+#include <linux/notifier.h>
#include <asm/atomic.h>
#include <linux/cn_proc.h>
@@ -46,7 +47,7 @@ static inline void get_seq(__u32 *ts, in
put_cpu_var(proc_event_counts);
}
-void proc_fork_connector(struct task_struct *task)
+static void proc_fork_connector(struct task_struct *task)
{
struct cn_msg *msg;
struct proc_event *ev;
@@ -69,10 +70,10 @@ void proc_fork_connector(struct task_str
msg->ack = 0; /* not used */
msg->len = sizeof(*ev);
/* If cn_netlink_send() failed, the data is not sent */
- cn_netlink_send(msg, CN_IDX_PROC, GFP_KERNEL);
+ cn_netlink_send(msg, CN_IDX_PROC, GFP_ATOMIC);
}
-void proc_exec_connector(struct task_struct *task)
+static void proc_exec_connector(struct task_struct *task)
{
struct cn_msg *msg;
struct proc_event *ev;
@@ -92,10 +93,10 @@ void proc_exec_connector(struct task_str
memcpy(&msg->id, &cn_proc_event_id, sizeof(msg->id));
msg->ack = 0; /* not used */
msg->len = sizeof(*ev);
- cn_netlink_send(msg, CN_IDX_PROC, GFP_KERNEL);
+ cn_netlink_send(msg, CN_IDX_PROC, GFP_ATOMIC);
}
-void proc_id_connector(struct task_struct *task, int which_id)
+static void proc_id_connector(struct task_struct *task, int which_id)
{
struct cn_msg *msg;
struct proc_event *ev;
@@ -123,10 +124,10 @@ void proc_id_connector(struct task_struc
memcpy(&msg->id, &cn_proc_event_id, sizeof(msg->id));
msg->ack = 0; /* not used */
msg->len = sizeof(*ev);
- cn_netlink_send(msg, CN_IDX_PROC, GFP_KERNEL);
+ cn_netlink_send(msg, CN_IDX_PROC, GFP_ATOMIC);
}
-void proc_exit_connector(struct task_struct *task)
+static void proc_exit_connector(struct task_struct *task)
{
struct cn_msg *msg;
struct proc_event *ev;
@@ -148,7 +149,7 @@ void proc_exit_connector(struct task_str
memcpy(&msg->id, &cn_proc_event_id, sizeof(msg->id));
msg->ack = 0; /* not used */
msg->len = sizeof(*ev);
- cn_netlink_send(msg, CN_IDX_PROC, GFP_KERNEL);
+ cn_netlink_send(msg, CN_IDX_PROC, GFP_ATOMIC);
}
/*
@@ -209,6 +210,45 @@ static void cn_proc_mcast_ctl(void *data
cn_proc_ack(err, msg->seq, msg->ack);
}
+
+/*
+ * Dispatch task watcher events to the appropriate process event
+ * generation function.
+ */
+static int cn_proc_watch_task(struct notifier_block *nb, unsigned long val,
+ void *t)
+{
+ struct task_struct *task = t;
+ int rc = NOTIFY_OK;
+
+ switch (get_watch_event(val)) {
+ case WATCH_TASK_CLONE:
+ proc_fork_connector(task);
+ break;
+ case WATCH_TASK_EXEC:
+ proc_exec_connector(task);
+ break;
+ case WATCH_TASK_UID:
+ proc_id_connector(task, PROC_EVENT_UID);
+ break;
+ case WATCH_TASK_GID:
+ proc_id_connector(task, PROC_EVENT_GID);
+ break;
+ case WATCH_TASK_EXIT:
+ proc_exit_connector(task);
+ break;
+ default: /* we don't care about WATCH_TASK_INIT|FREE because we
+ don't keep per-task info */
+ rc = NOTIFY_DONE; /* ignore all other notifications */
+ break;
+ }
+ return rc;
+}
+
+static struct notifier_block __read_mostly cn_proc_nb = {
+ .notifier_call = cn_proc_watch_task,
+};
+
/*
* cn_proc_init - initialization entry point
*
@@ -221,9 +261,14 @@ static int __init cn_proc_init(void)
if ((err = cn_add_callback(&cn_proc_event_id, "cn_proc",
&cn_proc_mcast_ctl))) {
printk(KERN_WARNING "cn_proc failed to register\n");
- return err;
+ goto out;
}
- return 0;
+
+ err = register_task_watcher(&cn_proc_nb);
+ if (err != 0)
+ cn_del_callback(&cn_proc_event_id);
+out:
+ return err;
}
module_init(cn_proc_init);
diff -puN fs/exec.c~task-watchers-register-process-events-task-watcher fs/exec.c
--- a/fs/exec.c~task-watchers-register-process-events-task-watcher
+++ a/fs/exec.c
@@ -47,7 +47,6 @@
#include <linux/syscalls.h>
#include <linux/rmap.h>
#include <linux/acct.h>
-#include <linux/cn_proc.h>
#include <linux/audit.h>
#include <linux/notifier.h>
@@ -1087,7 +1086,6 @@ int search_binary_handler(struct linux_b
bprm->file = NULL;
current->did_exec = 1;
notify_watchers(WATCH_TASK_EXEC, current);
- proc_exec_connector(current);
return retval;
}
read_lock(&binfmt_lock);
diff -puN include/linux/cn_proc.h~task-watchers-register-process-events-task-watcher include/linux/cn_proc.h
--- a/include/linux/cn_proc.h~task-watchers-register-process-events-task-watcher
+++ a/include/linux/cn_proc.h
@@ -95,26 +95,4 @@ struct proc_event {
} exit;
} event_data;
};
-
-#ifdef __KERNEL__
-#ifdef CONFIG_PROC_EVENTS
-void proc_fork_connector(struct task_struct *task);
-void proc_exec_connector(struct task_struct *task);
-void proc_id_connector(struct task_struct *task, int which_id);
-void proc_exit_connector(struct task_struct *task);
-#else
-static inline void proc_fork_connector(struct task_struct *task)
-{}
-
-static inline void proc_exec_connector(struct task_struct *task)
-{}
-
-static inline void proc_id_connector(struct task_struct *task,
- int which_id)
-{}
-
-static inline void proc_exit_connector(struct task_struct *task)
-{}
-#endif /* CONFIG_PROC_EVENTS */
-#endif /* __KERNEL__ */
#endif /* CN_PROC_H */
diff -puN kernel/exit.c~task-watchers-register-process-events-task-watcher kernel/exit.c
--- a/kernel/exit.c~task-watchers-register-process-events-task-watcher
+++ a/kernel/exit.c
@@ -31,7 +31,6 @@
#include <linux/syscalls.h>
#include <linux/signal.h>
#include <linux/posix-timers.h>
-#include <linux/cn_proc.h>
#include <linux/mutex.h>
#include <linux/futex.h>
#include <linux/compat.h>
@@ -936,7 +935,6 @@ fastcall NORET_TYPE void do_exit(long co
if (tsk->binfmt)
module_put(tsk->binfmt->module);
- proc_exit_connector(tsk);
exit_notify(tsk);
#ifdef CONFIG_NUMA
mpol_free(tsk->mempolicy);
diff -puN kernel/fork.c~task-watchers-register-process-events-task-watcher kernel/fork.c
--- a/kernel/fork.c~task-watchers-register-process-events-task-watcher
+++ a/kernel/fork.c
@@ -43,7 +43,6 @@
#include <linux/profile.h>
#include <linux/rmap.h>
#include <linux/acct.h>
-#include <linux/cn_proc.h>
#include <linux/delayacct.h>
#include <linux/taskstats_kern.h>
@@ -1254,10 +1253,8 @@ static struct task_struct *copy_process(
spin_unlock(¤t->sighand->siglock);
write_unlock_irq(&tasklist_lock);
notify_result = notify_watchers(WATCH_TASK_CLONE, p);
- if (notify_result & NOTIFY_STOP_MASK)
- goto bad_fork_cleanup_namespaces;
- proc_fork_connector(p);
- return p;
+ if (!(notify_result & NOTIFY_STOP_MASK))
+ return p;
bad_fork_cleanup_namespace:
exit_namespace(p);
diff -puN kernel/sys.c~task-watchers-register-process-events-task-watcher kernel/sys.c
--- a/kernel/sys.c~task-watchers-register-process-events-task-watcher
+++ a/kernel/sys.c
@@ -859,7 +859,6 @@ asmlinkage long sys_setregid(gid_t rgid,
current->gid = new_rgid;
key_fsgid_changed(current);
notify_watchers(WATCH_TASK_GID, current);
- proc_id_connector(current, PROC_EVENT_GID);
return 0;
}
@@ -900,7 +899,6 @@ asmlinkage long sys_setgid(gid_t gid)
key_fsgid_changed(current);
notify_watchers(WATCH_TASK_GID, current);
- proc_id_connector(current, PROC_EVENT_GID);
return 0;
}
@@ -991,7 +989,6 @@ asmlinkage long sys_setreuid(uid_t ruid,
key_fsuid_changed(current);
notify_watchers(WATCH_TASK_UID, current);
- proc_id_connector(current, PROC_EVENT_UID);
return security_task_post_setuid(old_ruid, old_euid, old_suid, LSM_SETID_RE);
}
@@ -1040,7 +1037,6 @@ asmlinkage long sys_setuid(uid_t uid)
key_fsuid_changed(current);
notify_watchers(WATCH_TASK_UID, current);
- proc_id_connector(current, PROC_EVENT_UID);
return security_task_post_setuid(old_ruid, old_euid, old_suid, LSM_SETID_ID);
}
@@ -1090,7 +1086,6 @@ asmlinkage long sys_setresuid(uid_t ruid
key_fsuid_changed(current);
notify_watchers(WATCH_TASK_UID, current);
- proc_id_connector(current, PROC_EVENT_UID);
return security_task_post_setuid(old_ruid, old_euid, old_suid, LSM_SETID_RES);
}
@@ -1144,7 +1139,6 @@ asmlinkage long sys_setresgid(gid_t rgid
key_fsgid_changed(current);
notify_watchers(WATCH_TASK_GID, current);
- proc_id_connector(current, PROC_EVENT_GID);
return 0;
}
@@ -1188,7 +1182,6 @@ asmlinkage long sys_setfsuid(uid_t uid)
key_fsuid_changed(current);
notify_watchers(WATCH_TASK_UID, current);
- proc_id_connector(current, PROC_EVENT_UID);
security_task_post_setuid(old_fsuid, (uid_t)-1, (uid_t)-1, LSM_SETID_FS);
@@ -1218,7 +1211,6 @@ asmlinkage long sys_setfsgid(gid_t gid)
current->fsgid = gid;
key_fsgid_changed(current);
notify_watchers(WATCH_TASK_GID, current);
- proc_id_connector(current, PROC_EVENT_GID);
}
return old_fsgid;
}
_
Patches currently in -mm which might be from matthltc@xxxxxxxxxx are
kernel-bug-fixing-for-kernel-kmodc.patch
task-watchers-register-process-events-task-watcher.patch
task-watchers-refactor-process-events.patch
task-watchers-make-process-events-configurable-as.patch
task-watchers-allow-task-watchers-to-block.patch
task-watchers-register-audit-task-watcher.patch
task-watchers-register-per-task-delay-accounting.patch
task-watchers-register-profile-as-a-task-watcher.patch
task-watchers-add-support-for-per-task-watchers.patch
task-watchers-register-semundo-task-watcher.patch
task-watchers-register-per-task-semundo-watcher.patch
ipc-replace-kmalloc-and-memset-in-get_undo_list-with-kzalloc.patch
-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
