106 lines
3.2 KiB
Diff
106 lines
3.2 KiB
Diff
From: Ben Hutchings <ben@decadent.org.uk>
|
|
Date: Sun, 30 May 2010 22:43:38 +0100
|
|
Subject: [PATCH 1/2] cgroups: Allow memory cgroup support to be included but
|
|
disabled
|
|
|
|
Memory cgroup support has some run-time overhead, so it's useful to
|
|
include it in a distribution kernel without enabling it by default.
|
|
Add a kernel config option to disable it by default and a kernel
|
|
parameter 'cgroup_enable' as the opposite to 'cgroup_disable'.
|
|
|
|
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
|
|
[Bastian Blank: Rename CGROUP_MEM_RES_CTLR_DISABLED to MEMCG_DISABLED]
|
|
---
|
|
Documentation/kernel-parameters.txt | 4 ++--
|
|
init/Kconfig | 8 ++++++++
|
|
kernel/cgroup.c | 20 ++++++++++++++++----
|
|
mm/memcontrol.c | 3 +++
|
|
4 files changed, 29 insertions(+), 6 deletions(-)
|
|
|
|
--- a/Documentation/kernel-parameters.txt
|
|
+++ b/Documentation/kernel-parameters.txt
|
|
@@ -449,8 +449,8 @@ bytes respectively. Such letter suffixes
|
|
ccw_timeout_log [S390]
|
|
See Documentation/s390/CommonIO for details.
|
|
|
|
- cgroup_disable= [KNL] Disable a particular controller
|
|
- Format: {name of the controller(s) to disable}
|
|
+ cgroup_disable= [KNL] Disable/enable a particular controller
|
|
+ cgroup_enable= Format: {name of the controller(s) to disable/enable}
|
|
{Currently supported controllers - "memory"}
|
|
|
|
checkreqprot [SELINUX] Set initial checkreqprot flag value.
|
|
--- a/init/Kconfig
|
|
+++ b/init/Kconfig
|
|
@@ -845,6 +845,14 @@ config MEMCG
|
|
This config option also selects MM_OWNER config option, which
|
|
could in turn add some fork/exit overhead.
|
|
|
|
+config MEMCG_DISABLED
|
|
+ bool "Memory Resource Controller disabled by default"
|
|
+ depends on MEMCG
|
|
+ default n
|
|
+ help
|
|
+ Disable the memory group resource controller unless explicitly
|
|
+ enabled using the kernel parameter "cgroup_enable=memory".
|
|
+
|
|
config MEMCG_SWAP
|
|
bool "Memory Resource Controller Swap Extension"
|
|
depends on MEMCG && SWAP
|
|
--- a/kernel/cgroup.c
|
|
+++ b/kernel/cgroup.c
|
|
@@ -5193,7 +5193,7 @@ static void cgroup_release_agent(struct
|
|
mutex_unlock(&cgroup_mutex);
|
|
}
|
|
|
|
-static int __init cgroup_disable(char *str)
|
|
+static int __init cgroup_set_disabled(char *str, int value)
|
|
{
|
|
int i;
|
|
char *token;
|
|
@@ -5213,17 +5213,29 @@ static int __init cgroup_disable(char *s
|
|
continue;
|
|
|
|
if (!strcmp(token, ss->name)) {
|
|
- ss->disabled = 1;
|
|
- printk(KERN_INFO "Disabling %s control group"
|
|
- " subsystem\n", ss->name);
|
|
+ ss->disabled = value;
|
|
+ printk(KERN_INFO
|
|
+ "%sabling %s control group subsystem\n",
|
|
+ value ? "Dis" : "En", ss->name);
|
|
break;
|
|
}
|
|
}
|
|
}
|
|
return 1;
|
|
}
|
|
+
|
|
+static int __init cgroup_disable(char *str)
|
|
+{
|
|
+ return cgroup_set_disabled(str, 1);
|
|
+}
|
|
__setup("cgroup_disable=", cgroup_disable);
|
|
|
|
+static int __init cgroup_enable(char *str)
|
|
+{
|
|
+ return cgroup_set_disabled(str, 0);
|
|
+}
|
|
+__setup("cgroup_enable=", cgroup_enable);
|
|
+
|
|
/*
|
|
* Functons for CSS ID.
|
|
*/
|
|
--- a/mm/memcontrol.c
|
|
+++ b/mm/memcontrol.c
|
|
@@ -6789,6 +6789,9 @@ static void mem_cgroup_move_task(struct
|
|
|
|
struct cgroup_subsys mem_cgroup_subsys = {
|
|
.name = "memory",
|
|
+#ifdef CONFIG_MEMCG_DISABLED
|
|
+ .disabled = 1,
|
|
+#endif
|
|
.subsys_id = mem_cgroup_subsys_id,
|
|
.css_alloc = mem_cgroup_css_alloc,
|
|
.css_online = mem_cgroup_css_online,
|