diff options
Diffstat (limited to 'kernel/cgroup_freezer.c')
-rw-r--r-- | kernel/cgroup_freezer.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/kernel/cgroup_freezer.c b/kernel/cgroup_freezer.c index 321e2a007d25..c287627bed3d 100644 --- a/kernel/cgroup_freezer.c +++ b/kernel/cgroup_freezer.c | |||
@@ -173,24 +173,25 @@ static int freezer_can_attach(struct cgroup_subsys *ss, | |||
173 | 173 | ||
174 | /* | 174 | /* |
175 | * Anything frozen can't move or be moved to/from. | 175 | * Anything frozen can't move or be moved to/from. |
176 | * | ||
177 | * Since orig_freezer->state == FROZEN means that @task has been | ||
178 | * frozen, so it's sufficient to check the latter condition. | ||
179 | */ | 176 | */ |
180 | 177 | ||
181 | if (is_task_frozen_enough(task)) | 178 | freezer = cgroup_freezer(new_cgroup); |
179 | if (freezer->state != CGROUP_THAWED) | ||
182 | return -EBUSY; | 180 | return -EBUSY; |
183 | 181 | ||
184 | freezer = cgroup_freezer(new_cgroup); | 182 | rcu_read_lock(); |
185 | if (freezer->state == CGROUP_FROZEN) | 183 | if (__cgroup_freezing_or_frozen(task)) { |
184 | rcu_read_unlock(); | ||
186 | return -EBUSY; | 185 | return -EBUSY; |
186 | } | ||
187 | rcu_read_unlock(); | ||
187 | 188 | ||
188 | if (threadgroup) { | 189 | if (threadgroup) { |
189 | struct task_struct *c; | 190 | struct task_struct *c; |
190 | 191 | ||
191 | rcu_read_lock(); | 192 | rcu_read_lock(); |
192 | list_for_each_entry_rcu(c, &task->thread_group, thread_group) { | 193 | list_for_each_entry_rcu(c, &task->thread_group, thread_group) { |
193 | if (is_task_frozen_enough(c)) { | 194 | if (__cgroup_freezing_or_frozen(c)) { |
194 | rcu_read_unlock(); | 195 | rcu_read_unlock(); |
195 | return -EBUSY; | 196 | return -EBUSY; |
196 | } | 197 | } |