diff options
author | Vladimir Davydov <vdavydov@parallels.com> | 2014-12-12 16:55:13 -0800 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-12-13 12:42:47 -0800 |
commit | 4e701d7b37789d1aeb0015210b373912e5d30733 (patch) | |
tree | 38257203476057e32f32de241f0d6dc2b7e11891 /net | |
parent | 900a38f027b37b55ebe157a0cb31de351b91e4e2 (diff) | |
download | op-kernel-dev-4e701d7b37789d1aeb0015210b373912e5d30733.zip op-kernel-dev-4e701d7b37789d1aeb0015210b373912e5d30733.tar.gz |
memcg: only check memcg_kmem_skip_account in __memcg_kmem_get_cache
__memcg_kmem_get_cache can recurse if it calls kmalloc (which it does if
the cgroup's kmem cache doesn't exist), because kmalloc may call
__memcg_kmem_get_cache internally again. To avoid the recursion, we use
the task_struct->memcg_kmem_skip_account flag.
However, there's no need checking the flag in memcg_kmem_newpage_charge,
because there's no way how this function could result in recursion, if
called from memcg_kmem_get_cache. So let's remove the redundant code.
Signed-off-by: Vladimir Davydov <vdavydov@parallels.com>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Michal Hocko <mhocko@suse.cz>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions