aboutsummaryrefslogtreecommitdiffstats
path: root/mm
diff options
context:
space:
mode:
authorMichal Hocko <mhocko@suse.cz>2013-04-29 18:07:43 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2013-04-29 18:54:34 -0400
commitacb6d558f4c8fbacc8e774c9d7737220a3777882 (patch)
tree862853828c3a666455dc78a6f807e764d8ab5ad1 /mm
parentee5df0570c8af2610b28ab79bd8f8f8195687773 (diff)
memcg: do not check for do_swap_account in mem_cgroup_{read,write,reset}
Since commit 2d11085e404f ("memcg: do not create memsw files if swap accounting is disabled") memsw files are created only if memcg swap accounting is enabled so it doesn't make any sense to check for it explicitly in mem_cgroup_read(), mem_cgroup_write() and mem_cgroup_reset(). Signed-off-by: Michal Hocko <mhocko@suse.cz> Cc: Kamezawa Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> Cc: Tejun Heo <tj@kernel.org> Acked-by: Johannes Weiner <hannes@cmpxchg.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r--mm/memcontrol.c9
1 files changed, 0 insertions, 9 deletions
diff --git a/mm/memcontrol.c b/mm/memcontrol.c
index 2bdac3ececd0..d280016039f2 100644
--- a/mm/memcontrol.c
+++ b/mm/memcontrol.c
@@ -5025,9 +5025,6 @@ static ssize_t mem_cgroup_read(struct cgroup *cont, struct cftype *cft,
5025 type = MEMFILE_TYPE(cft->private); 5025 type = MEMFILE_TYPE(cft->private);
5026 name = MEMFILE_ATTR(cft->private); 5026 name = MEMFILE_ATTR(cft->private);
5027 5027
5028 if (!do_swap_account && type == _MEMSWAP)
5029 return -EOPNOTSUPP;
5030
5031 switch (type) { 5028 switch (type) {
5032 case _MEM: 5029 case _MEM:
5033 if (name == RES_USAGE) 5030 if (name == RES_USAGE)
@@ -5162,9 +5159,6 @@ static int mem_cgroup_write(struct cgroup *cont, struct cftype *cft,
5162 type = MEMFILE_TYPE(cft->private); 5159 type = MEMFILE_TYPE(cft->private);
5163 name = MEMFILE_ATTR(cft->private); 5160 name = MEMFILE_ATTR(cft->private);
5164 5161
5165 if (!do_swap_account && type == _MEMSWAP)
5166 return -EOPNOTSUPP;
5167
5168 switch (name) { 5162 switch (name) {
5169 case RES_LIMIT: 5163 case RES_LIMIT:
5170 if (mem_cgroup_is_root(memcg)) { /* Can't set limit on root */ 5164 if (mem_cgroup_is_root(memcg)) { /* Can't set limit on root */
@@ -5241,9 +5235,6 @@ static int mem_cgroup_reset(struct cgroup *cont, unsigned int event)
5241 type = MEMFILE_TYPE(event); 5235 type = MEMFILE_TYPE(event);
5242 name = MEMFILE_ATTR(event); 5236 name = MEMFILE_ATTR(event);
5243 5237
5244 if (!do_swap_account && type == _MEMSWAP)
5245 return -EOPNOTSUPP;
5246
5247 switch (name) { 5238 switch (name) {
5248 case RES_MAX_USAGE: 5239 case RES_MAX_USAGE:
5249 if (type == _MEM) 5240 if (type == _MEM)