diff options
| author |   Tejun Heo <tj@kernel.org> | 2013-06-03 19:13:55 -0700 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2013-06-05 12:00:33 -0700 |
| commit | cc5943a7816ba6c00639837a62131386619548dc (patch) | |
| tree | 53027a91d8fa13ef177292cdc32e2b87d56d8284 /kernel/cgroup.c | |
| parent | cgroup: mark "tasks" cgroup file as insane (diff) | |
| download | linux-dev-cc5943a7816ba6c00639837a62131386619548dc.tar.xz linux-dev-cc5943a7816ba6c00639837a62131386619548dc.zip | |
cgroup: mark "notify_on_release" and "release_agent" cgroup files insane
The empty cgroup notification mechanism currently implemented in
cgroup is tragically outdated. Forking and execing userland process
stopped being a viable notification mechanism more than a decade ago.
We're gonna have a saner mechanism. Let's make it clear that this
abomination is going away.
Mark "notify_on_release" and "release_agent" with CFTYPE_INSANE.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Li Zefan <lizefan@huawei.com>
Diffstat (limited to 'kernel/cgroup.c')
| -rw-r--r-- | kernel/cgroup.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c index 1e0f445b5b88..b3bb8a393642 100644 --- a/kernel/cgroup.c +++ b/kernel/cgroup.c @@ -4052,6 +4052,7 @@ static struct cftype files[] = { }, { .name = "notify_on_release", + .flags = CFTYPE_INSANE, .read_u64 = cgroup_read_notify_on_release, .write_u64 = cgroup_write_notify_on_release, }, @@ -4073,7 +4074,7 @@ static struct cftype files[] = { }, { .name = "release_agent", - .flags = CFTYPE_ONLY_ON_ROOT, + .flags = CFTYPE_INSANE | CFTYPE_ONLY_ON_ROOT, .read_seq_string = cgroup_release_agent_show, .write_string = cgroup_release_agent_write, .max_write_len = PATH_MAX, |