diff options
Diffstat (limited to 'mm/memcontrol.c')
-rw-r--r-- | mm/memcontrol.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/mm/memcontrol.c b/mm/memcontrol.c index 435f08dac8bf..861037070f66 100644 --- a/mm/memcontrol.c +++ b/mm/memcontrol.c | |||
@@ -51,6 +51,7 @@ static int really_do_swap_account __initdata = 1; /* for remember boot option*/ | |||
51 | #define do_swap_account (0) | 51 | #define do_swap_account (0) |
52 | #endif | 52 | #endif |
53 | 53 | ||
54 | static DEFINE_MUTEX(memcg_tasklist); /* can be hold under cgroup_mutex */ | ||
54 | 55 | ||
55 | /* | 56 | /* |
56 | * Statistics for memory cgroup. | 57 | * Statistics for memory cgroup. |
@@ -827,7 +828,9 @@ static int __mem_cgroup_try_charge(struct mm_struct *mm, | |||
827 | 828 | ||
828 | if (!nr_retries--) { | 829 | if (!nr_retries--) { |
829 | if (oom) { | 830 | if (oom) { |
831 | mutex_lock(&memcg_tasklist); | ||
830 | mem_cgroup_out_of_memory(mem_over_limit, gfp_mask); | 832 | mem_cgroup_out_of_memory(mem_over_limit, gfp_mask); |
833 | mutex_unlock(&memcg_tasklist); | ||
831 | mem_over_limit->last_oom_jiffies = jiffies; | 834 | mem_over_limit->last_oom_jiffies = jiffies; |
832 | } | 835 | } |
833 | goto nomem; | 836 | goto nomem; |
@@ -2211,10 +2214,12 @@ static void mem_cgroup_move_task(struct cgroup_subsys *ss, | |||
2211 | struct cgroup *old_cont, | 2214 | struct cgroup *old_cont, |
2212 | struct task_struct *p) | 2215 | struct task_struct *p) |
2213 | { | 2216 | { |
2217 | mutex_lock(&memcg_tasklist); | ||
2214 | /* | 2218 | /* |
2215 | * FIXME: It's better to move charges of this process from old | 2219 | * FIXME: It's better to move charges of this process from old |
2216 | * memcg to new memcg. But it's just on TODO-List now. | 2220 | * memcg to new memcg. But it's just on TODO-List now. |
2217 | */ | 2221 | */ |
2222 | mutex_unlock(&memcg_tasklist); | ||
2218 | } | 2223 | } |
2219 | 2224 | ||
2220 | struct cgroup_subsys mem_cgroup_subsys = { | 2225 | struct cgroup_subsys mem_cgroup_subsys = { |